Closed Bug 909877 (gaia-apzc) Opened 11 years ago Closed 10 years ago

[meta] Turn on APZC for all of gaia


(Firefox OS Graveyard :: Gaia, defect)

Gonk (Firefox OS)
Not set


(blocking-b2g:1.3+, b2g-v1.2 wontfix, b2g-v1.3 fixed, b2g-v1.3T fixed, b2g-v1.4 fixed)

1.3 C2/1.4 S2(17jan)
blocking-b2g 1.3+
Tracking Status
b2g-v1.2 --- wontfix
b2g-v1.3 --- fixed
b2g-v1.3T --- fixed
b2g-v1.4 --- fixed


(Reporter: kats, Assigned: kats)



(Keywords: perf, Whiteboard: [c=handeye p= s=2014.01.17 u=1.3])


(1 obsolete file)

Currently subframe scrolling in Gaia apps happens using BrowserElementPanning.js and occurs synchronously. This bug tracks all the work needed to switch that over to AsyncPanZoomController's async panning.
For the record this is what I'm using as my current hack to "turn on APZC everywhere". It may not be correct but it's good enough to find out what bugs need fixing.
Alias: gaia-apzc
We should keep an eye on the FPS (e.g., bug 912134) as we convert the apps, as a sanity check that the performance is not regressing with these changes.
To turn this on by default, check 929728 which introduced the preference.  Not ready for it yet.
blocking-b2g: --- → 1.3+
Spoke with nhirata in person about this - he's going to drive the QA testing on this.
Flags: in-moztrap?(nhirata.bugzilla)
Is there any UX design required for this? Or is APZC mainly related to performance?
Flags: needinfo?(bugmail.mozilla)
Flags: needinfo?(21)
(In reply to Rob MacDonald [:robmac] from comment #6)
> Is there any UX design required for this? Or is APZC mainly related to
> performance?

Afaict APZC in this bug is mostly related to performance. 

I believe that there are some UX questions that can be discussed into relevant bugs. For example about overscroll indicators in bug 775469. Or about the panning heuristic, or event about when to display scrollbars and when not, etc...

IMHO the main consideration you should have here is that APZC will unify the scrolling behavior between Gaia Web Browser and Apps running in Firefox OS. The behavior will be derived from the current behavior of the browser app, so if there are things that UX does not like in it, feel free to open bugs to that blocks this one and let's discuss the details there.
Flags: needinfo?(21)
Keywords: perf
Whiteboard: [FFOS_perf] → [c= p= s= u=]
Depends on: 943112
No longer depends on: 943112
Blocks: 845690
(In reply to Rob MacDonald [:robmac] from comment #6)
> Is there any UX design required for this? Or is APZC mainly related to
> performance?

What Vivien said.
Flags: needinfo?(bugmail.mozilla)
Is this gonna be in 1.3 after all?
Depends on: 945789
(In reply to Jan Jongboom [:janjongboom] from comment #10)
> Is this gonna be in 1.3 after all?

The target is 1.3.
Depends on: 946545
Depends on: 947354
(From FxOS Perf triage) I thought we didn't 1.3+ meta bug?
Flags: needinfo?(praghunath)
(In reply to Hubert Figuiere [:hub] from comment #12)
> (From FxOS Perf triage) I thought we didn't 1.3+ meta bug?

It doesn't hurt, and it makes it easier to make sure dependent bugs are triaged.  Sometimes we forget to put a 1.3? or 1.3+ on the blockers.
The patch doesn't apply to the current 1.3 build.  Need to talk to kats to figure out where to apply the patch.
Comment on attachment 796178 [details] [diff] [review]
WIP - turn on APZC on everything in gaia

The patch is obsolete now. There is an option in the gaia settings to enable APZ
Attachment #796178 - Attachment is obsolete: true
Depends on: 951290
Depends on: 951342
Depends on: 951357
Depends on: 950993
Depends on: 951433
Depends on: 951308
Depends on: 951321
Depends on: 951458
Depends on: 957188

What are the next steps here?
Flags: needinfo?(praghunath) → needinfo?(21)
Pull request pending merge at to enable by default.
(In reply to Kartikaya Gupta ( from comment #17)
> Pull request pending merge at
> to enable by default.

Are you looking for someone to review that patch?
There's not much to review. Milan asked me to make the change and fabrice said he would merge it once the travis run is green.
Depends on: 957790
(In reply to Fabrice Desré [:fabrice] from comment #20)
> 236e4f96cbfdaf2c90c12ebf61107dd274e7fd73

Should we mark this fixed with this landing then?
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [c= p= s= u=] → [c=handeye p= s=2014.01.17 u=1.3]
I'm unsure of the gaia branching model but I believe this will need to be merged to 1.3 as well.
No longer depends on: 957925
Depends on: 942750
cannot scroll in FTU, see bug 958208
Depends on: 958208
This does not uplift cleanly:

$ git diff
* Unmerged path build/config/common-settings.json

$ git status
Unmerged paths:
  (use "git add/rm <file>..." as appropriate to mark resolution)

	deleted by us:      build/config/common-settings.json
I made a rebased version on the latest mozillaorg/v1.3 code:

Please let me know if you require it in some other format.
Flags: needinfo?(jhford)
Depends on: 951259
Depends on: 958245
[v1.3 226aa05] Bug 909877 - Enable APZ for all of gaia. rs=milan
Flags: needinfo?(jhford)
Next step has already been done. Clearing.
Flags: needinfo?(21)
No longer depends on: 951259
No longer depends on: 958036
Blocks: 958650
No longer blocks: 958650
Depends on: 958650
No longer depends on: 950301
Depends on: 959199
Depends on: 959198
No longer depends on: 942750
Depends on: 959866
No longer depends on: 959866
No longer depends on: 947337
Depends on: 959414
Depends on: 960163
Target Milestone: --- → 1.3 C2/1.4 S2(17jan)
No longer depends on: 958230
No longer depends on: 959425
Depends on: 962791
generic apz test created to cover a lot of things to check.
Flags: in-moztrap?(nhirata.bugzilla) → in-moztrap+
No longer blocks: 823206
Depends on: 965004
No longer depends on: 965004
Depends on: 964997
Depends on: 964981
Depends on: 965353
No longer depends on: 965353
Depends on: 964421
Depends on: 966476
Depends on: 966507
Depends on: 966510
Depends on: 968145
Depends on: 969483
No longer depends on: 966510
No longer depends on: 959414
Depends on: 972675
No longer depends on: 972675
Depends on: 975033
Depends on: 977295
No longer depends on: 977295
Depends on: 980041
Depends on: 980679
Depends on: 982888
Depends on: 987771
Depends on: 989403
No longer depends on: 989403
Depends on: 996991
You need to log in before you can comment on or make changes to this bug.