Title: [135316] trunk/Source
- Revision
- 135316
- Author
- t...@chromium.org
- Date
- 2012-11-20 14:50:37 -0800 (Tue, 20 Nov 2012)
Log Message
When calling DocumentStyleSheetCollection::addUserSheet, pass in a user sheet
https://bugs.webkit.org/show_bug.cgi?id=102835
Reviewed by Ojan Vafai.
After r135082, Chromium browser_tests were triggering the ASSERT in
StyleResolver::collectRulesFromUserStyleSheets. Add an ASSERT that will
trigger earlier and make it clear in the Chromium code that we're always
inserting user level styles.
Source/WebCore:
No new tests, no behavior change except no longer triggering the StyleResolver ASSERT
in Chromium browser_tests.
* dom/DocumentStyleSheetCollection.cpp:
(WebCore::DocumentStyleSheetCollection::addUserSheet):
Source/WebKit/chromium:
* src/WebDocument.cpp:
(WebKit::WebDocument::insertUserStyleSheet):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (135315 => 135316)
--- trunk/Source/WebCore/ChangeLog 2012-11-20 22:41:29 UTC (rev 135315)
+++ trunk/Source/WebCore/ChangeLog 2012-11-20 22:50:37 UTC (rev 135316)
@@ -1,3 +1,21 @@
+2012-11-20 Tony Chang <t...@chromium.org>
+
+ When calling DocumentStyleSheetCollection::addUserSheet, pass in a user sheet
+ https://bugs.webkit.org/show_bug.cgi?id=102835
+
+ Reviewed by Ojan Vafai.
+
+ After r135082, Chromium browser_tests were triggering the ASSERT in
+ StyleResolver::collectRulesFromUserStyleSheets. Add an ASSERT that will
+ trigger earlier and make it clear in the Chromium code that we're always
+ inserting user level styles.
+
+ No new tests, no behavior change except no longer triggering the StyleResolver ASSERT
+ in Chromium browser_tests.
+
+ * dom/DocumentStyleSheetCollection.cpp:
+ (WebCore::DocumentStyleSheetCollection::addUserSheet):
+
2012-11-20 Bear Travis <betra...@adobe.com>
[CSS Exclusions] Support outside-shape value on shape-inside
Modified: trunk/Source/WebCore/dom/DocumentStyleSheetCollection.cpp (135315 => 135316)
--- trunk/Source/WebCore/dom/DocumentStyleSheetCollection.cpp 2012-11-20 22:41:29 UTC (rev 135315)
+++ trunk/Source/WebCore/dom/DocumentStyleSheetCollection.cpp 2012-11-20 22:50:37 UTC (rev 135316)
@@ -193,6 +193,7 @@
void DocumentStyleSheetCollection::addUserSheet(PassRefPtr<StyleSheetContents> userSheet)
{
+ ASSERT(userSheet->isUserStyleSheet());
m_userStyleSheets.append(CSSStyleSheet::create(userSheet, m_document));
m_document->styleResolverChanged(RecalcStyleImmediately);
}
Modified: trunk/Source/WebKit/chromium/ChangeLog (135315 => 135316)
--- trunk/Source/WebKit/chromium/ChangeLog 2012-11-20 22:41:29 UTC (rev 135315)
+++ trunk/Source/WebKit/chromium/ChangeLog 2012-11-20 22:50:37 UTC (rev 135316)
@@ -1,3 +1,18 @@
+2012-11-20 Tony Chang <t...@chromium.org>
+
+ When calling DocumentStyleSheetCollection::addUserSheet, pass in a user sheet
+ https://bugs.webkit.org/show_bug.cgi?id=102835
+
+ Reviewed by Ojan Vafai.
+
+ After r135082, Chromium browser_tests were triggering the ASSERT in
+ StyleResolver::collectRulesFromUserStyleSheets. Add an ASSERT that will
+ trigger earlier and make it clear in the Chromium code that we're always
+ inserting user level styles.
+
+ * src/WebDocument.cpp:
+ (WebKit::WebDocument::insertUserStyleSheet):
+
2012-11-20 Istiaque Ahmed <lazy...@chromium.org>
Provide page/window coordinates to plugin's local coordinates translation in WebPluginContainer.
Modified: trunk/Source/WebKit/chromium/src/WebDocument.cpp (135315 => 135316)
--- trunk/Source/WebKit/chromium/src/WebDocument.cpp 2012-11-20 22:41:29 UTC (rev 135315)
+++ trunk/Source/WebKit/chromium/src/WebDocument.cpp 2012-11-20 22:50:37 UTC (rev 135316)
@@ -194,12 +194,13 @@
return WebDocumentType(constUnwrap<Document>()->doctype());
}
-void WebDocument::insertUserStyleSheet(const WebString& sourceCode, UserStyleLevel level)
+void WebDocument::insertUserStyleSheet(const WebString& sourceCode, UserStyleLevel)
{
RefPtr<Document> document = unwrap<Document>();
+ // FIXME: We currently ignore the passed in UserStyleLevel. http://crbug.com/162096
RefPtr<StyleSheetContents> parsedSheet = StyleSheetContents::create(document.get());
- parsedSheet->setIsUserStyleSheet(level == UserStyleUserLevel);
+ parsedSheet->setIsUserStyleSheet(true);
parsedSheet->parseString(sourceCode);
document->styleSheetCollection()->addUserSheet(parsedSheet.release());
}
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo/webkit-changes
