Bug 148282 - [EFL] Revise PlatformWebView ctor according to r188718
Summary: [EFL] Revise PlatformWebView ctor according to r188718
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit EFL (show other bugs)
Version: WebKit Local Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Joonghun Park
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-08-20 19:26 PDT by Joonghun Park
Modified: 2015-08-22 11:39 PDT (History)
4 users (show)

See Also:


Attachments
Patch (3.12 KB, patch)
2015-08-20 19:31 PDT, Joonghun Park
gyuyoung.kim: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Joonghun Park 2015-08-20 19:26:11 PDT
https://bugs.webkit.org/show_bug.cgi?id=147996 changed PlatformWebView ctor's argument to use WKPageConfigurationRef instead of WKContextRef and WKPageGroupRef respectively.
So change the ctor argument aligning with it.
Comment 1 Joonghun Park 2015-08-20 19:31:00 PDT
Created attachment 259565 [details]
Patch
Comment 2 Gyuyoung Kim 2015-08-20 21:28:53 PDT
Comment on attachment 259565 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=259565&action=review

> Source/WebKit2/PlatformEfl.cmake:68
> +    UIProcess/API/C/WKPageConfigurationRef.cpp

Wrong alphabet order.
Comment 3 Gyuyoung Kim 2015-08-20 21:30:58 PDT
Committed r188751: <http://trac.webkit.org/changeset/188751>
Comment 4 Gyuyoung Kim 2015-08-20 21:31:32 PDT
(In reply to comment #3)
> Committed r188751: <http://trac.webkit.org/changeset/188751>

Land after fixing it.
Comment 5 Joonghun Park 2015-08-20 22:24:27 PDT
(In reply to comment #2)
> Comment on attachment 259565 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=259565&action=review
> 
> > Source/WebKit2/PlatformEfl.cmake:68
> > +    UIProcess/API/C/WKPageConfigurationRef.cpp
> 
> Wrong alphabet order.

If I place files as below,

UIProcess/API/C/CoordinatedGraphics/WKView.cpp
UIProcess/API/C/WKPageConfigurationRef.cpp

then when I run check-webkit-style script, I meet the log like this.

Source/WebKit2/PlatformEfl.cmake:69:  Alphabetical sorting problem. "UIProcess/API/C/WKPageConfigurationRef.cpp" should be before "UIProcess/API/C/CoordinatedGraphics/WKView.cpp".

It seems that WKPageConfigurationRef.cpp is precede to WKView.cpp 
because the former is file name and the latter is an intermediate path name I think.

So I guess the current alphabetical order is the right one.
Comment 6 Gyuyoung Kim 2015-08-20 22:30:51 PDT
(In reply to comment #5)
> (In reply to comment #2)
> > Comment on attachment 259565 [details]
> > Patch
> > 
> > View in context:
> > https://bugs.webkit.org/attachment.cgi?id=259565&action=review
> > 
> > > Source/WebKit2/PlatformEfl.cmake:68
> > > +    UIProcess/API/C/WKPageConfigurationRef.cpp
> > 
> > Wrong alphabet order.
> 
> If I place files as below,
> 
> UIProcess/API/C/CoordinatedGraphics/WKView.cpp
> UIProcess/API/C/WKPageConfigurationRef.cpp
> 
> then when I run check-webkit-style script, I meet the log like this.
> 
> Source/WebKit2/PlatformEfl.cmake:69:  Alphabetical sorting problem.
> "UIProcess/API/C/WKPageConfigurationRef.cpp" should be before
> "UIProcess/API/C/CoordinatedGraphics/WKView.cpp".
> 
> It seems that WKPageConfigurationRef.cpp is precede to WKView.cpp 
> because the former is file name and the latter is an intermediate path name
> I think.
> 
> So I guess the current alphabetical order is the right one.

Yes, it just needed to have a empty line. :) I fixed it.

http://trac.webkit.org/changeset/188751/trunk/Source/WebKit2/PlatformEfl.cmake
Comment 7 Joonghun Park 2015-08-20 22:35:50 PDT
(In reply to comment #6)
> (In reply to comment #5)
> > (In reply to comment #2)
> > > Comment on attachment 259565 [details]
> > > Patch
> > > 
> > > View in context:
> > > https://bugs.webkit.org/attachment.cgi?id=259565&action=review
> > > 
> > > > Source/WebKit2/PlatformEfl.cmake:68
> > > > +    UIProcess/API/C/WKPageConfigurationRef.cpp
> > > 
> > > Wrong alphabet order.
> > 
> > If I place files as below,
> > 
> > UIProcess/API/C/CoordinatedGraphics/WKView.cpp
> > UIProcess/API/C/WKPageConfigurationRef.cpp
> > 
> > then when I run check-webkit-style script, I meet the log like this.
> > 
> > Source/WebKit2/PlatformEfl.cmake:69:  Alphabetical sorting problem.
> > "UIProcess/API/C/WKPageConfigurationRef.cpp" should be before
> > "UIProcess/API/C/CoordinatedGraphics/WKView.cpp".
> > 
> > It seems that WKPageConfigurationRef.cpp is precede to WKView.cpp 
> > because the former is file name and the latter is an intermediate path name
> > I think.
> > 
> > So I guess the current alphabetical order is the right one.
> 
> Yes, it just needed to have a empty line. :) I fixed it.
> 
> http://trac.webkit.org/changeset/188751/trunk/Source/WebKit2/PlatformEfl.
> cmake

Ah, I missed the empty line. Thank you :)
Comment 8 Michael Catanzaro 2015-08-22 11:39:59 PDT
Thanks Joonghun. I moved it from PlatformEfl.cmake to CMakeLists.txt in r188826, since it's needed by every port.