Closed Bug 1387264 Opened 7 years ago Closed 7 years ago

Intermittent testLoad | Pixel at 0, 0 - Color rgba(0,0,0,0) not close enough to expected rgb(0,0,0)

Categories

(Firefox for Android Graveyard :: Testing, defect, P5)

defect

Tracking

(Not tracked)

RESOLVED DUPLICATE of bug 1390059

People

(Reporter: intermittent-bug-filer, Assigned: gbrown)

References

Details

(Keywords: intermittent-failure, Whiteboard: [stockwell disabled])

I'm a little concerned that this may have been "caused" by the recent switch from Ubuntu 12 to Ubuntu 16 - will watch for a few days.

Also, I don't understand why (0,0,0) is not close enough to (0,0,0,0) -- that seems very strange!
Assignee: nobody → gbrown
(In reply to Geoff Brown [:gbrown] from comment #1)
> Also, I don't understand why (0,0,0) is not close enough to (0,0,0,0) --
> that seems very strange!

I see now: the issue is that we expect alpha == 0xff, but we are intermittently seeing alpha == 0.

https://dxr.mozilla.org/mozilla-central/rev/47248637eafa9a38dade8dc3aa6c4736177c8d8d/mobile/android/tests/browser/robocop/src/org/mozilla/gecko/FennecMochitestAssert.java#188


:snorp -- Any idea how this might be happening, or who might be interested?
Flags: needinfo?(snorp)
The 0 alpha is probably from where we set the SurfaceView background to transparent until we get the first paint from Gecko. We do this to avoid showing the default (white) SurfaceView background while the first page loads. My guess is that we are not waiting long enough in the robocop test to take a screenshot. Indeed, a brief inspection of the code[0] supports this. Jim, can you take a further look? I think we need to wait for a better event than DOMContentLoaded.


[0] https://dxr.mozilla.org/mozilla-central/rev/47248637eafa9a38dade8dc3aa6c4736177c8d8d/mobile/android/tests/browser/robocop/src/org/mozilla/gecko/tests/BaseTest.java#201
Flags: needinfo?(snorp) → needinfo?(nchen)
And I see that wait in logcats for recent failures:

https://public-artifacts.taskcluster.net/brQQINiHQ9Sat03YK1SNHA/0/public/test_info//logcat-emulator-5554.log

08-09 09:42:27.048  1312  1312 D Robocop : Received drawFinished notification
08-09 09:42:27.857  1312  1312 D Robocop : Received drawFinished notification
...
08-09 09:42:28.058  1312  1327 D Robocop : received event Content:DOMContentLoaded
08-09 09:42:28.058  1312  1325 D Robocop : unblocked on expecter for Content:DOMContentLoaded
08-09 09:42:28.058  1312  1325 I Robocop : EventExpecter: no longer listening for Content:DOMContentLoaded
...
08-09 09:42:29.608  1312  1312 D Robocop : Received drawFinished notification
08-09 09:42:30.388  1312  1312 D Robocop : Received drawFinished notification
...
08-09 09:42:40.398  1312  1325 I Robocop : PaintExpecter: no longer listening for events
...
08-09 09:42:45.768  1312  1325 I Robocop : {"message":"Color rgba(0,0,0,0) not close enough to expected rgb(0,0,0)","time":1502296965772,"expected":"PASS","source":"robocop","status":"FAIL","test":"testLoad","thread":null,"subtest":"Pixel at 0, 0","action":"test_status","pid":null}
See Also: → 1390059
Assignee: gbrown → nobody
I disabled testFlingCorrectness in bug 1390059, this is still failing often, is there anyone who could work on this?
Assignee: nobody → gbrown
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → DUPLICATE
Whiteboard: [stockwell needswork]
Whiteboard: [stockwell needswork] → [stockwell disabled]
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.