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

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

Issue 11269022: Add Vector2d classes that represent offsets, instead of using Point. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: RenderText fixup 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/notifications/balloon_collection_impl.cc
diff --git a/chrome/browser/notifications/balloon_collection_impl.cc b/chrome/browser/notifications/balloon_collection_impl.cc
index 4057eef542528dbf493139433cd6321e9df6b7aa..541c12b211aac21066dbedb4cc74e7b9ee4668b7 100644
--- a/chrome/browser/notifications/balloon_collection_impl.cc
+++ b/chrome/browser/notifications/balloon_collection_impl.cc
@@ -137,7 +137,7 @@ void BalloonCollectionImpl::OnBalloonClosed(Balloon* source) {
Balloons::const_iterator it = balloons.begin();
if (layout_.RequiresOffsets()) {
- gfx::Point offset;
+ gfx::Vector2d offset;
bool apply_offset = false;
while (it != balloons.end()) {
if (*it == source) {
@@ -249,7 +249,7 @@ void BalloonCollectionImpl::CancelOffsets() {
for (Balloons::const_iterator it = balloons.begin();
it != balloons.end();
++it)
- (*it)->set_offset(gfx::Point(0, 0));
+ (*it)->set_offset(gfx::Vector2d());
PositionBalloons(true);
}

Powered by Google App Engine
This is Rietveld 408576698