Bug 746876 - Fix double-tap coordinates in robocop test and re-enable it. r=gbrown, a=test-only
authorKartikaya Gupta <kgupta@mozilla.com>
Thu, 30 May 2013 09:58:29 -0400
changeset 142796 83d8259caa7b551d47a712fcf3f0e506a9d8c0e3
parent 142795 dc00e08878b8bd1bb4f9a2d36a295d94758a4334
child 142797 9a6f07b8abbf292e69a30555b758e16aa7772746
push id2579
push userakeybl@mozilla.com
push dateMon, 24 Jun 2013 18:52:47 +0000
treeherdermozilla-beta@b69b7de8a05a [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersgbrown, test-only
bugs746876
milestone23.0a2
Bug 746876 - Fix double-tap coordinates in robocop test and re-enable it. r=gbrown, a=test-only
mobile/android/base/tests/robocop.ini
mobile/android/base/tests/test_bug720538.java.in
--- a/mobile/android/base/tests/robocop.ini
+++ b/mobile/android/base/tests/robocop.ini
@@ -6,17 +6,17 @@
 [testBookmark]
 [testBookmarklets]
 [testMigration]
 [testLoad]
 [testNewTab]
 [testOrderedBroadcast]
 [testPrefsObserver]
 [testPanCorrectness]
-# [test_bug720538] # see bug 746876
+[test_bug720538]
 [testFlingCorrectness]
 [testOverscroll]
 [testAxisLocking]
 [testAboutPage]
 # [testWebContentContextMenu] # see bug 862493
 [testPasswordProvider]
 # [testPasswordEncrypt] # see bug 824067
 [testFormHistory]
--- a/mobile/android/base/tests/test_bug720538.java.in
+++ b/mobile/android/base/tests/test_bug720538.java.in
@@ -53,17 +53,17 @@ public class test_bug720538 extends Pixe
             mAsserter.ispixel(painted.getPixelAt(mDriver.getGeckoWidth() - 51, 100), 0, 0, 0xFF, "Checking for iframe a few pixels from the right edge");
             mAsserter.ispixel(painted.getPixelAt(mDriver.getGeckoWidth() - 1, 100), 0, 0x80, 0, "Checking page background the right of the iframe");
         } finally {
             painted.close();
         }
 
         // now we do double-tap again to zoom out and wait for the animation to finish, as before
         paintExpecter = mActions.expectPaint();
-        meh.doubleTap(100, mDriver.getGeckoWidth() / 2);
+        meh.doubleTap(mDriver.getGeckoWidth() / 2, 100);
         painted = waitForPaint(paintExpecter);
         paintExpecter.unregisterListener();
 
         try {
             // and now we check a pixel at the bottom of the view to ensure that we have the page
             // background and not some checkerboarding. use the second-last row of pixels instead of
             // the last row because the last row is subject to rounding and clipping errors
             for (int y = 2; y < 10; y++) {