Bug 21101

Summary:

REGRESSION: Overflow scrollbar inactive after blurring and focusing the window

Product:

WebKit

Reporter:

Timothy Hatcher <timothy>

Component:

Layout and Rendering

Assignee:

Dave Hyatt <hyatt>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

hyatt

Priority:

Version:

528+ (Nightly build)

Hardware:

Mac

OS:

OS X 10.5

URL:

http://www.youtube.com/watch?v=XjkCrfylq-E

Attachments:

Description	Flags
Screenshot showing the bug	none
Patch	none
Patch	hyatt: review+

Description Timothy Hatcher 2008-09-25 08:27:20 PDT

The overflow scroll area's scrollbar on YouTube will correctly show an inactive look when the window is blurred. But focusing the window again does not bring back the active scrollbar look. See screenshot.

Comment 1 Timothy Hatcher 2008-09-25 08:27:50 PDT

Created attachment 23796 [details]
Screenshot showing the bug

Comment 2 Dave Hyatt 2008-09-25 18:05:34 PDT

Not the scrollbar's fault.  The focus controller is buggy and lying.

See RenderLayer.cpp:

Page* page = renderer()->document()->frame()->page();
return page && page->focusController()->isActive();

This is a bug that just happens to have been exposed by the scrollbar.

Comment 3 Dave Hyatt 2008-09-25 18:18:38 PDT

Created attachment 23830 [details]
Patch

Comment 4 Dave Hyatt 2008-09-25 18:19:24 PDT

Created attachment 23831 [details]
Patch

Comment 5 Timothy Hatcher 2008-09-25 18:20:48 PDT

Comment on attachment 23830 [details]
Patch

Wow.

Comment 6 Dave Hyatt 2008-09-25 18:22:42 PDT

Comment on attachment 23831 [details]
Patch

Tim meant to plus this one.

Comment 7 Dave Hyatt 2008-09-25 18:23:32 PDT

Fixed in r36919.