diff options
| author | ben@chromium.org <ben@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-02-23 17:50:56 +0000 |
|---|---|---|
| committer | ben@chromium.org <ben@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-02-23 17:50:56 +0000 |
| commit | 0a119c96c644fce867059839ab24fb7cc38e4047 (patch) | |
| tree | 40c94228fe028f0df40a9a6522fd77c0123211de /views/animation | |
| parent | 529623ed3b5c444fa30203fb982c2e69c97825a5 (diff) | |
| download | chromium_src-0a119c96c644fce867059839ab24fb7cc38e4047.zip chromium_src-0a119c96c644fce867059839ab24fb7cc38e4047.tar.gz chromium_src-0a119c96c644fce867059839ab24fb7cc38e4047.tar.bz2 | |
Dramatically simplify view painting by getting rid of RootView's redundant knowledge of invalid rects.The OS is capable of doing this for us (see: InvalidateRect, etc.), so we should just defer to it.I was also able to remove all of the custom logic in WindowWin::OnNCPaint and replace it with a simple SetMsgHandled check.http://crbug.com/72040TEST=none
Review URL: http://codereview.chromium.org/6469096
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@75758 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'views/animation')
| -rw-r--r-- | views/animation/bounds_animator.cc | 2 | ||||
| -rw-r--r-- | views/animation/bounds_animator_unittest.cc | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/views/animation/bounds_animator.cc b/views/animation/bounds_animator.cc index 20a6616..09d880e 100644 --- a/views/animation/bounds_animator.cc +++ b/views/animation/bounds_animator.cc @@ -236,7 +236,7 @@ void BoundsAnimator::AnimationContainerProgressed( // Adjust for rtl. repaint_bounds_.set_x(parent_->GetMirroredXWithWidthInView( repaint_bounds_.x(), repaint_bounds_.width())); - parent_->SchedulePaintInRect(repaint_bounds_, false); + parent_->SchedulePaintInRect(repaint_bounds_); repaint_bounds_.SetRect(0, 0, 0, 0); } diff --git a/views/animation/bounds_animator_unittest.cc b/views/animation/bounds_animator_unittest.cc index 7dc0f37..a899f8e 100644 --- a/views/animation/bounds_animator_unittest.cc +++ b/views/animation/bounds_animator_unittest.cc @@ -75,7 +75,7 @@ bool OwnedDelegate::canceled_ = false; class TestView : public views::View { public: TestView() {} - virtual void SchedulePaintInRect(const gfx::Rect& r, bool urgent) { + virtual void SchedulePaintInRect(const gfx::Rect& r) { if (dirty_rect_.IsEmpty()) dirty_rect_ = r; else |