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

Unified Diff: chrome/browser/ui/views/notifications/balloon_view_views.cc

Issue 11148007: views: Fix a bunch of dchecks. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fixes Created 8 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/views/notifications/balloon_view_views.cc
diff --git a/chrome/browser/ui/views/notifications/balloon_view_views.cc b/chrome/browser/ui/views/notifications/balloon_view_views.cc
index 94716980799b62d26ce1ebfb5661158d4ef435e3..391b646e05d61a9337f45813bd4f18249b6b812e 100644
--- a/chrome/browser/ui/views/notifications/balloon_view_views.cc
+++ b/chrome/browser/ui/views/notifications/balloon_view_views.cc
@@ -169,10 +169,9 @@ void BalloonViewImpl::OnWorkAreaChanged() {
collection_->DisplayChanged();
}
-void BalloonViewImpl::ButtonPressed(views::Button* sender,
- const ui::Event&) {
+void BalloonViewImpl::ButtonPressed(views::Button* sender, const ui::Event&) {
// The only button currently is the close button.
- DCHECK(sender == close_button_);
+ DCHECK_EQ(close_button_, sender);
Close(true);
}
@@ -238,7 +237,7 @@ void BalloonViewImpl::Update() {
}
void BalloonViewImpl::AnimationProgressed(const ui::Animation* animation) {
- DCHECK(animation == animation_.get());
+ DCHECK_EQ(animation_.get(), animation);
// Linear interpolation from start to end position.
double e = animation->GetCurrentValue();

Powered by Google App Engine
This is Rietveld 408576698