http://trac.webkit.org/changeset/112201
https://bugs.webkit.org/show_bug.cgi?id=82302
Breaks chromium's WebPageSerializerTest.HTMLNodes test
(Requested by pfeldman on #webkit).
Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2012-03-27
* loader/cache/CachedCSSStyleSheet.cpp:
(WebCore::CachedCSSStyleSheet::error):
(WebCore):
* loader/cache/CachedCSSStyleSheet.h:
(CachedCSSStyleSheet):
* loader/cache/CachedFont.cpp:
(WebCore):
(WebCore::CachedFont::error):
* loader/cache/CachedFont.h:
(CachedFont):
* loader/cache/CachedImage.cpp:
(WebCore::CachedImage::error):
* loader/cache/CachedResource.h:
(CachedResource):
* loader/cache/CachedScript.cpp:
(WebCore::CachedScript::error):
(WebCore):
* loader/cache/CachedScript.h:
(CachedScript):
* loader/cache/CachedXSLStyleSheet.cpp:
(WebCore::CachedXSLStyleSheet::error):
(WebCore):
* loader/cache/CachedXSLStyleSheet.h:
(CachedXSLStyleSheet):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@112245
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
+2012-03-27 Sheriff Bot <webkit.review.bot@gmail.com>
+
+ Unreviewed, rolling out r112201.
+ http://trac.webkit.org/changeset/112201
+ https://bugs.webkit.org/show_bug.cgi?id=82302
+
+ Breaks chromium's WebPageSerializerTest.HTMLNodes test
+ (Requested by pfeldman on #webkit).
+
+ * loader/cache/CachedCSSStyleSheet.cpp:
+ (WebCore::CachedCSSStyleSheet::error):
+ (WebCore):
+ * loader/cache/CachedCSSStyleSheet.h:
+ (CachedCSSStyleSheet):
+ * loader/cache/CachedFont.cpp:
+ (WebCore):
+ (WebCore::CachedFont::error):
+ * loader/cache/CachedFont.h:
+ (CachedFont):
+ * loader/cache/CachedImage.cpp:
+ (WebCore::CachedImage::error):
+ * loader/cache/CachedResource.h:
+ (CachedResource):
+ * loader/cache/CachedScript.cpp:
+ (WebCore::CachedScript::error):
+ (WebCore):
+ * loader/cache/CachedScript.h:
+ (CachedScript):
+ * loader/cache/CachedXSLStyleSheet.cpp:
+ (WebCore::CachedXSLStyleSheet::error):
+ (WebCore):
+ * loader/cache/CachedXSLStyleSheet.h:
+ (CachedXSLStyleSheet):
+
2012-03-27 Levi Weintraub <leviw@chromium.org>
LayoutRepainter: Remove unused constructor parameter and update to LayoutUnits
c->setCSSStyleSheet(m_resourceRequest.url(), m_response.url(), m_decoder->encoding().name(), this);
}
+void CachedCSSStyleSheet::error(CachedResource::Status status)
+{
+ setStatus(status);
+ ASSERT(errorOccurred());
+ setLoading(false);
+ checkNotify();
+}
+
bool CachedCSSStyleSheet::canUseSheet(bool enforceMIMEType, bool* hasValidMIMEType) const
{
if (errorOccurred())
virtual void setEncoding(const String&);
virtual String encoding() const;
virtual void data(PassRefPtr<SharedBuffer> data, bool allDataReceived);
+ virtual void error(CachedResource::Status);
+
+ void checkNotify();
private:
bool canUseSheet(bool enforceMIMEType, bool* hasValidMIMEType) const;
virtual PurgePriority purgePriority() const { return PurgeLast; }
protected:
- virtual void checkNotify();
-
RefPtr<TextResourceDecoder> m_decoder;
String m_decodedSheetText;
};
c->fontLoaded(this);
}
+
+void CachedFont::error(CachedResource::Status status)
+{
+ setStatus(status);
+ ASSERT(errorOccurred());
+ setLoading(false);
+ checkNotify();
+}
+
}
virtual void didAddClient(CachedResourceClient*);
virtual void data(PassRefPtr<SharedBuffer> data, bool allDataReceived);
+ virtual void error(CachedResource::Status);
virtual void allClientsRemoved();
+
+ void checkNotify();
+
void beginLoadIfNeeded(CachedResourceLoader* dl);
bool stillNeedsLoad() const { return !m_loadInitiated; }
#endif
private:
- virtual void checkNotify();
FontCustomPlatformData* m_fontData;
bool m_loadInitiated;
void CachedImage::error(CachedResource::Status status)
{
- CachedResource::error(status);
checkShouldPaintBrokenImage();
clear();
+ setStatus(status);
+ ASSERT(errorOccurred());
m_data.clear();
notifyObservers();
+ setLoading(false);
+ checkNotify();
}
void CachedImage::setResponse(const ResourceResponse& response)
double loadFinishTime() const { return m_loadFinishTime; }
protected:
- virtual void checkNotify();
+ void checkNotify();
void setEncodedSize(unsigned);
void setDecodedSize(unsigned);
checkNotify();
}
+void CachedScript::error(CachedResource::Status status)
+{
+ setStatus(status);
+ ASSERT(errorOccurred());
+ setLoading(false);
+ checkNotify();
+}
+
void CachedScript::destroyDecodedData()
{
m_script = String();
virtual void setEncoding(const String&);
virtual String encoding() const;
virtual void data(PassRefPtr<SharedBuffer> data, bool allDataReceived);
+ virtual void error(Status);
virtual void destroyDecodedData();
#if USE(JSC)
c->setXSLStyleSheet(m_resourceRequest.url(), m_response.url(), m_sheet);
}
+void CachedXSLStyleSheet::error(CachedResource::Status status)
+{
+ setStatus(status);
+ ASSERT(errorOccurred());
+ setLoading(false);
+ checkNotify();
+}
+
#endif
}
virtual void setEncoding(const String&);
virtual String encoding() const;
virtual void data(PassRefPtr<SharedBuffer> data, bool allDataReceived);
+ virtual void error(CachedResource::Status);
+
+ void checkNotify();
protected:
- virtual void checkNotify();
-
String m_sheet;
RefPtr<TextResourceDecoder> m_decoder;
};