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

Unified Diff: chrome/browser/notifications/balloon.cc

Issue 5611001: Fix notifications being too large on OSX. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 10 years 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
« no previous file with comments | « no previous file | chrome/browser/notifications/balloon_host.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/notifications/balloon.cc
diff --git a/chrome/browser/notifications/balloon.cc b/chrome/browser/notifications/balloon.cc
index 86bf50a99a34eceb2e85054a0c5a2329edda51d5..d9f91114733c6bb6bba0d2698b2e180095070b5b 100644
--- a/chrome/browser/notifications/balloon.cc
+++ b/chrome/browser/notifications/balloon.cc
@@ -28,20 +28,7 @@ void Balloon::SetPosition(const gfx::Point& upper_left, bool reposition) {
}
void Balloon::SetContentPreferredSize(const gfx::Size& size) {
- gfx::Size new_size(size);
-#if defined(OS_MACOSX)
- // TODO(levin): Make all of the code that went in with this change to be
- // cross-platform. See http://crbug.com/64720
- // Only allow the size of notifications to grow. This stops the balloon
- // from jumping between sizes due to dynamic content. For example, the
- // balloon's contents may adjust due to changes in
- // document.body.clientHeight.
- new_size.set_height(std::max(new_size.height(), content_size_.height()));
-
- if (content_size_ == new_size)
- return;
-#endif
- collection_->ResizeBalloon(this, new_size);
+ collection_->ResizeBalloon(this, size);
}
void Balloon::set_view(BalloonView* balloon_view) {
« no previous file with comments | « no previous file | chrome/browser/notifications/balloon_host.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698