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

Unified Diff: chrome/browser/ui/exclusive_access/exclusive_access_bubble.cc

Issue 1972543002: Fullscreen / mouselock bubble: Use new theming even when flag disabled. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@fullscreen-remove-permission
Patch Set: Respond to review. Created 4 years, 7 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/exclusive_access/exclusive_access_bubble.cc
diff --git a/chrome/browser/ui/exclusive_access/exclusive_access_bubble.cc b/chrome/browser/ui/exclusive_access/exclusive_access_bubble.cc
index 142e304104b3b12e1d5fcfca9eaffbe5b0153306..a1850446894825f201b4fa0cef7ea47ba7268626 100644
--- a/chrome/browser/ui/exclusive_access/exclusive_access_bubble.cc
+++ b/chrome/browser/ui/exclusive_access/exclusive_access_bubble.cc
@@ -181,10 +181,15 @@ base::string16 ExclusiveAccessBubble::GetCurrentAllowButtonText() const {
base::string16 ExclusiveAccessBubble::GetInstructionText(
const base::string16& accelerator) const {
+#if defined(OS_MACOSX)
+ // On Mac, if simplified-fullscreen is disabled, the Cocoa code is unable to
+ // deal with the special formatting returned by GetInstructionTextForType, so
+ // just return the old string.
if (!ExclusiveAccessManager::IsSimplifiedFullscreenUIEnabled()) {
return l10n_util::GetStringFUTF16(IDS_FULLSCREEN_PRESS_ESC_TO_EXIT_SENTENCE,
accelerator);
}
+#endif
return exclusive_access_bubble::GetInstructionTextForType(bubble_type_,
accelerator);

Powered by Google App Engine
This is Rietveld 408576698