Bug 852489. Part 0: Disable part of testSystemPages for intermittent failures. r=gbrown
authorRobert O'Callahan <robert@ocallahan.org>
Thu, 28 Mar 2013 23:58:30 +1300
changeset 126681 2e930513d923b81fadab2a78ce566926b94bd139
parent 126680 4a57008543aa8cbb0b6072b0c7c166b57249c3d2
child 126682 2aa2721736af1a7a1815509dafd98d0b41947d46
push idunknown
push userunknown
push dateunknown
reviewersgbrown
bugs852489
milestone22.0a1
Bug 852489. Part 0: Disable part of testSystemPages for intermittent failures. r=gbrown
mobile/android/base/tests/testSystemPages.java.in
--- a/mobile/android/base/tests/testSystemPages.java.in
+++ b/mobile/android/base/tests/testSystemPages.java.in
@@ -25,30 +25,34 @@ public class testSystemPages extends Pix
         String urls [] = {  "about:firefox", "about:rights", "about:home", "about:addons", "about:downloads", "about:buildconfig", "about:"};
         String expectedUrls [] = { "about:apps", "about:downloads", "about:addons", "about:"};
         String menuItems [] = { "Apps", "Downloads", "Add-ons", "Settings"};
 
         /* This first section loads system pages from the awesome bar
            and checks that the pages are loaded in the same tab */
         checkUrl(urls);
 
-        /* Verify that the search field is not in the focus by pressing back. That will load the previous
-           about: page if there is no the keyboard to dismiss, meaning that the search field was not in focus */
-        loadUrl("about:config");
+        // Following part of the test disabled due to intermittent failures in bug 797615
+        // made permanent in bug 852489
+        if (false) {
+            /* Verify that the search field is not in the focus by pressing back. That will load the previous
+               about: page if there is no the keyboard to dismiss, meaning that the search field was not in focus */
+            loadUrl("about:config");
 
-        paintExpecter = mActions.expectPaint();// Set up listener to catch the page load
+            paintExpecter = mActions.expectPaint();// Set up listener to catch the page load
 
-        // Press back to verify if the keyboard is dismissed or the previous about: page loads
-        mActions.sendSpecialKey(Actions.SpecialKey.BACK);
-        PaintedSurface painted = waitForPaint(paintExpecter);
+            // Press back to verify if the keyboard is dismissed or the previous about: page loads
+            mActions.sendSpecialKey(Actions.SpecialKey.BACK);
+            PaintedSurface painted = waitForPaint(paintExpecter);
 
-        verifyUrl("about:"); // Verify that the previous about: page is loaded, meaning no keyboard was present
+            verifyUrl("about:"); // Verify that the previous about: page is loaded, meaning no keyboard was present
 
-        // The second section loads system pages from Firefox menu in different tabs
-        loadFromMenu(menuItems, expectedUrls);
+            // The second section loads system pages from Firefox menu in different tabs
+            loadFromMenu(menuItems, expectedUrls);
+        }
     }
 
     // Load from Url the about: pages,verify the Url and the tabs number
     public void checkUrl(String urls []) {
         for (String url:urls) {
             loadAndPaint(url);
             verifyTabCount(mExpectedTabCount);
             verifyUrl(url);