Open Bug 1566083 Opened 5 years ago Updated 2 years ago

Make sure we test the performance of async zoom

Categories

(Core :: Panning and Zooming, task, P3)

task

Tracking

()

People

(Reporter: jrmuizel, Unassigned)

References

(Blocks 1 open bug)

Details

(Whiteboard: wr-android)

I suspect we could probably add a test to tscroll that does this.

Priority: -- → P3

making this block our android meta bug because I imagine we would want this before shipping to release

Assignee: nobody → jmuizelaar

Botond, do you have any ideas for how to drive an async zoom programmatically?

Flags: needinfo?(botond)

Something like this.

Flags: needinfo?(botond)

Or, if you prefer, via double-tap.

Blocks: wr-73-android
No longer blocks: wr-android-mvp
Blocks: wr-74-android
No longer blocks: wr-73-android
No longer blocks: wr-74-android

Can we do this using Raptor or do we need Talos for this?

The perftest team currently has no plans to run Talos on android.

Flags: needinfo?(botond)

I'm not too familiar with either, but based on a brief conversation with Markus, it should be doable with Raptor (e.g. we should be able to access APIs like the ones mentioned in comment 3 / comment 4 in a Raptor test).

Markus suggested reaching out to :jmaher about details about how Raptor tests work.

Flags: needinfo?(botond)
Blocks: wr-android
Whiteboard: wr-android

@botond: With async zoom coming to desktop as well, does it make more sense for this to go to the APZ component instead of wr-android?

Flags: needinfo?(botond)

I think we'll want performance tests on both mobile and desktop, as they have different performance characteristics (due to e.g. different screen sizes, different levels of device power). That could however take the form of running the same test on multiple platforms.

I don't feel strongly about which component we track the bug in.

Flags: needinfo?(botond)
No longer blocks: wr-android

Guessing we can close this now?

Flags: needinfo?(botond)

Did we end up writing any tests in this area?

If not, I think it's worth keeping this open (but we can move it into Panning and Zooming, as the purpose of the test at this point would no longer be anything related to rolling out WebRender, but rather to ensure pinch-zoom performance doesn't regress in general going forward).

Flags: needinfo?(botond) → needinfo?(jmuizelaar)

I don't think we wrote any tests.

Flags: needinfo?(jmuizelaar)
Assignee: jmuizelaar → nobody
Component: Graphics: WebRender → Panning and Zooming
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.