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

Unified Diff: chrome/browser/ui/cocoa/info_bubble_view.mm

Issue 11103042: New custom styling for action box menu on Os X. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Check that icon exists before setting it. 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/cocoa/info_bubble_view.mm
diff --git a/chrome/browser/ui/cocoa/info_bubble_view.mm b/chrome/browser/ui/cocoa/info_bubble_view.mm
index b5b9ce8f8867a472ddf231ee02e5f04765e0c953..a123813009a803ea0eaec280a7a0f3c172631d09 100644
--- a/chrome/browser/ui/cocoa/info_bubble_view.mm
+++ b/chrome/browser/ui/cocoa/info_bubble_view.mm
@@ -19,6 +19,7 @@
arrowLocation_ = info_bubble::kTopLeft;
alignment_ = info_bubble::kAlignArrowToAnchor;
cornerFlags_ = info_bubble::kRoundedAllCorners;
+ backgroundColor_.reset([NSColor whiteColor]);
Scott Hess - ex-Googler 2012/10/12 23:02:47 I think [NSColor whiteColor] logically returns an
beaudoin 2012/10/13 00:20:55 Done.
}
return self;
}
@@ -71,7 +72,7 @@
[bezier lineToPoint:NSMakePoint(arrowStart.x + info_bubble::kBubbleArrowWidth,
arrowStart.y)];
[bezier closePath];
- [[NSColor whiteColor] set];
+ [backgroundColor_ set];
[bezier fill];
}
@@ -86,4 +87,12 @@
return arrowTip;
}
+- (NSColor*)backgroundColor {
+ return backgroundColor_;
+}
+
+- (void)setBackgroundColor:(NSColor*)backgroundColor {
+ backgroundColor_.reset([backgroundColor retain]);
+}
+
@end

Powered by Google App Engine
This is Rietveld 408576698