Closed Bug 1606365 Opened 5 years ago Closed 3 years ago

[Browsertime][WebRender] Orange frame still persists after the navigation has started

Categories

(Testing :: Performance, defect, P3)

Version 3
defect

Tracking

(Not tracked)

RESOLVED WORKSFORME

People

(Reporter: sefeng, Assigned: sefeng)

References

Details

Attachments

(1 file)

Attached video browsertime.MOV

Check around 5 seconds of the video, you can see the navigation had started, but the orange frame was still persist. This was Fenix running Moto G5. (also reproducible on Pixel 3)

This only occurs when WebRender is enabled.

Summary: [Browsertime][WebRender] Orange frame is still persist after the navigation has started → [Browsertime][WebRender] Orange frame still persists after the navigation has started
Blocks: 1601004

Jessie - Can you get someone on the WR android team to take a look at this? Perhaps the visual update from removing the DOM orange frame before starting the navigation is getting delayed?

For reference, browsertime uses this (browsertime/lib/core/seleniumRunner.js) to make the orange frame's removal mark the the start of a load:

 const navigate = `(function() {
     const orange = document.getElementById('browsertime-orange');
     if (orange) {
       orange.parentNode.removeChild(orange);
     }
     window.requestAnimationFrame(function(){
       window.requestAnimationFrame(function(){
         window.location="${url}";
       });
     });
   })();`;
Flags: needinfo?(jbonisteel)
Flags: needinfo?(jbonisteel) → needinfo?(ktaeleman)
Blocks: wr-android
Flags: needinfo?(ktaeleman)
Priority: -- → P3
Flags: needinfo?(ktaeleman)

Jamie: Do you think this could be related to the black screen issues we've been seeing?

Blocks: wr-73-android
No longer blocks: wr-android
Flags: needinfo?(ktaeleman) → needinfo?(jnicol)

It sounds plausible, yes. I'll investigate.

Sean or Randell, I've never used browsertime before. Would it be possible to give a slightly more detailed STR?

Flags: needinfo?(jnicol)
Assignee: nobody → sefeng
Priority: P3 → P1

Hi Jamie,

In latest m-c, you should be able to do ./mach browsertime --setup, and it will install browsertime for you, please make sure the output says

ffmpeg:   OK
convert:  OK
compare:  OK
Pillow:   OK
SSIM:     OK

And then you can use this command to reproduce it. You need to install the performanceTest apk to your phone, you can get it here

./mach browsertime -- --android -b firefox --firefox.android.package org.mozilla.fenix.performancetest --video true --visualMetrics true --firefox.android.intentArgument=--ez --firefox.android.intentArgument=TURBO_MODE --firefox.android.intentArgument=false --firefox.android.intentArgument=-a --firefox.android.intentArgument=android.intent.action.VIEW --firefox.android.intentArgument=-d --firefox.android.intentArgument="data:," --firefox.windowRecorder false --firefox.android.activity org.mozilla.fenix.browser.BrowserPerformanceTestActivity -n 1 https://ebay.com
Flags: needinfo?(jnicol)

We theorize this should be addressed by bug 1581868

FWIW, I don't think this is related to the black screen issue. I think it should probably be debugged separately.

I spoke too soon!

We worked around this by adding a white div behind the orange one.

Changing the priority of the bug due to workaround unblocking browsertime.

Blocks: wr-android
No longer blocks: wr-73-android
Priority: P1 → P3

For the record, this is the workaround https://github.com/mozilla/browsertime/pull/60

Blocks: wr-android-testing
No longer blocks: wr-android

:jnicol, there's an outstanding ni? in this bug from :sefeng - has it been answered?

Flags: needinfo?(jnicol)
Flags: needinfo?(jnicol)

I don't think the needinfo was asking anything, but instead was just to ensure I saw Sean's answer. So yes.

(I think I was able to reproduce using Sean's instructions, but didn't investigate any further due to the workaround.)

Flags: needinfo?(jnicol)

:sefeng, do you know if this is still an issue?

Flags: needinfo?(sefeng)

I removed the workaround we did in comment 10 from browsertime, and was unable to reproduce it by using the command from comment 4 against https://firefox-ci-tc.services.mozilla.com/tasks/index/mobile.v2.fenix.beta.2020.09.23.latest.

So looks like this is not an issue anymore.

Flags: needinfo?(sefeng)
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → WORKSFORME

Can you remove the workaround from upstream browsertime?

Flags: needinfo?(sefeng)
Flags: needinfo?(sefeng)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: