[Webkit-unassigned] [Bug 32534] New: Confusing, possibly buggy code in RenderLayer::updateScrollInfoAfterLayout()

[bugzilla-daemon at webkit.org]

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

           Summary: Confusing, possibly buggy code in
                    RenderLayer::updateScrollInfoAfterLayout()
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: PC
        OS/Version: All
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: WebCore Misc.
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: pkasting at google.com
                CC: hyatt at apple.com, darin at apple.com,
                    simon.fraser at apple.com


(Mentioned this to smfr on IRC, he didn't know what to make of it either and
suggested filing this bug)

At the bottom of RenderLayer::updateScrollInfoAfterLayout() is some code that
resets the horizontal and vertical scrollbars.  The blocks are identical,
except that the horizontal scrollbar gets a setValue() call and the vertical
scrollbar doesn't.  Is this a bug?  If not can we comment about why?

This dates from 2005, when darin checked in r11521 to fix bug 5826.  Before
this change, neither block had a setValue(); the change added the setValue()
call to just the horizontal scrollbar block.

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