Created attachment 51741 [details] Patch Add support for expanding iframes to their contents, just as done on the iPhone.
After landed we will probably need to rebase the test results with our Qt buildbot.
(In reply to comment #1) > After landed we will probably need to rebase the test results with our Qt > buildbot. When you do that, please let me know which revisions for the test result fixes need to be cherry-picked.
Attachment 51741 [details] did not build on mac: Build output: http://webkit-commit-queue.appspot.com/results/1323023
Comment on attachment 51741 [details] Patch Looks good, r=me I don't have any really substantial comments just one naming objection: > - if (flattenFrameSet()) > + if (shouldResizeFramesToContent()) > positionFramesWithFlattening(); Why this renaming? The naming was discussed quite a bit, lets stick with the "flatten" terminology now. You now have shouldResizeFramesToContent() method in both RenderFrameSet and RenderPartObject (with rather different semantics), even though those are not really related classes. Why do they need to have same names? Why not stick with RenderFrameSet::flattenFrameSet() and add RenderPartObject::flattenFrame() or similar?
Created attachment 51760 [details] Patch 2 (should build on mac, win etc)
Attachment 51760 [details] did not build on mac: Build output: http://webkit-commit-queue.appspot.com/results/1444004
Created attachment 51767 [details] follow up patch for "patch 2"
Comment on attachment 51767 [details] follow up patch for "patch 2" We will put it on the cq when the other land.
Created attachment 51773 [details] Patch 2b (should build on mac, win etc) Uploaded for Kenneth with Mac fix: -__ZN7WebCore8Settings28setFrameFlatteningEnabledEb +__ZN7WebCore8Settings25setFrameFlatteningEnabledEb
Comment on attachment 51767 [details] follow up patch for "patch 2" Please upload one patch that works without having to patch it again with a second one.
Created attachment 51904 [details] Patch
Comment on attachment 51904 [details] Patch Landed in 56718
(In reply to comment #11) > Created an attachment (id=51904) [details] > Patch Mac expected files updated: http://trac.webkit.org/changeset/56720 http://trac.webkit.org/changeset/56722
Comment on attachment 51741 [details] Patch Cleared Antti Koivisto's review+ from obsolete attachment 51741 [details] so that this bug does not appear in http://webkit.org/pending-commit.
Comment on attachment 51904 [details] Patch Cleared Antti Koivisto's review+ from obsolete attachment 51904 [details] so that this bug does not appear in http://webkit.org/pending-commit.
cherry-pick-for-backport: <r56718>
cherry-pick-for-backport: <r56720>
cherry-pick-for-backport: <r56722>
Revision r56718 cherry-picked into qtwebkit-2.0 with commit b74f1de36d3e514dc6e1d6293cd7834286071380
Revision r56720 cherry-picked into qtwebkit-2.0 with commit eb76591b64df4c459a9b291a85163f09748a4c98
Revision r56722 cherry-picked into qtwebkit-2.0 with commit bb360f64ed44718420472bdcc41a8060e0bfc1b8