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

Side by Side Diff: chrome/browser/ui/views/frame/browser_frame_win.cc

Issue 11788007: While building the system menu for the Chrome browser frame we should add the separator after the (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 7 years, 11 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/ui/views/frame/browser_frame_win.h" 5 #include "chrome/browser/ui/views/frame/browser_frame_win.h"
6 6
7 #include <dwmapi.h> 7 #include <dwmapi.h>
8 #include <shellapi.h> 8 #include <shellapi.h>
9 #include <set> 9 #include <set>
10 10
(...skipping 453 matching lines...) Expand 10 before | Expand all | Expand 10 after
464 464
465 DwmExtendFrameIntoClientArea(GetNativeView(), &margins); 465 DwmExtendFrameIntoClientArea(GetNativeView(), &margins);
466 } 466 }
467 467
468 void BrowserFrameWin::BuildSystemMenuForBrowserWindow() { 468 void BrowserFrameWin::BuildSystemMenuForBrowserWindow() {
469 system_menu_contents_->AddSeparator(ui::NORMAL_SEPARATOR); 469 system_menu_contents_->AddSeparator(ui::NORMAL_SEPARATOR);
470 470
471 if (chrome::CanOpenTaskManager()) { 471 if (chrome::CanOpenTaskManager()) {
472 system_menu_contents_->AddItemWithStringId(IDC_TASK_MANAGER, 472 system_menu_contents_->AddItemWithStringId(IDC_TASK_MANAGER,
473 IDS_TASK_MANAGER); 473 IDS_TASK_MANAGER);
474 system_menu_contents_->AddSeparator(ui::NORMAL_SEPARATOR);
474 } 475 }
475 system_menu_contents_->AddSeparator(ui::NORMAL_SEPARATOR);
476 system_menu_contents_->AddItemWithStringId(IDC_RESTORE_TAB, IDS_RESTORE_TAB); 476 system_menu_contents_->AddItemWithStringId(IDC_RESTORE_TAB, IDS_RESTORE_TAB);
477 system_menu_contents_->AddItemWithStringId(IDC_NEW_TAB, IDS_NEW_TAB); 477 system_menu_contents_->AddItemWithStringId(IDC_NEW_TAB, IDS_NEW_TAB);
478 AddFrameToggleItems(); 478 AddFrameToggleItems();
479 // If it's a regular browser window with tabs, we don't add any more items, 479 // If it's a regular browser window with tabs, we don't add any more items,
480 // since it already has menus (Page, Chrome). 480 // since it already has menus (Page, Chrome).
481 } 481 }
482 482
483 void BrowserFrameWin::BuildSystemMenuForAppOrPopupWindow() { 483 void BrowserFrameWin::BuildSystemMenuForAppOrPopupWindow() {
484 Browser* browser = browser_view()->browser(); 484 Browser* browser = browser_view()->browser();
485 if (browser->is_app() && chrome::CanOpenTaskManager()) { 485 if (browser->is_app() && chrome::CanOpenTaskManager()) {
(...skipping 129 matching lines...) Expand 10 before | Expand all | Expand 10 after
615 615
616 //////////////////////////////////////////////////////////////////////////////// 616 ////////////////////////////////////////////////////////////////////////////////
617 // NativeBrowserFrame, public: 617 // NativeBrowserFrame, public:
618 618
619 // static 619 // static
620 NativeBrowserFrame* NativeBrowserFrame::CreateNativeBrowserFrame( 620 NativeBrowserFrame* NativeBrowserFrame::CreateNativeBrowserFrame(
621 BrowserFrame* browser_frame, 621 BrowserFrame* browser_frame,
622 BrowserView* browser_view) { 622 BrowserView* browser_view) {
623 return new BrowserFrameWin(browser_frame, browser_view); 623 return new BrowserFrameWin(browser_frame, browser_view);
624 } 624 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698