Bug 1518633 part 15 - Remove unneeded mouse click test_bug633762. r=mattwoodrow
authorRyan Hunt <rhunt@eqrion.net>
Thu, 10 Jan 2019 18:20:39 -0600
changeset 510499 ad851d4345c08f7e0e5d5578652004194a6e667f
parent 510498 e31c9494b9a6472aa391fc52a7b503562859a3a6
child 510500 b62455787e3f0de267d2bd99764e633ac470df72
push id10547
push userffxbld-merge
push dateMon, 21 Jan 2019 13:03:58 +0000
treeherdermozilla-beta@24ec1916bffe [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmattwoodrow
bugs1518633, 633762
milestone66.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 1518633 part 15 - Remove unneeded mouse click test_bug633762. r=mattwoodrow This mouse click seems superfluous, as window.focus() is called immediately after. In addition, this click is somehow causing a page up scroll, as it's clicking a slider frame. This causes the test to fail with scroll anchoring enabled, for some reason. Removing this click seems to be the easiest solution, as it doesn't seem intentional. Differential Revision: https://phabricator.services.mozilla.com/D16276
layout/generic/test/test_bug633762.html
--- a/layout/generic/test/test_bug633762.html
+++ b/layout/generic/test/test_bug633762.html
@@ -23,18 +23,17 @@ var doc;
 function runTests() {
   var i = document.getElementById("i");
   doc = i.contentDocument;
   var win = i.contentWindow;
   // set display none on b
   doc.getElementById("b").style.display = "none";
   // flush layout
   doc.documentElement.offsetLeft;
-  // click in middle of iframe document to give it focus
-  synthesizeMouseAtCenter(i, {}, win);
+  // focus on the iframe
   win.focus();
   // record scrolltop
   scrollTopBefore = doc.body.scrollTop;
   // send up arrow key event
   sendKey("UP");
   
   window.requestAnimationFrame(finish);
 }