[Webkit-unassigned] [Bug 29167] REGRESSION(r48064): mint.com loses scrollbars after coming out of edit mode

[bugzilla-daemon at webkit.org]

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


Mark Mentovai <mark at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #42665|                            |commit-queue?
               Flag|                            |




--- Comment #15 from Mark Mentovai <mark at chromium.org>  2009-11-06 11:56:41 PDT ---
Created an attachment (id=42665)
 --> (https://bugs.webkit.org/attachment.cgi?id=42665)
Removed parameter name from declaration

Regarding FrameView::layout():
> Can’t you just use m_canHaveScrollbars to choose unconditionally between
> ScrollbarAlwaysOff and ScrollbarAuto?

I don't think so.  The code in here never did that before, it always considered
the existing scrollbar state before proceeding.  I think that the
non-m_firstLayout case depends on this to do the right thing.

I didn't check now, but I'm pretty sure that I did try to make this change back
when I was working on bug 28614, and that it didn't do the right thing.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.