[Webkit-unassigned] [Bug 99101] [EFL][WK2] Rename EflViewportHandler to PageViewportControllerClientEfl

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Oct 18 18:43:51 PDT 2012


https://bugs.webkit.org/show_bug.cgi?id=99101





--- Comment #6 from Gyuyoung Kim <gyuyoung.kim at samsung.com>  2012-10-18 18:44:45 PST ---
(In reply to comment #5)

> > What is purpose of pageViewportController ? It looks this member variable is not used in this patch.
> After renaming this class, it now becomes a client of PageViewportController, so I added pageViewportController, since it doesn't make sense to have a client without the class it is client of.
> Subsequent patches will make use of pageViewportController. 
> If you feel that pageViewportController should be added later, I can remove it from this patch and add it in a later patch.

I prefer to add it when using actually. It would be good if you add it when you prepare new patch.

-- 
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