Bug 632048 - Update auto-hide bookmarks popup test [r=mbrubeck]
authorWes Johnston <wjohnston@mozilla.com>
Thu, 24 Mar 2011 10:42:03 -0700
changeset 67534 3333554d36df9d5e19a39147e63e3cc67b25ea9d
parent 67533 4fed5e3c56291a09d33eecfc108d798a2e34f5f6
child 67535 bcaa623a76961dd178763dd276bd0bfd0072543b
push id1
push userroot
push dateTue, 26 Apr 2011 22:38:44 +0000
treeherdermozilla-beta@bfdb6e623a36 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmbrubeck
bugs632048
Bug 632048 - Update auto-hide bookmarks popup test [r=mbrubeck]
mobile/chrome/tests/browser_bookmarks_star.js
--- a/mobile/chrome/tests/browser_bookmarks_star.js
+++ b/mobile/chrome/tests/browser_bookmarks_star.js
@@ -64,28 +64,22 @@ gTests.push({
   onPageLoad: function() {
     let starbutton = document.getElementById("tool-star");
     starbutton.click();
 
     waitFor(gCurrentTest.onPopupReady1, function() { return document.getElementById("bookmark-popup").hidden == false; });
   },
 
   onPopupReady1: function() {
-    // Popup should auto-hide after 2 seconds on the initial bookmark with star
+    // Popup should no longer auto-hide, see bug 63204
     setTimeout(gCurrentTest.onPopupGone, 3000);
   },
 
   onPopupGone: function() {
-    // Make sure it's hidden again
-    is(document.getElementById("bookmark-popup").hidden, true, "Bookmark popup should be auto-hidden");
-
-    // Let's make it appear again and continue the test
-    let starbutton = document.getElementById("tool-star");
-    starbutton.click();
-
+    is(document.getElementById("bookmark-popup").hidden, false, "Bookmark popup should not be auto-hidden");
     waitFor(gCurrentTest.onPopupReady2, function() { return document.getElementById("bookmark-popup").hidden == false; });
   },
 
   onPopupReady2: function() {
     // Let's make it disappear again by clicking the star again
     var starbutton = document.getElementById("tool-star");
     starbutton.click();