Bug 76717

Summary:

WebScrollbar implementation doesn't validate SetValue calls against document size

Product:

WebKit

Reporter:

Raymes Khoury <raymes>

Component:

New Bugs

Assignee:

Nobody <webkit-unassigned>

Status:

UNCONFIRMED

Severity:

Normal

CC:

jam, pkasting

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
Patch	none

Raymes Khoury

Reported 2012-01-20 10:48:30 PST

Chromium WebScrollbarImpl's setValue() accepts values which are outside the document size and setDocumentSize() accepts values even when they are lower than the current scroll position. According to brettw's comment on the chromium.org bug (http://code.google.com/p/chromium/issues/detail?id=89367), the following should be the correct behavior: -If setValue() is passed a value > documentSize, then nothing should happen. -If setDocumentSize() is passed a size < the scroll position, then the scroll position should be capped at the document size.

Attachments
Patch (2.14 KB, patch) 2012-01-20 10:59 PST, Raymes Khoury	no flags	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Raymes Khoury

Comment 1 2012-01-20 10:59:47 PST

Created attachment 123344 [details] Patch

Peter Kasting

Comment 2 2012-01-24 13:45:42 PST

It seems like rather than patching this in the Chromium-specific implementation, we'd want this behavior in the cross-platform code.

Raymes Khoury

Comment 3 2012-01-24 13:54:03 PST

I thought that might be the case. I'll look into how that might work but any ideas/pointers are appreciated.

Note You need to log in before you can comment on or make changes to this bug.