Attachment 519316 Details for Bug 614643

[patch] Patch (v1)

614643.patch (text/plain), 2.63 KB, created by (no longer active)

(hide)

Description:

Filename:

MIME Type:

Creator: (no longer active)

Size: 2.63 KB

patch

obsolete

># HG changeset patch
># Parent e687f3b2ef778c8a7ea338cd9fc75fddaa8c36d6
># User Ehsan Akhgari <ehsan@mozilla.com>
>Bug 614643 - Cause a repaint no matter what so that the test becomes more robust
>
>
>diff --git a/toolkit/content/tests/browser/browser_bug295977_autoscroll_overflow.js b/toolkit/content/tests/browser/browser_bug295977_autoscroll_overflow.js
>--- a/toolkit/content/tests/browser/browser_bug295977_autoscroll_overflow.js
>+++ b/toolkit/content/tests/browser/browser_bug295977_autoscroll_overflow.js
>@@ -66,17 +66,16 @@ function test()
>      * so request and force redraws to get the chance to check for scrolling at
>      * all.
>      */
>     window.mozRequestAnimationFrame();
>   }
> 
>   waitForExplicitFinish();
>   var dataUri = 'data:text/html,<body><style type="text/css">div { display: inline-block; }</style>\
>-    <div id="forceredraw" style="height: 1px"></div>\
>     <div id="a" style="width: 100px; height: 100px; overflow: hidden;"><div style="width: 200px; height: 200px;"></div></div>\
>     <div id="b" style="width: 100px; height: 100px; overflow: auto;"><div style="width: 200px; height: 200px;"></div></div>\
>     <div id="c" style="width: 100px; height: 100px; overflow-x: auto; overflow-y: hidden;"><div style="width: 200px; height: 200px;"></div></div>\
>     <div id="d" style="width: 100px; height: 100px; overflow-y: auto; overflow-x: hidden;"><div style="width: 200px; height: 200px;"></div></div>\
>     <select id="e" style="width: 100px; height: 100px;" multiple="multiple"><option>aaaaaaaaaaaaaaaaaaaaaaaa</option><option>a</option><option>a</option>\
>     <option>a</option><option>a</option><option>a</option><option>a</option><option>a</option><option>a</option><option>a</option>\
>     <option>a</option><option>a</option><option>a</option><option>a</option><option>a</option><option>a</option><option>a</option></select>\
>     <select id="f" style="width: 100px; height: 100px;"><option>a</option><option>aaaaaaaaaaaaaaaaaaaaaaaaaaaaaa</option><option>a</option>\
>@@ -93,17 +92,17 @@ function test()
>     gBrowser.selectedBrowser.removeEventListener("pageshow", onLoad, false);
>     waitForFocus(onFocus, content);
>   }
> 
>   function onFocus() {
>     doc = gBrowser.contentDocument;
>     // force redraws, so we actually get AfterPaint events
>     window.addEventListener("MozBeforePaint", function(ev) {
>-      doc.getElementById("forceredraw").style.left = ev.timeStamp % 100;
>+      doc.body.appendChild(doc.createTextNode('.'));
>     }, false);
>     nextTest();
>   }
> 
>   function endTest() {
>     // restore the changed prefs
>     if (Services.prefs.prefHasUserValue(kPrefName_AutoScroll))
>       Services.prefs.clearUserPref(kPrefName_AutoScroll);

Flags:

neil: review+

Actions: View | Diff | Review

Attachments on bug 614643: 519316 | 520375