Bug 836818 - Delete robocop testOverscroll; r=kats
authorGeoff Brown <gbrown@mozilla.com>
Mon, 07 Jul 2014 13:19:12 -0600
changeset 192768 f9ee3701b0cbdcaa7e0bedb6d7b6a593902c4706
parent 192767 006053858a182dedbf5d118657dd9123768dcb57
child 192769 9b42ae417853e0d0b888506e832dadf6bba46d21
push id7651
push usercbook@mozilla.com
push dateTue, 08 Jul 2014 13:28:32 +0000
treeherderfx-team@05cfda67b9db [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerskats
bugs836818
milestone33.0a1
Bug 836818 - Delete robocop testOverscroll; r=kats
mobile/android/base/tests/robocop.ini
mobile/android/base/tests/robocop_autophone.ini
mobile/android/base/tests/testOverscroll.java
--- a/mobile/android/base/tests/robocop.ini
+++ b/mobile/android/base/tests/robocop.ini
@@ -54,19 +54,16 @@ skip-if = android_version == "10" || pro
 [testHomeProvider]
 [testLoad]
 [testMailToContextMenu]
 # [testMasterPassword] disabled for being finicky, see bug 1033013
 # disabled on 2.3; bug 979603
 # disabled on 4.0; bug 1006242
 # skip-if = android_version == "10" || android_version == "15"
 [testNewTab]
-[testOverscroll]
-# disabled on 2.3; bug 836818
-skip-if = android_version == "10"
 [testPanCorrectness]
 # disabled on x86 only; bug 927476
 skip-if = processor == "x86"
 # [testPasswordEncrypt] # see bug 824067
 [testPasswordProvider]
 # [testPermissions] # see bug 757475
 [testPictureLinkContextMenu]
 [testPrefsObserver]
--- a/mobile/android/base/tests/robocop_autophone.ini
+++ b/mobile/android/base/tests/robocop_autophone.ini
@@ -36,19 +36,16 @@
 # fails on gs2, nexus one, lg revolution, droid pro, nexus s
 
 #[testLoad]
 # fails on gs2, nexus one, lg revolution, droid pro, nexus s
 
 [testNewTab]
 # fails on nexus s random crash [@ libpvrANDROID_WSEGL.so + 0x73c]
 
-#[testOverscroll]
-# fails on gs2, nexus one, lg revolution, nexus s
-
 #[testPanCorrectness]
 # fails on gs2, nexus one, lg revolution, nexus s
 
 #[testPasswordEncrypt]
 # fails on gs2, nexus one, lg revolution, nexus s
 
 [testPasswordProvider]
 
@@ -79,9 +76,9 @@
 # fails on gs2, nexus one, lg revolution, nexus s
 
 #[testWebContentContextMenu]
 # fails on gs2, nexus one, lg revolution, nexus s
 
 #[test_bug720538]
 # fails on gs2, nexus one, lg revolution, nexus s
 
-[testBrowserSearchVisibility]
\ No newline at end of file
+[testBrowserSearchVisibility]
deleted file mode 100644
--- a/mobile/android/base/tests/testOverscroll.java
+++ /dev/null
@@ -1,48 +0,0 @@
-package org.mozilla.gecko.tests;
-
-import org.mozilla.gecko.Actions;
-import org.mozilla.gecko.PaintedSurface;
-
-/**
- * Basic test to check bounce-back from overscroll.
- * - Load the page and verify it draws
- * - Drag page downwards by 100 pixels into overscroll, verify it snaps back.
- * - Drag page rightwards by 100 pixels into overscroll, verify it snaps back.
- */
-public class testOverscroll extends PixelTest {
-    public void testOverscroll() {
-        String url = getAbsoluteUrl("/robocop/robocop_boxes.html");
-
-        MotionEventHelper meh = new MotionEventHelper(getInstrumentation(), mDriver.getGeckoLeft(), mDriver.getGeckoTop());
-
-        blockForGeckoReady();
-
-        // load page and check we're at 0,0
-        loadAndVerifyBoxes(url);
-
-        // drag page downwards by 100 pixels so that it goes into overscroll.
-        // wait for it to bounce back and check we're at the right spot.
-        // the screen size is small). Note that since we only go into overscroll
-        // and back this should NOT trigger a gecko-paint
-        Actions.RepeatedEventExpecter paintExpecter = mActions.expectPaint();
-        meh.dragSync(10, 50, 10, 150);
-        PaintedSurface painted = waitWithNoPaint(paintExpecter);
-        paintExpecter.unregisterListener();
-        try {
-            checkScrollWithBoxes(painted, 0, 0);
-        } finally {
-            painted.close();
-        }
-
-        // drag page rightwards to go into overscroll on the left. let it bounce and verify.
-        paintExpecter = mActions.expectPaint();
-        meh.dragSync(50, 10, 150, 10);
-        painted = waitWithNoPaint(paintExpecter);
-        paintExpecter.unregisterListener();
-        try {
-            checkScrollWithBoxes(painted, 0, 0);
-        } finally {
-            painted.close();
-        }
-    }
-}