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

Issue 6523001: Merge 78431 - Some Scrollbar functions assume an attached ScrollableArea but ... (Closed)

Created:
9 years, 10 months ago by Peter Kasting
Modified:
9 years, 7 months ago
Reviewers:
weinig
CC:
chromium-reviews
Base URL:
http://svn.webkit.org/repository/webkit/branches/chromium/648/
Visibility:
Public.

Description

Merge 78431 - Some Scrollbar functions assume an attached ScrollableArea but can be called without one https://bugs.webkit.org/show_bug.cgi?id=54262 Reviewed by Dan Bernstein. Make sure that all calls to the scrollable area are null checked. * platform/Scrollbar.cpp: (WebCore::Scrollbar::autoscrollPressedPart): (WebCore::Scrollbar::moveThumb): (WebCore::Scrollbar::mouseMoved): TBR=weinig@apple.com Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=78505

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+21 lines, -5 lines) Patch
M Source/WebCore/ChangeLog View 1 chunk +14 lines, -0 lines 0 comments Download
M Source/WebCore/platform/Scrollbar.cpp View 3 chunks +7 lines, -5 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
Peter Kasting
9 years, 10 months ago (2011-02-14 22:08:39 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698