Bug 1260630 - Don't use a CPOW to change style in content in browser_computed_refresh-on-style-change_01.js. r=bgrins, a=test-only
authorPatrick Brosset <pbrosset@mozilla.com>
Fri, 13 May 2016 12:48:18 +0200
changeset 332936 7a3a4f2cd15ac522e45c387d5b0101b5386070cb
parent 332935 6b106532d75033e35afa780800f6efe2d11b0074
child 332937 360e90227281e8675e72db6a1eda64c29a3ee77c
push id6048
push userkmoir@mozilla.com
push dateMon, 06 Jun 2016 19:02:08 +0000
treeherdermozilla-beta@46d72a56c57d [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersbgrins, test-only
bugs1260630
milestone48.0a2
Bug 1260630 - Don't use a CPOW to change style in content in browser_computed_refresh-on-style-change_01.js. r=bgrins, a=test-only MozReview-Commit-ID: 5tRJTzQLl2V
devtools/client/inspector/computed/test/browser_computed_refresh-on-style-change_01.js
--- a/devtools/client/inspector/computed/test/browser_computed_refresh-on-style-change_01.js
+++ b/devtools/client/inspector/computed/test/browser_computed_refresh-on-style-change_01.js
@@ -4,28 +4,27 @@
 
 "use strict";
 
 // Tests that the computed view refreshes when the current node has its style
 // changed.
 
 const TEST_URI = "<div id='testdiv' style='font-size:10px;'>Test div!</div>";
 
-add_task(function*() {
+add_task(function* () {
   yield addTab("data:text/html;charset=utf-8," + encodeURIComponent(TEST_URI));
-  let {inspector, view} = yield openComputedView();
+  let {inspector, view, testActor} = yield openComputedView();
   yield selectNode("#testdiv", inspector);
 
   let fontSize = getComputedViewPropertyValue(view, "font-size");
   is(fontSize, "10px", "The computed view shows the right font-size");
 
   info("Changing the node's style and waiting for the update");
   let onUpdated = inspector.once("computed-view-refreshed");
-  // FIXME: use the testActor to set style on the node.
-  content.document.querySelector("#testdiv")
-                  .style.cssText = "font-size: 15px; color: red;";
+  yield testActor.setAttribute("#testdiv", "style",
+                               "font-size: 15px; color: red;");
   yield onUpdated;
 
   fontSize = getComputedViewPropertyValue(view, "font-size");
   is(fontSize, "15px", "The computed view shows the updated font-size");
   let color = getComputedViewPropertyValue(view, "color");
   is(color, "rgb(255, 0, 0)", "The computed view also shows the color now");
 });