While building the system menu for the Chrome browser frame we should add the separator after the

TaskManager option only if the TaskManager option was added to the menu in the first place.

This causes a DCHECK for consecutive separators to fire on Chrome Windows 8 metro which does not have the
TaskManager option in the menu.

BUG=none
Review URL: https://codereview.chromium.org/11788007

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@175245 0039d316-1c4b-4281-b951-d872f2087c98

chrome/browser/ui/views/frame/browser_frame_win.cc

@@ -471,8 +471,8 @@ void BrowserFrameWin::BuildSystemMenuForBrowserWindow() {
   if (chrome::CanOpenTaskManager()) {
     system_menu_contents_->AddItemWithStringId(IDC_TASK_MANAGER,
                                                IDS_TASK_MANAGER);
+    system_menu_contents_->AddSeparator(ui::NORMAL_SEPARATOR);
   }
-  system_menu_contents_->AddSeparator(ui::NORMAL_SEPARATOR);
   system_menu_contents_->AddItemWithStringId(IDC_RESTORE_TAB, IDS_RESTORE_TAB);
   system_menu_contents_->AddItemWithStringId(IDC_NEW_TAB, IDS_NEW_TAB);
   AddFrameToggleItems();