Closed Bug 663123 Opened 13 years ago Closed 13 years ago

Intermittent browser_dragger.js | Sanity check. Blank document cannot be panned left or right. - Got 800, expected 1024

Categories

(Firefox for Android Graveyard :: Panning/Zooming, defect)

Product:
Firefox for Android Graveyard
Component:
Panning/Zooming
Platform:
ARM
Android
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
VERIFIED FIXED
Milestone:
Firefox 7

People

(Reporter: philor, Assigned: mfinkle)

References

Details

(Keywords: intermittent-failure, Whiteboard: [QA!])

Attachments

(1 file)

patch
893 bytes, patch
blassey
: review+
Details | Diff | Splinter Review 
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1307605367.1307605827.31328.gz
Android Tegra 250 mozilla-central opt test browser-chrome on 2011/06/09 00:42:47
s: tegra-025

TEST-INFO | chrome://mochitests/content/browser/mobile/chrome/tests/browser_dragger.js | Browser document width is 800
TEST-INFO | chrome://mochitests/content/browser/mobile/chrome/tests/browser_dragger.js | Window width is 1024
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/tests/browser_dragger.js | Sanity check. Blank document cannot be panned left or right. - Got 800, expected 1024
Attached patch patch — — Splinter Review 
This patch assumes a few things:
1. We do not want the webpage to be wider than the browser. We do not want the page to pan when we test, only the chrome
2. There is some odd IPC viewport timing issue that is keeping the page from scaling to exactly the same width as the window.

We should be able to ignore assumption #2 and just change the is(docWidth, winWidth, ...) to ok(docWidth <= winWidth, ...)

Currently, the actual drag tests pass even if the docWidth is 800, further supporting my plan.
Assignee: nobody → mark.finkle
Attachment #541954 - Flags: review?
Comment on attachment 541954 [details] [diff] [review]
patch

Review of attachment 541954 [details] [diff] [review]:
-----------------------------------------------------------------
Attachment #541954 - Flags: review? → review+
Whiteboard: [orange] → [orange][inbound]
Pushed:
http://hg.mozilla.org/mozilla-central/rev/c791a6319860
Status: NEW → RESOLVED
Closed: 13 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Whiteboard: [orange][inbound] → [orange]
Target Milestone: --- → Firefox 7
Whiteboard: [orange] → [orange][QA?]
No test failures in the last 40 pushes at https://tbpl.mozilla.org/. Marking Verified.
Status: RESOLVED → VERIFIED
Whiteboard: [orange][QA?] → [orange][QA!]
Whiteboard: [orange][QA!] → [QA!]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: