+2012-04-16 Andrei Burago <aburago@chromium.org>
+
+ Auto-sized frames may be taller than expected
+ https://bugs.webkit.org/show_bug.cgi?id=84106
+
+ Reviewed by David Levin.
+
+ No new tests. The repro steps require using Chrome notifications on Win.
+
+ * page/FrameView.cpp:
+ (WebCore::FrameView::autoSizeIfEnabled):
+
2012-04-13 James Robinson <jamesr@chromium.org>
[chromium] Expose WebVideoLayer to Platform API and port WebMediaPlayerClientImpl to using it
if (!m_didRunAutosize)
resize(frameRect().width(), m_minAutoSize.height());
+ IntSize size = frameRect().size();
+
// Do the resizing twice. The first time is basically a rough calculation using the preferred width
// which may result in a height change during the second iteration.
for (int i = 0; i < 2; i++) {
// Update various sizes including contentsSize, scrollHeight, etc.
document->updateLayoutIgnorePendingStylesheets();
- IntSize size = frameRect().size();
int width = documentView->minPreferredLogicalWidth();
int height = documentRenderBox->scrollHeight();
IntSize newSize(width, height);