diff options
| author | Steve Block <steveblock@google.com> | 2011-06-08 08:26:01 -0700 |
|---|---|---|
| committer | Android (Google) Code Review <android-gerrit@google.com> | 2011-06-08 08:26:01 -0700 |
| commit | 3742ac093d35d923c81693096ab6671e9b147700 (patch) | |
| tree | c2add9100f789dad45ef1ec5328bddde02c47a4c /Source/WebCore/html/HTMLLinkElement.cpp | |
| parent | 901401d90459bc22580842455d4588b9a697514d (diff) | |
| parent | e5926f4a0d6adc9ad4a75824129f117181953560 (diff) | |
| download | external_webkit-3742ac093d35d923c81693096ab6671e9b147700.zip external_webkit-3742ac093d35d923c81693096ab6671e9b147700.tar.gz external_webkit-3742ac093d35d923c81693096ab6671e9b147700.tar.bz2 | |
Merge changes I55c6d71a,Ifb3277d4,Ia1b847a2,I7ba9cf3f,Ida2b2a8a,I1280ec90,I72f818d5,I2e3b588b,I9a4e6289,Ia724c78b,Icd8612c8,Ie31b15d7,Ie125edae,I77941a88,I89dae78b,I3516e5ca,I1a4c17b5,I2c4ecc1a,I9c8e6537,Ifac13115,Ie1f80e09,Ia541ed77,I60ce9d78
* changes:
Merge WebKit at r82507: Update ThirdPartyProject.prop
Merge WebKit at r82507: Cherry-pick change r88166 to add INSPECTOR guards to ScriptProfiler
Merge WebKit at r82507: Work around a V8 bug
Merge WebKit at r82507: JNIType renamed to JavaType
Merge WebKit at r82507: IconDatabaseClient interface expanded
Merge WebKit at r82507: Don't use new loss-free code path in HTMLCanvasElement::toDataURL()
Merge WebKit at r82507: IcondDatabaseBase::iconForPageURL() renamed
Merge WebKit at r82507: IconDatabaseBase::Open() signature changed
Merge WebKit at r82507: Node::isContentEditable() renamed
Merge WebKit at r82507: Use icon database through IconDatabaseBase
Merge WebKit at r82507: toInputElement() is now a member of Node
Merge WebKit at r82507: FrameLoaderClient::objectContentType() signature changed
Merge WebKit at r82507: StringImpl::computeHash() removed
Merge WebKit at r82507: Stub out FontPlatformData::setOrientation()
Merge WebKit at r82507: Path::strokeBoundingRect() is now const
Merge WebKit at r82507: Add missing UnusedParam.h include in ApplicationCacheGroup.cpp
Merge WebKit at r82507: Continue to use Android's version of FontPlatformData.h
Merge WebKit at r82507: Update signature of FontCustomPlatformData::fontPlatformData()
Merge WebKit at r82507: Fix conflicts due to JNI refactoring
Merge WebKit at r82507: Fix conflicts due to new StorageTracker
Merge WebKit at r82507: Fix conflicts
Merge WebKit at r82507: Fix makefiles
Merge WebKit at r82507: Initial merge by git
Diffstat (limited to 'Source/WebCore/html/HTMLLinkElement.cpp')
| -rw-r--r-- | Source/WebCore/html/HTMLLinkElement.cpp | 39 |
1 files changed, 28 insertions, 11 deletions
diff --git a/Source/WebCore/html/HTMLLinkElement.cpp b/Source/WebCore/html/HTMLLinkElement.cpp index 04cfda4..67262d9 100644 --- a/Source/WebCore/html/HTMLLinkElement.cpp +++ b/Source/WebCore/html/HTMLLinkElement.cpp @@ -145,6 +145,8 @@ void HTMLLinkElement::parseMappedAttribute(Attribute* attr) #if ENABLE(LINK_PREFETCH) else if (attr->name() == onloadAttr) setAttributeEventListener(eventNames().loadEvent, createAttributeEventListener(this, attr)); + else if (attr->name() == onerrorAttr) + setAttributeEventListener(eventNames().errorEvent, createAttributeEventListener(this, attr)); #endif else { if (attr->name() == titleAttr && m_sheet) @@ -203,6 +205,17 @@ void HTMLLinkElement::tokenizeRelAttribute(const AtomicString& rel, RelAttribute } } +bool HTMLLinkElement::checkBeforeLoadEvent() +{ + RefPtr<Document> originalDocument = document(); + if (!dispatchBeforeLoadEvent(m_url)) + return false; + // A beforeload handler might have removed us from the document or changed the document. + if (!inDocument() || document() != originalDocument) + return false; + return true; +} + void HTMLLinkElement::process() { if (!inDocument() || m_isInShadowTree) { @@ -214,8 +227,11 @@ void HTMLLinkElement::process() // IE extension: location of small icon for locationbar / bookmarks // We'll record this URL per document, even if we later only use it in top level frames - if (m_relAttribute.m_isIcon && m_url.isValid() && !m_url.isEmpty()) + if (m_relAttribute.m_isIcon && m_url.isValid() && !m_url.isEmpty()) { + if (!checkBeforeLoadEvent()) + return; document()->setIconURL(m_url.string(), type); + } #ifdef ANDROID_APPLE_TOUCH_ICON if ((m_relAttribute.m_isTouchIcon || m_relAttribute.m_isPrecomposedTouchIcon) && m_url.isValid() @@ -235,6 +251,8 @@ void HTMLLinkElement::process() #if ENABLE(LINK_PREFETCH) if (m_relAttribute.m_isLinkPrefetch && m_url.isValid() && document()->frame()) { + if (!checkBeforeLoadEvent()) + return; m_cachedLinkPrefetch = document()->cachedResourceLoader()->requestLinkPrefetch(m_url); if (m_cachedLinkPrefetch) m_cachedLinkPrefetch->addClient(this); @@ -256,11 +274,7 @@ void HTMLLinkElement::process() m_cachedSheet = 0; } - RefPtr<Document> originalDocument = document(); - if (!dispatchBeforeLoadEvent(m_url)) - return; - // A beforeload handler might have removed us from the document or changed the document. - if (!inDocument() || document() != originalDocument) + if (!checkBeforeLoadEvent()) return; m_loading = true; @@ -413,16 +427,19 @@ bool HTMLLinkElement::isLoading() const void HTMLLinkElement::onloadTimerFired(Timer<HTMLLinkElement>* timer) { ASSERT_UNUSED(timer, timer == &m_onloadTimer); - dispatchEvent(Event::create(eventNames().loadEvent, false, false)); + if (m_cachedLinkPrefetch->errorOccurred()) + dispatchEvent(Event::create(eventNames().errorEvent, false, false)); + else + dispatchEvent(Event::create(eventNames().loadEvent, false, false)); + + m_cachedLinkPrefetch->removeClient(this); + m_cachedLinkPrefetch = 0; } void HTMLLinkElement::notifyFinished(CachedResource* resource) { m_onloadTimer.startOneShot(0); - if (m_cachedLinkPrefetch.get() == resource) { - m_cachedLinkPrefetch->removeClient(this); - m_cachedLinkPrefetch = 0; - } + ASSERT(m_cachedLinkPrefetch.get() == resource); } #endif |
