Summary: | [EFL] Add FrameLoaderClientEfl.{cpp,h} to WebKit/efl/WebCoreSupport | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Leandro Pereira <leandro> | ||||||
Component: | Platform | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Enhancement | CC: | barbieri, bunhere, commit-queue, gustavo, gyuyoung.kim, hyuki.a.kim, hyuki.kim, joone.hur, rakuco, tonikitoo | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Other | ||||||||
OS: | Linux | ||||||||
Attachments: |
|
Description
Leandro Pereira
2010-03-09 06:19:07 PST
Created attachment 50298 [details]
Add FrameLoaderClientEfl to WK/efl/WebCoreSupport.
Comment on attachment 50298 [details] Add FrameLoaderClientEfl to WK/efl/WebCoreSupport. On the mobile you probably want to say Mobile Safari and not Safari in the UA. > +PassRefPtr<Frame> FrameLoaderClientEfl::createFrame(const KURL& url, const String& name, HTMLFrameOwnerElement* ownerElement, > + const String& referrer, bool allowsScrolling, int marginWidth, int marginHeight) Please keep these method definitions on one line. Apart from that, it looks fine as an initial commit. Created attachment 53483 [details]
Add FrameLoaderClientEfl to WK/efl/WebCoreSupport.
Kenneth already r+'ed this patch.
I've fixed the createFrame definition.
Regarding the user agent, our API supports changing the user agent, so this won't be a huge issue if someone develops a mobile browser with the EFL port. This will be fixed, of course.
Comment on attachment 53483 [details] Add FrameLoaderClientEfl to WK/efl/WebCoreSupport. Clearing flags on attachment: 53483 Committed r57716: <http://trac.webkit.org/changeset/57716> All reviewed patches have been landed. Closing bug. |