Commit 42c4da71 authored by mihaip@chromium.org's avatar mihaip@chromium.org

[Chromium] REGRESSION (r95725): Resizing a window doesn't resize the contents

https://bugs.webkit.org/show_bug.cgi?id=68730

Reviewed by James Robinson.

Source/WebCore:

Adds a missing contentsResized() call in ScrollView::setFrameRect.

Test: fast/dom/Window/window-resize-contents.html

* platform/ScrollView.cpp:
(WebCore::ScrollView::setFrameRect):

LayoutTests:

Test for resizing of the window triggering resizing of contents.

* fast/dom/Window/window-resize-contents-expected.txt: Added.
* fast/dom/Window/window-resize-contents.html: Added.


git-svn-id: http://svn.webkit.org/repository/webkit/trunk@95897 268f45cc-cd09-0410-ab3c-d52691b4dbfc
parent dca7a664
2011-09-23 Mihai Parparita <mihaip@chromium.org>
[Chromium] REGRESSION (r95725): Resizing a window doesn't resize the contents
https://bugs.webkit.org/show_bug.cgi?id=68730
Reviewed by James Robinson.
Test for resizing of the window triggering resizing of contents.
* fast/dom/Window/window-resize-contents-expected.txt: Added.
* fast/dom/Window/window-resize-contents.html: Added.
2011-09-23 Mihai Parparita <mihaip@chromium.org>
Rebaseline fast/ruby/ruby-text-before-after-content.html for Chromium Mac.
CONSOLE MESSAGE: line 18: Initial reference node dimensions 800 x 600
CONSOLE MESSAGE: line 20: Increasing window size by 10 x 10
CONSOLE MESSAGE: line 22: Post-resize reference node dimensions 810 x 610
This test checks that the yellow reference DOM node (which should be as big as the window) gets resized when the window is resized.
To avoid relayouts and repaints caused by DOM-based logging, it doesn't output anything. Please check the console for confirmation that the node dimensions increase by 10x10.
<!DOCTYPE html>
<html>
<body>
<div id="reference" style="position: absolute; top: 0; bottom: 0; left: 0; right: 0; background: yellow; z-index: 1"></div>
<div style="position: absolute; z-index: 2">
<p>This test checks that the yellow reference DOM node (which should be as
big as the window) gets resized when the window is resized.</p>
<p>To avoid relayouts and repaints caused by DOM-based logging, it doesn't
output anything. Please check the console for confirmation that the node
dimensions increase by 10x10.</p>
</div>
<script>
if (window.layoutTestController)
layoutTestController.dumpAsText();
var referenceNode = document.getElementById('reference');
console.log('Initial reference node dimensions ' +
referenceNode.offsetWidth + ' x ' + referenceNode.offsetHeight);
console.log('Increasing window size by 10 x 10');
window.resizeBy(10, 10);
console.log('Post-resize reference node dimensions ' +
referenceNode.offsetWidth + ' x ' + referenceNode.offsetHeight);
</script>
</body>
</html>
2011-09-23 Mihai Parparita <mihaip@chromium.org>
[Chromium] REGRESSION (r95725): Resizing a window doesn't resize the contents
https://bugs.webkit.org/show_bug.cgi?id=68730
Reviewed by James Robinson.
Adds a missing contentsResized() call in ScrollView::setFrameRect.
Test: fast/dom/Window/window-resize-contents.html
* platform/ScrollView.cpp:
(WebCore::ScrollView::setFrameRect):
2011-09-23 Adam Klein <adamk@chromium.org>
Add ENABLE_MUTATION_OBSERVERS feature flag
......@@ -827,6 +827,9 @@ void ScrollView::setFrameRect(const IntRect& newRect)
frameRectsChanged();
updateScrollbars(scrollOffset());
if (!m_useFixedLayout)
contentsResized();
}
void ScrollView::frameRectsChanged()
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment