Bug 1199023. Remove 'scrolling="no"' from iframe in which mochitest-chrome tests are run. r=ted
☠☠ backed out by 8803d34bd9f8 ☠ ☠
authorTimothy Nikkel <tnikkel@gmail.com>
Wed, 23 Sep 2015 01:29:37 -0500
changeset 292540 d7d0413af62bb490f7d9386652eeea8bea7b81be
parent 292539 3bcd4e72fab8ef9f436d0a7bd03afde61e69db32
child 292541 5192b8cce2d9b1c4c1c2bd28dadef59703d74cfc
push id74874
push usertnikkel@gmail.com
push dateSun, 10 Apr 2016 20:35:24 +0000
treeherdermozilla-inbound@d7d0413af62b [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">