RESOLVED FIXED 129016
Clean up WKContentView initialization 
https://bugs.webkit.org/show_bug.cgi?id=129016
Summary Clean up WKContentView initialization
Simon Fraser (smfr)
Reported 2014-02-18 18:32:37 PST
Clean up WKContentView initialization
Attachments
Patch (5.94 KB, patch)
2014-02-18 18:34 PST, Simon Fraser (smfr) 		benjamin: review+
benjamin: commit-queue-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Simon Fraser (smfr)
Comment 1 2014-02-18 18:34:15 PST
Created attachment 224575 [details] Patch
Benjamin Poulain
Comment 2 2014-02-18 18:37:02 PST
Comment on attachment 224575 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=224575&action=review > Source/WebKit2/UIProcess/API/ios/WKViewIOS.mm:246 > - _contentView = adoptNS([[WKContentView alloc] initWithFrame:bounds contextRef:contextRef pageGroupRef:pageGroupRef relatedToPage:relatedPage]); > + WebKit::WebPageConfiguration webPageConfiguration; > + webPageConfiguration.pageGroup = toImpl(pageGroupRef); > + webPageConfiguration.relatedPage = toImpl(relatedPage); > + > + _contentView = adoptNS([[WKContentView alloc] initWithFrame:bounds context:*toImpl(contextRef) configuration:std::move(webPageConfiguration)]); You need to update WKWebView accordingly :(
Simon Fraser (smfr)
Comment 3 2014-02-18 20:06:35 PST
WKWebView already uses the correct initializer.
Simon Fraser (smfr)
Comment 4 2014-02-18 20:07:59 PST
http://trac.webkit.org/changeset/164349
Note You need to log in before you can comment on or make changes to this bug.