574045 - [e10s] Tiles don't update while panning

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: Panning/Zooming, defect)

Description

[Josh Matthews [:jdm]]

Using e10s+m-b: if I pan around a page, no new tiles are drawn until I stop panning.

Comment 1

[Matt Brubeck (:mbrubeck)]

Attachment: patch

Uses stechz's code from http://hg.mozilla.org/users/pavlov_mozilla.com/mobile-e10s/file/e4580e37a10f/chrome/content/browser.js#l1445

Comment 2

[Matt Brubeck (:mbrubeck)]

Even on non-e10s builds this is quite usable on a Nexus One.  (Very slightly jerky, but much better than scrolling through screenfuls of nothing but checkerboard.)

Comment 3

[Benjamin Stover (:stechz)]

Comment on attachment 455172 [details] [diff] [review]
patch

If you are going to include the setPrefetch changes in this patch, we need to probably remove the idle observer from browserview.

Comment 4

[Matt Brubeck (:mbrubeck)]

Attachment: patch v2

Remove the idleServiceObserver and setPrefetch calls from the dragger.

Comment 5

[Benjamin Stover (:stechz)]

Pushed http://hg.mozilla.org/mobile-browser/rev/8e8bf249d469

Comment 6

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

I'm not sure to understand why we removed completely the calls to the idleObserver? This one was added to prevent the tiles to be changes once you've start panning. Even if this is going to be removed soon with Layers we should probably keep it for now.