Bug 64606 – Overlay scrollbars don't pulse when an overflow area is unhidden

• Status: RESOLVED
• Resolution: FIXED
• Component: WebCore Misc.

[Tim Horton]

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.

[Tim Horton]

Created attachment 101077 [details]
patch

[Tim Horton]

I'll reupload when I get home with a proper SVN patch, didn't realize the bots don't like git binary patches.

[Tim Horton]

Created attachment 101087 [details]
revised, using svn-create-patch, hopefully the binaries will work now!

[WebKit Review Bot]

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.

[Tim Horton]

Created attachment 101088 [details]
style fix

[WebKit Review Bot]

Comment on attachment 101088 [details]
style fix

Clearing flags on attachment: 101088

Committed r91146: <http://trac.webkit.org/changeset/91146>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.