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 292705 b71d47c19e79cb7375a388b2c26391de878a4503
parent 292704 6915c6f22667e848e00db978e34924723c83b3f6
child 292706 80051b8b7bf49e9750a9dc2ba1c8883632100c68
push id74957
push usertnikkel@gmail.com
push dateTue, 12 Apr 2016 03:19:57 +0000
treeherdermozilla-inbound@80051b8b7bf4 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersted
bugs1199023, 1177018
milestone48.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
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">