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

Unified Diff: chrome/browser/app_controller_mac.mm

Issue 10837158: mac: Delete more 10.5-only code (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rsesek Created 8 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/app_controller_mac.mm
diff --git a/chrome/browser/app_controller_mac.mm b/chrome/browser/app_controller_mac.mm
index 2cc8390aadc49d9f281728665841595d4b578960..52f8f9576de59519f3ad0057878033d792adf8ca 100644
--- a/chrome/browser/app_controller_mac.mm
+++ b/chrome/browser/app_controller_mac.mm
@@ -1026,12 +1026,7 @@ const AEEventClass kAECloudPrintUninstallClass = 'GCPu';
NSMenu* mainMenu = [NSApp mainMenu];
NSMenuItem* profileMenu = [mainMenu itemWithTag:IDC_PROFILE_MAIN_MENU];
- // On Leopard, hiding main menubar items does not work. This manifests itself
- // in Chromium as squished menu items <http://crbug.com/90753>. To prevent
- // this, remove the Profile menu on Leopard, regardless of the user's
- // multiprofile state.
- if (!ProfileManager::IsMultipleProfilesEnabled() ||
- base::mac::IsOSLeopard()) {
+ if (!ProfileManager::IsMultipleProfilesEnabled()) {
[mainMenu removeItem:profileMenu];
return;
}

Powered by Google App Engine
This is Rietveld 408576698