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

Unified Diff: chrome/browser/ui/cocoa/extensions/extension_installed_bubble_controller.mm

Issue 2717603003: [Mac] Make bubble arrow location/position RTL-aware (Closed)
Patch Set: Now with proper enum formatting Created 3 years, 10 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/extensions/extension_installed_bubble_controller.mm
diff --git a/chrome/browser/ui/cocoa/extensions/extension_installed_bubble_controller.mm b/chrome/browser/ui/cocoa/extensions/extension_installed_bubble_controller.mm
index deed5a8c0b80d661c818e8284b0235366dd7f980..a4d6598648122cd3103e42e14d68d8f07e844fce 100644
--- a/chrome/browser/ui/cocoa/extensions/extension_installed_bubble_controller.mm
+++ b/chrome/browser/ui/cocoa/extensions/extension_installed_bubble_controller.mm
@@ -285,9 +285,9 @@ std::unique_ptr<BubbleUi> ExtensionInstalledBubble::BuildBubbleUi() {
if (installedBubble_ &&
installedBubble_->anchor_position() ==
ExtensionInstalledBubble::ANCHOR_OMNIBOX) {
- [self.bubble setArrowLocation:info_bubble::kTopLeft];
+ [self.bubble setArrowLocation:info_bubble::kTopLeading];
} else {
- [self.bubble setArrowLocation:info_bubble::kTopRight];
+ [self.bubble setArrowLocation:info_bubble::kTopTrailing];
}
// Set appropriate icon, resizing if necessary.

Powered by Google App Engine
This is Rietveld 408576698