Attachment 511912 Details for Bug 626016

[patch] Test change

test-qt (text/plain), 1.78 KB, created by :Felipe Gomes (needinfo for replies!)

(hide)

Description:

Filename:

MIME Type:

Creator: :Felipe Gomes (needinfo for replies!)

Size: 1.78 KB

patch

obsolete

>diff --git a/modules/plugin/test/mochitest/test_visibility.html b/modules/plugin/test/mochitest/test_visibility.html
>--- a/modules/plugin/test/mochitest/test_visibility.html
>+++ b/modules/plugin/test/mochitest/test_visibility.html
>@@ -17,51 +17,44 @@
>   <script type="application/javascript">
>   SimpleTest.waitForExplicitFinish();
> 
>   var didPaint = function() {};
> 
>   function startTest() {
>     netscape.security.PrivilegeManager.enablePrivilege("UniversalXPConnect");
> 
>-    if (SimpleTest.testPluginIsOOP()) {
>-      if (p.getPaintCount() < 1) {
>-        setTimeout(startTest, 0);
>-        return;
>-      }
>-    }
>-
>     didPaint = function() {
>       ok(false, "Plugin should not paint until it is visible!");
>     };
> 
>     ok(!p.isVisible(), "Plugin should not be visible.");
>-    paintCountIs(p, 0, "Plugin should not have painted.");
>+    is(p.getPaintCount(), 0, "Plugin should not have painted.");
> 
>     didPaint = part2;
> 
>     p.style.visibility = 'visible';
>   }
> 
>   function part2() {
>     ok(p.isVisible(), "Plugin should now be visible.");
>-    paintCountIs(p, 1, "Plugin should have painted once.");
>+    is(p.getPaintCount(), 1, "Plugin should have painted once.");
> 
>     didPaint = part3;
> 
>     p.setColor('FF0000FF'); // this causes an invalidate/repaint
>   }
> 
>   const kTimeout = 5000; // 5 seconds
>   var part4GiveUp;
>   var part4Interval;
> 
>   function part3() {
>     ok(p.isVisible(), "Plugin should still be visible.");
>-    paintCountIs(p, 2, "Plugin should have painted twice.");
>+    is(p.getPaintCount(), 2, "Plugin should have painted twice.");
> 
>     didPaint = function() {
>       ok(false, "Plugin should not paint when it is invisible.");
>     };
> 
>     p.style.visibility = 'hidden';
> 
>     part4GiveUp = Date.now() + kTimeout;

Actions: View | Diff | Review

Attachments on bug 626016: 511844 | 511912 | 512395