We observed console.log bleeding into the next test.
Created attachment 155745 [details] Patch
Comment on attachment 155745 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=155745&action=review > Source/WebCore/ChangeLog:10 > + (ColorPicker.prototype._layout): We put the width and height into > + variables so we don't call window.onresize twice. Why window.onresize is called twice with the current code?
(In reply to comment #2) > (From update of attachment 155745 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=155745&action=review > > > Source/WebCore/ChangeLog:10 > > + (ColorPicker.prototype._layout): We put the width and height into > > + variables so we don't call window.onresize twice. > > Why window.onresize is called twice with the current code? My theory is that this._element.offsetHeight is causing a layout, which asks the window to resize immediately.
Comment on attachment 155745 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=155745&action=review >>> Source/WebCore/ChangeLog:10 >>> + variables so we don't call window.onresize twice. >> >> Why window.onresize is called twice with the current code? > > My theory is that this._element.offsetHeight is causing a layout, which asks the window to resize immediately. I don't think so. Layout doesn't change the window size basically. Did you reproduce the problem locally, and confirmed the fix by this change?
(In reply to comment #4) > > My theory is that this._element.offsetHeight is causing a layout, which asks the window to resize immediately. > > I don't think so. Layout doesn't change the window size basically. > Did you reproduce the problem locally, and confirmed the fix by this change? Here is the stack trace. It seems to be true. WebCore::EventHandler::sendResizeEvent() WebCore::FrameView::performPostLayoutTasks() WebCore::FrameView::layout(bool) WebCore::RenderWidget::updateWidgetPosition() WebCore::RenderView::updateWidgetPositions() WebCore::FrameView::performPostLayoutTasks() WebCore::FrameView::layout(bool) WebCore::Document::updateLayout() WebCore::Document::updateLayout() WebCore::Document::updateLayoutIgnorePendingStylesheets() WebCore::Element::offsetHeight() ...
So, did you reproduce the problem locally, and confirmed the fix?
(In reply to comment #6) > So, did you reproduce the problem locally, and confirmed the fix? Yes, I did.
Comment on attachment 155745 [details] Patch ok
Comment on attachment 155745 [details] Patch Clearing flags on attachment: 155745 Committed r124326: <http://trac.webkit.org/changeset/124326>
All reviewed patches have been landed. Closing bug.