Bug 64606

Summary: Overlay scrollbars don't pulse when an overflow area is unhidden
Product: WebKit Reporter: Tim Horton <thorton>
Component: WebCore Misc.Assignee: Tim Horton <thorton>
Status: RESOLVED FIXED    
Severity: Normal CC: bdakin, simon.fraser, webkit.review.bot
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Macintosh   
OS: Unspecified   
Attachments:
Description Flags
testcase
none
patch
none
revised, using svn-create-patch, hopefully the binaries will work now!
none
style fix none

Description Tim Horton 2011-07-15 11:09:24 PDT
Created attachment 101005 [details]
testcase

<rdar://problem/9390674>

Making the overlay area visible in the attached test case (by pressing the button twice) should cause the overlay scrollbars to pulse.

This doesn't happen currently because we don't keep the system's scrollbar implementation's enabled state up to date with the enabled state of our internal scrollbar representation (we only set it when painting the scrollers!)

This would appear to have regressed in r84682, but that's only because that revision made us only hit the pulsing and scroller-painting code once (previously we'd hit it more than once, and the scrollbars would be enabled the second time through due to having been updated the first time around!)

I have a patch to fix, will post shortly.
Comment 1 Tim Horton 2011-07-15 17:33:02 PDT
Created attachment 101077 [details]
patch
Comment 2 Tim Horton 2011-07-15 17:54:19 PDT
I'll reupload when I get home with a proper SVN patch, didn't realize the bots don't like git binary patches.
Comment 3 Tim Horton 2011-07-15 20:25:27 PDT
Created attachment 101087 [details]
revised, using svn-create-patch, hopefully the binaries will work now!
Comment 4 WebKit Review Bot 2011-07-15 20:28:10 PDT
Attachment 101087 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/WebCore.exp.in', u'Source/W..." exit_code: 1

Source/WebCore/platform/ScrollbarTheme.h:45:  Missing space inside { }.  [whitespace/braces] [5]
Total errors found: 1 in 12 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 5 Tim Horton 2011-07-15 20:30:22 PDT
Created attachment 101088 [details]
style fix
Comment 6 WebKit Review Bot 2011-07-15 22:15:49 PDT
Comment on attachment 101088 [details]
style fix

Clearing flags on attachment: 101088

Committed r91146: <http://trac.webkit.org/changeset/91146>
Comment 7 WebKit Review Bot 2011-07-15 22:15:55 PDT
All reviewed patches have been landed.  Closing bug.