Remove redundant updateViewportArguments() call when page is restored from page cache.
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 29 Mar 2012 12:53:08 +0000 (12:53 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 29 Mar 2012 12:53:08 +0000 (12:53 +0000)
commit5d9797f70f45ce73d6bea4b621a29128793280e1
tree690ba0f051815381efd1822a3a09f91df3c679d9
parent8c2aedd5d0ee4908bcc6ef4e4f3f626867168cf3
Remove redundant updateViewportArguments() call when page is restored from page cache.
https://bugs.webkit.org/show_bug.cgi?id=82500

Patch by Zalan Bujtas <zbujtas@gmail.com> on 2012-03-29
Reviewed by Kenneth Rohde Christiansen.

.:

Add manual test for history navigation with viewport width check.

* ManualTests/viewport-width-test-after-history-navigation.html: Added.

Source/WebCore:

Document::updateViewportArguments() is called twice, while restoring a page from page cache.
First, it is called when the document is set on the mainframe and later, it is called
when page cache finished the restoration. Since viewport arguments don't change between
the 2 calls, it's safe to remove the second.

Manual test added. Viewport value updates heavily depend on UI process code.

* dom/Document.cpp:
(WebCore::Document::documentDidResumeFromPageCache):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@112526 268f45cc-cd09-0410-ab3c-d52691b4dbfc
ChangeLog
ManualTests/viewport-width-test-after-history-navigation.html [new file with mode: 0644]
Source/WebCore/ChangeLog
Source/WebCore/dom/Document.cpp