Diff
Modified: trunk/LayoutTests/ChangeLog (87685 => 87686)
--- trunk/LayoutTests/ChangeLog 2011-05-30 14:13:32 UTC (rev 87685)
+++ trunk/LayoutTests/ChangeLog 2011-05-30 14:50:13 UTC (rev 87686)
@@ -1,3 +1,20 @@
+2011-05-30 Sheriff Bot <[email protected]>
+
+ Unreviewed, rolling out r87683.
+ http://trac.webkit.org/changeset/87683
+ https://bugs.webkit.org/show_bug.cgi?id=61729
+
+ Breaks inspector tests on Chromium (Requested by apavlov on
+ #webkit).
+
+ * http/tests/inspector/inspect-iframe-from-different-domain.html:
+ * inspector/elements/elements-delete-inline-style.html:
+ * inspector/elements/elements-panel-styles.html:
+ * inspector/styles/styles-iframe.html:
+ * inspector/styles/styles-update-from-js-expected.txt:
+ * inspector/styles/styles-update-from-js.html:
+ * inspector/styles/styles-url-linkify.html:
+
2011-05-30 Gabor Rapcsanyi <[email protected]>
Rubber-stamped by Csaba Osztrogonác.
Modified: trunk/LayoutTests/http/tests/inspector/inspect-iframe-from-different-domain.html (87685 => 87686)
--- trunk/LayoutTests/http/tests/inspector/inspect-iframe-from-different-domain.html 2011-05-30 14:13:32 UTC (rev 87685)
+++ trunk/LayoutTests/http/tests/inspector/inspect-iframe-from-different-domain.html 2011-05-30 14:50:13 UTC (rev 87686)
@@ -16,8 +16,7 @@
function test()
{
- InspectorTest.selectNodeWithId("iframe-body");
- InspectorTest.addSniffer(WebInspector.StylesSidebarPane.prototype, "_rebuildUpdate", step1);
+ InspectorTest.selectNodeWithId("iframe-body", step1);
function step1()
{
Modified: trunk/LayoutTests/inspector/elements/elements-delete-inline-style.html (87685 => 87686)
--- trunk/LayoutTests/inspector/elements/elements-delete-inline-style.html 2011-05-30 14:13:32 UTC (rev 87685)
+++ trunk/LayoutTests/inspector/elements/elements-delete-inline-style.html 2011-05-30 14:50:13 UTC (rev 87686)
@@ -7,15 +7,14 @@
function test()
{
WebInspector.showPanel("elements");
- InspectorTest.selectNodeWithId("inspected");
- InspectorTest.addSniffer(WebInspector.StylesSidebarPane.prototype, "_rebuildUpdate", step1);
+ InspectorTest.selectNodeWithId("inspected", step1);
function step1(node)
{
InspectorTest.addResult("Before style property removal:");
InspectorTest.dumpSelectedElementStyles(true);
node.removeAttribute("style");
- InspectorTest.addSniffer(WebInspector.StylesSidebarPane.prototype, "_rebuildUpdate", step2);
+ InspectorTest.runAfterPendingDispatches(step2);
}
function step2()
Modified: trunk/LayoutTests/inspector/elements/elements-panel-styles.html (87685 => 87686)
--- trunk/LayoutTests/inspector/elements/elements-panel-styles.html 2011-05-30 14:13:32 UTC (rev 87685)
+++ trunk/LayoutTests/inspector/elements/elements-panel-styles.html 2011-05-30 14:50:13 UTC (rev 87686)
@@ -8,8 +8,7 @@
function test()
{
- InspectorTest.selectNodeWithId("foo");
- InspectorTest.addSniffer(WebInspector.StylesSidebarPane.prototype, "_rebuildUpdate", step1);
+ InspectorTest.selectNodeWithId("foo", step1);
function step1()
{
Modified: trunk/LayoutTests/inspector/styles/styles-iframe.html (87685 => 87686)
--- trunk/LayoutTests/inspector/styles/styles-iframe.html 2011-05-30 14:13:32 UTC (rev 87685)
+++ trunk/LayoutTests/inspector/styles/styles-iframe.html 2011-05-30 14:50:13 UTC (rev 87686)
@@ -30,8 +30,7 @@
{
InspectorTest.addResult("Main frame style:");
InspectorTest.dumpSelectedElementStyles(true, false, true);
- InspectorTest.selectNodeWithId("iframeBody");
- InspectorTest.addSniffer(WebInspector.StylesSidebarPane.prototype, "_rebuildUpdate", step2);
+ InspectorTest.selectNodeWithId("iframeBody", step2);
}
function step2()
Modified: trunk/LayoutTests/inspector/styles/styles-update-from-js-expected.txt (87685 => 87686)
--- trunk/LayoutTests/inspector/styles/styles-update-from-js-expected.txt 2011-05-30 14:13:32 UTC (rev 87685)
+++ trunk/LayoutTests/inspector/styles/styles-update-from-js-expected.txt 2011-05-30 14:50:13 UTC (rev 87686)
@@ -1,6 +1,9 @@
Tests that changes to an inline style from _javascript_ are reflected in the Styles pane and Elements tree.
-Modified "style" attribute
+
+Running: testInit
+
+Running: testSetStyleAttribute
<div id="container" style="color: #daC0DE; border: 1px solid black;"></div>
[expanded] element.style { ()
color: #DAC0DE;
@@ -19,13 +22,15 @@
border-left-color: black;
-Modified cssText
+
+Running: testSetStyleCSSText
<div id="container" style="color: rgb(192, 255, 238); "></div>
[expanded] element.style { ()
color: #C0FFEE;
-Modified parsed attributes
+
+Running: testSetViaParsedAttributes
<div id="container" style="color: rgb(192, 255, 238); border-top-style: dashed; border-right-style: dashed; border-bottom-style: dashed; border-left-style: dashed; border-top-color: green; border-right-color: green; border-bottom-color: green; border-left-color: green; border-top-width: 3px; border-right-width: 3px; border-bottom-width: 3px; border-left-width: 3px; "></div>
[expanded] element.style { ()
color: #C0FFEE;
Modified: trunk/LayoutTests/inspector/styles/styles-update-from-js.html (87685 => 87686)
--- trunk/LayoutTests/inspector/styles/styles-update-from-js.html 2011-05-30 14:13:32 UTC (rev 87685)
+++ trunk/LayoutTests/inspector/styles/styles-update-from-js.html 2011-05-30 14:50:13 UTC (rev 87686)
@@ -22,36 +22,54 @@
function test()
{
- var sniffCount = 0;
+ InspectorTest.runTestSuite([
- InspectorTest.selectNodeWithId("container", installSniffer);
+ function testInit(next)
+ {
+ InspectorTest.selectNodeWithId("container", next);
+ },
- function installSniffer()
- {
- InspectorTest.addSniffer(WebInspector.StylesSidebarPane.prototype, "_rebuildUpdate", snifferCallback, true);
- InspectorTest.evaluateInPage("modifyStyleAttribute()");
- }
+ function testSetStyleAttribute(next)
+ {
+ var updateCount = 0;
+ function callback()
+ {
+ // Expect two updates while setting attribute.
+ if (++updateCount === 2) {
+ dumpAttributeAndStyles();
+ next();
+ }
+ }
+ InspectorTest.addSniffer(WebInspector.StylesSidebarPane.prototype, "_rebuildUpdate", callback, true);
+ InspectorTest.evaluateInPage("modifyStyleAttribute()");
+ },
- function snifferCallback()
- {
- switch (++sniffCount) {
- case 1:
- InspectorTest.addResult("Modified \"style\" attribute");
+ function testSetStyleCSSText(next)
+ {
+ var updateCount = 0;
+ function callback()
+ {
+ // Expect two updates while setting css text.
+ if (++updateCount === 2) {
+ dumpAttributeAndStyles();
+ next();
+ }
+ }
+ InspectorTest.addSniffer(WebInspector.StylesSidebarPane.prototype, "_rebuildUpdate", callback, true);
+ InspectorTest.evaluateInPage("modifyCSSText()", callback);
+ },
+
+ function testSetViaParsedAttributes(next)
+ {
+ function callback()
+ {
dumpAttributeAndStyles();
- InspectorTest.evaluateInPage("modifyCSSText()");
- break;
- case 2:
- InspectorTest.addResult("Modified cssText");
- dumpAttributeAndStyles();
- InspectorTest.evaluateInPage("modifyParsedAttributes()");
- break;
- case 3:
- InspectorTest.addResult("Modified parsed attributes");
- dumpAttributeAndStyles();
- InspectorTest.completeTest();
- break;
+ next();
+ }
+ InspectorTest.addSniffer(WebInspector.StylesSidebarPane.prototype, "_rebuildUpdate", callback);
+ InspectorTest.evaluateInPage("modifyParsedAttributes()");
}
- }
+ ]);
function dumpAttributeAndStyles()
{
Modified: trunk/LayoutTests/inspector/styles/styles-url-linkify.html (87685 => 87686)
--- trunk/LayoutTests/inspector/styles/styles-url-linkify.html 2011-05-30 14:13:32 UTC (rev 87685)
+++ trunk/LayoutTests/inspector/styles/styles-url-linkify.html 2011-05-30 14:50:13 UTC (rev 87686)
@@ -52,9 +52,7 @@
{
InspectorTest.addResult("Link for a URI from CSS document:");
dumpHref();
- InspectorTest.selectNodeWithId("iframed");
- InspectorTest.addSniffer(WebInspector.StylesSidebarPane.prototype, "_rebuildUpdate", step2);
-
+ InspectorTest.selectNodeWithId("iframed", step2);
}
function step2()
Modified: trunk/Source/WebCore/ChangeLog (87685 => 87686)
--- trunk/Source/WebCore/ChangeLog 2011-05-30 14:13:32 UTC (rev 87685)
+++ trunk/Source/WebCore/ChangeLog 2011-05-30 14:50:13 UTC (rev 87686)
@@ -1,3 +1,15 @@
+2011-05-30 Sheriff Bot <[email protected]>
+
+ Unreviewed, rolling out r87683.
+ http://trac.webkit.org/changeset/87683
+ https://bugs.webkit.org/show_bug.cgi?id=61729
+
+ Breaks inspector tests on Chromium (Requested by apavlov on
+ #webkit).
+
+ * inspector/front-end/StylesSidebarPane.js:
+ (WebInspector.StylesSidebarPane.prototype.update):
+
2011-05-25 Alexander Pavlov <[email protected]>
Reviewed by Yury Semikhatsky.
Modified: trunk/Source/WebCore/inspector/front-end/StylesSidebarPane.js (87685 => 87686)
--- trunk/Source/WebCore/inspector/front-end/StylesSidebarPane.js 2011-05-30 14:13:32 UTC (rev 87685)
+++ trunk/Source/WebCore/inspector/front-end/StylesSidebarPane.js 2011-05-30 14:50:13 UTC (rev 87686)
@@ -235,21 +235,10 @@
this._refreshUpdate(node, computedStyle, editedSection);
}
- function reloadAllStyles()
- {
- delete this._reloadAllStylesTimer;
- WebInspector.cssModel.getStylesAsync(this._allStylesNodeId, stylesCallback.bind(this))
- }
-
if (refresh)
WebInspector.cssModel.getComputedStyleAsync(node.id, computedStyleCallback.bind(this));
- else {
- // Always refresh the node to request styles for.
- this._allStylesNodeId = node.id;
- if ("_reloadAllStylesTimer" in this)
- return;
- this._reloadAllStylesTimer = setTimeout(reloadAllStyles.bind(this), 0);
- }
+ else
+ WebInspector.cssModel.getStylesAsync(node.id, stylesCallback.bind(this));
},
_refreshUpdate: function(node, computedStyle, editedSection)
