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

Unified Diff: chrome/browser/ui/cocoa/infobars/infobar_container_controller.mm

Issue 435863002: Fix some infobar problems. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Updated CL Created 6 years, 4 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/infobars/infobar_container_controller.mm
diff --git a/chrome/browser/ui/cocoa/infobars/infobar_container_controller.mm b/chrome/browser/ui/cocoa/infobars/infobar_container_controller.mm
index 4e5df309324a4ed1e5d295403d12da23d8247fb5..f61429ea31361904b2c5f226ccc2d0c66fb64b5f 100644
--- a/chrome/browser/ui/cocoa/infobars/infobar_container_controller.mm
+++ b/chrome/browser/ui/cocoa/infobars/infobar_container_controller.mm
@@ -160,4 +160,12 @@
[infobarControllers_ removeObject:controller];
}
+- (void)setMaxTopArrowHeight:(NSInteger)height {
+ containerCocoa_->SetMaxTopArrowHeight(height);
+}
+
+- (NSInteger)defaultArrowHeight {
+ return infobars::InfoBar::kDefaultArrowTargetHeight;
Malcolm 2014/08/08 10:51:51 I prefer not to expose the infobars namespace out
Robert Sesek 2014/08/08 14:32:31 Why do you think it breaks encapsulation? By havin
Malcolm 2014/08/08 15:54:25 Sounds reasonable. From my point of view, InfoBarC
+}
+
@end

Powered by Google App Engine
This is Rietveld 408576698