Confirmed users
336
edits
Mobile/AsyncSubframePanning: Difference between revisions
→Notes and diagram
| (10 intermediate revisions by 4 users not shown) | |||
| Line 58: | Line 58: | ||
[[File:ApzcDesignPage1.png]] | [[File:ApzcDesignPage1.png]] | ||
[[File:9-.png]] | [[File:9-.png]] | ||
[[File:b2g-input-flow.png]] | |||
==Bugs== | ==Bugs== | ||
| Line 265: | Line 266: | ||
;status | ;status | ||
* kats | * kats | ||
** landed a bunch of stuff for event targeting | ** landed a bunch of stuff for event targeting ({{bug|890280}}, {{bug|906109}}) | ||
** filed a bug for disabling pinch zooming on subframes | ** filed a bug for disabling pinch zooming on subframes {{bug|906427}} | ||
** have patches up for review for more event targeting {{bug|898443}} | |||
* benwa | * benwa | ||
** camera crash bugs | ** camera crash bugs | ||
| Line 278: | Line 280: | ||
** turn on async panning for all apps on b2g | ** turn on async panning for all apps on b2g | ||
** investigate overflow div panning | ** investigate overflow div panning | ||
;need to look at | |||
* display port tuning | |||
* input events go through gecko thread | |||
** would be better if input events didn't | |||
===2013-08-27=== | |||
;status | |||
* kats | |||
** landed some focus model stuff (bug 898443, bug 906427, 907977) | |||
** have a patch for bug 907754 (incorrect scaling in subframes) which needs cleanup. Also landed bug 885023 which this fix required | |||
** looked at what things are like when APZC is turned on for all of gaia. Some weird issues but I will work through the big ones one at a time to see if they are hiding any other issues underneath | |||
;next | |||
* kats | |||
** going to work on tracking down the overflow:scroll div problem | |||
** work through the gaia issues I found (mostly using the contacts app as a test) | |||
===2013-09-03=== | |||
;status | |||
* kats | |||
** landed 907754 (zoom, overflow documents issues) | |||
** landing 910322 today after addressing some comments | |||
** overflow scroll sub-documents problems - probably bug in flattening code, waiting for mwoodrow and tn | |||
* BenWa | |||
** not working on this last week | |||
* botond | |||
** handing overscroll to the parent layer; have the first implementation going | |||
** will be looking at display port bugs | |||
** composition bounds of iframes, same as what jmathies was running at - have an r+ from tn | |||
;next steps | |||
* kats | |||
** may run out of stuff to look at - one bug to file, hit testing a bit off | |||
** blocked on waiting for info and reviews | |||
;notes | |||
* kats - issues with tiling on fennec? wrong resolution, wrong sizes? | |||
* benwa - you see progressive tile painting in the cases where you should not (e.g, updating existing content) | |||
===2013-09-10=== | |||
;status | |||
* kats | |||
** implementing parent APZC | |||
** talked to tn about overflow scroll problem | |||
*** frames construction isn't correct (there's a nested scroll frame) | |||
*** tn will think about it | |||
*** milan will follow up with tn tomorrow | |||
* botond | |||
** patch for scroll hand off posted | |||
*** issue with zoomed subframes | |||
** moving on to annotating the browser iframe | |||
* BenWa did nothing of relevance | |||
;next | |||
* botond to annotate the browser iframe | |||
===2013-09-24=== | |||
;status | |||
* kats | |||
** last week was mostly mobile work week | |||
** continued working on bug 912657. input events are currently not being sent to the APZC in the root process so that will need fixing. currently they are delivered from gonk/nsWindow.cpp directly to gecko, gecko does some hit testing, and if they are in the area of the content process, *then* they go to the APZC. instead nsWindow should send it directly to the APZC first. i will work on this, building off the work jimm has done in bug 915213. | |||
* botond | |||
** at a C++ standardization conference this week | |||
** last week he worked on bug 912666 and made good progress on special handling for the toolbar. the individual pieces seem to work but it depends on bug 912657 to tie it all together. | |||