Bug 1199023. Remove 'scrolling="no"' from iframe in which mochitest-chrome tests are run. r=ted
authorTimothy Nikkel <tnikkel@gmail.com>
Wed, 23 Sep 2015 01:29:37 -0500
changeset 316495 b71d47c19e79cb7375a388b2c26391de878a4503
parent 316494 6915c6f22667e848e00db978e34924723c83b3f6
child 316496 80051b8b7bf49e9750a9dc2ba1c8883632100c68
push id9480
push userjlund@mozilla.com
push dateMon, 25 Apr 2016 17:12:58 +0000
treeherdermozilla-aurora@0d6a91c76a9e [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersted
bugs1199023, 1177018
milestone48.0a1
Bug 1199023. Remove 'scrolling="no"' from iframe in which mochitest-chrome tests are run. r=ted This makes the iframe more like the iframe that houses the root content document, and more like almost every other iframe. This does cause real differences in practice when testing, eg https://bugzilla.mozilla.org/show_bug.cgi?id=1177018#c3
testing/mochitest/harness.xul
--- a/testing/mochitest/harness.xul
+++ b/testing/mochitest/harness.xul
@@ -79,17 +79,17 @@ function linkAndHookup(links) {
           <h2 id="fail">Todo: <span id="todo-count">0</span></h2>
         </div>
         <div class="clear"></div>
         <div id="current-test">
           <b>Currently Executing: <span id="current-test-path">_</span></b>
         </div>
         <div class="clear"></div>
         <div class="frameholder">
-          <iframe type="content" scrolling="no" id="testframe" width="550" height="350"></iframe>
+          <iframe type="content" id="testframe" width="550" height="350"></iframe>
         </div>
         <div class="clear"></div>
         <div class="toggle">
           <a href="#" id="toggleNonTests">Show Non-Tests</a>
           <br />
         </div>
         <div id="wrapper">
           <table cellpadding="0" cellspacing="0">