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

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

Issue 2555033003: Add ScopedMacRTL class for Cocoa browser RTL testing (Closed)
Patch Set: Whitespace Created 4 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
Index: chrome/browser/ui/cocoa/l10n_util.mm
diff --git a/chrome/browser/ui/cocoa/l10n_util.mm b/chrome/browser/ui/cocoa/l10n_util.mm
index 73db01d423dc3cc07cc90f4444bb793e6e918b41..7c3b4adb2c699ffc74dda1f93fe079e4f710f159 100644
--- a/chrome/browser/ui/cocoa/l10n_util.mm
+++ b/chrome/browser/ui/cocoa/l10n_util.mm
@@ -4,8 +4,6 @@
#import "chrome/browser/ui/cocoa/l10n_util.h"
-#include "base/i18n/rtl.h"
-#include "base/mac/mac_util.h"
#include "base/strings/string_util.h"
#include "base/strings/sys_string_conversions.h"
#import "third_party/google_toolbox_for_mac/src/AppKit/GTMUILocalizerAndLayoutTweaker.h"
@@ -84,16 +82,4 @@ NSString* TooltipForURLAndTitle(NSString* url, NSString* title) {
return [NSString stringWithFormat:@"%@\n%@", title, url];
}
-const base::Feature kExperimentalMacRTL{"ExperimentalMacRTL",
- base::FEATURE_DISABLED_BY_DEFAULT};
-
-bool ShouldDoExperimentalRTLLayout() {
- return base::i18n::IsRTL() &&
- base::FeatureList::IsEnabled(kExperimentalMacRTL);
-}
-
-bool ShouldFlipWindowControlsInRTL() {
- return ShouldDoExperimentalRTLLayout() && base::mac::IsAtLeastOS10_12();
-}
-
} // namespace cocoa_l10n_util

Powered by Google App Engine
This is Rietveld 408576698