Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(536)

Issue 7582015: Merge 92316 - Chromium Mac: Make sure scrollbars flash when web page loads (Closed)

Created:
9 years, 4 months ago by Nico
Modified:
9 years, 4 months ago
Reviewers:
Nico
CC:
chromium-reviews
Base URL:
http://svn.webkit.org/repository/webkit/branches/chromium/835/
Visibility:
Public.

Description

Merge 92316 - Chromium Mac: Make sure scrollbars flash when web page loads https://bugs.webkit.org/show_bug.cgi?id=65586 On slow web pages scrollbars wouldn't flash when the page was done loading. The problem was that we were flashing the scrollbar 0.1 second after the load operation had started. If the page was slow to load then we might not have scrollbars to flash. To work around this I added an extra check to make sure that we had indeed finished loading the page before flashing the scrollbars. Patch by Sailesh Agrawal <sail@chromium.org>; on 2011-08-03 Reviewed by James Robinson. * platform/chromium/ScrollAnimatorChromiumMac.mm: (WebCore::ScrollAnimatorChromiumMac::initialScrollbarPaintTimerFired): TBR=commit-queue@webkit.org Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=92485

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1782 lines, -24 lines) Patch
M Source/WebCore/ChangeLog View 1 chunk +1778 lines, -23 lines 0 comments Download
M Source/WebCore/platform/chromium/ScrollAnimatorChromiumMac.mm View 1 chunk +4 lines, -1 line 0 comments Download

Messages

Total messages: 1 (0 generated)
Nico
9 years, 4 months ago (2011-08-05 17:56:49 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698