RESOLVED FIXED 110345
[WK2][EFL] Move PageViewportController and PageViewportControllerClientEfl out from WebView 
https://bugs.webkit.org/show_bug.cgi?id=110345
Summary [WK2][EFL] Move PageViewportController and PageViewportControllerClientEfl ou...
Mikhail Pozdnyakov
Reported 2013-02-20 06:56:14 PST
SSIA. WebView does not need to know about both PageViewportController and PageViewportControllerClientEfl. Considering the fact that PageViewportController will stop being WK2 internal and will be based on WK2 C API, it's worth moving it to Ewk API layer - into EwkView.
Attachments
patch (8.74 KB, patch)
2013-02-20 07:45 PST, Mikhail Pozdnyakov 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Mikhail Pozdnyakov
Comment 1 2013-02-20 07:45:39 PST
Created attachment 189318 [details] patch
Kenneth Rohde Christiansen
Comment 2 2013-02-20 08:04:58 PST
Comment on attachment 189318 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=189318&action=review LGTM > Source/WebKit2/UIProcess/efl/WebView.cpp:155 > + m_ewkView->pageViewportControllerClient()->updateViewportSize(); Would be nice to get rid of m_ewkView from this class
Mikhail Pozdnyakov
Comment 3 2013-02-20 23:12:23 PST
(In reply to comment #2) > (From update of attachment 189318 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=189318&action=review > > LGTM > > > Source/WebKit2/UIProcess/efl/WebView.cpp:155 > > + m_ewkView->pageViewportControllerClient()->updateViewportSize(); > > Would be nice to get rid of m_ewkView from this class Sure. We'll remove m_ewkView usage here via WKViewClient API extension.
WebKit Review Bot
Comment 4 2013-02-21 04:00:34 PST
Comment on attachment 189318 [details] patch Clearing flags on attachment: 189318 Committed r143582: <http://trac.webkit.org/changeset/143582>
WebKit Review Bot
Comment 5 2013-02-21 04:00:39 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.