[Webkit-unassigned] [Bug 47320] limit sendContentResizeNotification when scrollbar really changed.
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Sun Oct 10 19:12:11 PDT 2010
https://bugs.webkit.org/show_bug.cgi?id=47320
--- Comment #5 from Ryuan Choi <ryuan.choi at samsung.com> 2010-10-10 19:12:10 PST ---
(In reply to comment #4)
> (In reply to comment #3)
> > (In reply to comment #2)
> > > (From update of attachment 70030 [details] [details] [details])
> > > Seems reasonable to me. Did you see any perf gain?
> >
> > Sorry about late answer.
> >
> > yes, I saw sluggish scrolling at our mobile browser using frameflattening option.
> > Those sites contains frameset or iframe.
> > And this patch scrolling this kinds of sites make very faster than before.
>
> Strange, only relayout could affect scrolling, so are these sites for some reason relayouting the whole time? Maybe it is better to look into this? or optimize the frame flattening layout code.
>
you're right. this patch want not to relayout if scrollbar wasn't changed.
If not, ScrollView calls contentsResized and visiblieContentsResized although scrollbar wasn't created because of frameflattening.
And my test site is http://postech.edu/
> Also, frame flattening is needed for a nice touch interaction model. That is way more important than the performance.
Yes, so we want to use frame flattening. But I found that it's too sluggish case and I believe that this patch remove it without any side effect.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list