Adding a menu item to take a screenshot under "More Tools" submenu in the Wrench menu.

chrome/browser/ui/browser_command_controller.cc

Index: chrome/browser/ui/browser_command_controller.cc
diff --git a/chrome/browser/ui/browser_command_controller.cc b/chrome/browser/ui/browser_command_controller.cc
index d6a20004e51ee4c50c827aafae7ebba977e109c5..8cac2c3f05f0b6a3cbed010475494c58e8addbdf 100644
--- a/chrome/browser/ui/browser_command_controller.cc
+++ b/chrome/browser/ui/browser_command_controller.cc
@@ -686,6 +686,11 @@ void BrowserCommandController::ExecuteCommandWithDisposition(
     case IDC_TASK_MANAGER:
       OpenTaskManager(browser_);
       break;
+#if defined(OS_CHROMEOS)

[tdanderson, 2014/01/09 20:28:40]

If you are surrounding the body of TakeScreenshot() with #if
defined(OS_CHROMEOS) (see my suggestion in browser_commands.cc) then I think you
can remove the ifdef here.

[mfomitchev, 2014/01/09 21:11:45]

Done.

+    case IDC_TAKE_SCREENSHOT:
+      TakeScreenshot();
+      break;
+#endif
 #if defined(GOOGLE_CHROME_BUILD)
     case IDC_FEEDBACK:
       OpenFeedbackDialog(browser_);
@@ -952,6 +957,7 @@ void BrowserCommandController::InitCommandState() {
   command_updater_.UpdateCommandEnabled(IDC_CREATE_SHORTCUTS, false);
   UpdateCommandsForDevTools();
   command_updater_.UpdateCommandEnabled(IDC_TASK_MANAGER, CanOpenTaskManager());
+  command_updater_.UpdateCommandEnabled(IDC_TAKE_SCREENSHOT, true);

[tdanderson, 2014/01/09 20:28:40]

I think this line should be ok as-is given that you're only adding the "take
screenshot" item to the wrench menu in the first place for CrOS.

[mfomitchev, 2014/01/09 21:11:45]

Done.

   command_updater_.UpdateCommandEnabled(IDC_SHOW_HISTORY,
                                         !profile()->IsGuestSession());
   command_updater_.UpdateCommandEnabled(IDC_SHOW_DOWNLOADS, true);