Open Bug 988858 (apz-threadsafe) Opened 11 years ago Updated 3 years ago

[meta] Ensure all of the APZ code is threadsafe

Categories

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

Product:
Core
Component:
Panning and Zooming
Type:
defect

Tracking

()

People

(Reporter: kats, Unassigned)

References

(Depends on 2 open bugs)

Details

(Keywords: correctness, meta, Whiteboard: gfx-noted)

We need to make sure that all the APZC code is threadsafe and has documented the threading assumptions. In particular this will be needed before we can either enable APZC on Fennec (where the input thread is different from the gecko thread) or tackle bug 930939.
Alias: apz-threadsafe
Seeing as how we already landed bug 930939 I don't think this is really "needed" before we can do APZ on Fennec. Removing dependencies, but leaving this bug around as there are probably some thready safety things we need to do in APZ code.
No longer blocks: apz-fennec, input-thread
Depends on: 1158970
Depends on: 1153979
Keywords: correctness
Whiteboard: gfx-noted
OS: Gonk (Firefox OS) → All
Depends on: 1477832
Depends on: 1549504

It seems like :decoder has found a systematic approach to discovering thread safety issues in APZ (and other) code via TSan. Thank you :)

Depends on: 1606807, 1606859, 1606661, 1601940, 1600567
Summary: [meta] Ensure all of the APZC code is threadsafe → [meta] Ensure all of the APZ code is threadsafe
See Also: → tsan
Depends on: 1607764
Depends on: 1615275
Depends on: 1664802
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.