Bug 47325

Summary: [Qt] API: ViewportConfiguration => ViewportArguments
Product: WebKit Reporter: Andreas Kling <kling>
Component: WebKit QtAssignee: QtWebKit Unassigned <webkit-qt-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: ademar, commit-queue, kamaji, kenneth
Priority: P2 Keywords: Qt, QtTriaged
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Proposed patch none

Andreas Kling
Reported 2010-10-06 21:46:31 PDT
Following the rename in WebCore, QWebPage::ViewportConfiguration should be renamed to QWebPage::ViewportArguments, and QWebPage::viewportConfigurationForSize() should be renamed to QWebPage::viewportAttributesForSize()
Attachments
Proposed patch (9.40 KB, patch)
2010-10-06 22:13 PDT, Andreas Kling 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Andreas Kling
Comment 1 2010-10-06 22:13:28 PDT
Created attachment 70035 [details] Proposed patch
WebKit Commit Bot
Comment 2 2010-10-07 12:41:53 PDT
Comment on attachment 70035 [details] Proposed patch Clearing flags on attachment: 70035 Committed r69331: <http://trac.webkit.org/changeset/69331>
WebKit Commit Bot
Comment 3 2010-10-07 12:41:58 PDT
All reviewed patches have been landed. Closing bug.
Ademar Reis
Comment 4 2010-10-08 02:59:34 PDT
Shouldn't WebKit2 API be changed as well (on trunk)?
Kenneth Rohde Christiansen
Comment 5 2010-10-08 04:37:01 PDT
(In reply to comment #4) > Shouldn't WebKit2 API be changed as well (on trunk)? The WebKit2 API will change as soon as we have the WebKit2 C API approved and integrated. This doesn't depend on our public release.
Ademar Reis
Comment 6 2010-10-08 04:43:17 PDT
Revision r69331 cherry-picked into qtwebkit-2.1 with commit 15761d2 <http://gitorious.org/webkit/qtwebkit/commit/15761d2>
Keith Kyzivat
Comment 7 2010-10-12 07:42:47 PDT
This breaks Symbian, as the .def files are now again, out of sync, and need to be re-frozen.
Note You need to log in before you can comment on or make changes to this bug.