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

chrome/browser/ui/browser_commands.cc

Index: chrome/browser/ui/browser_commands.cc
diff --git a/chrome/browser/ui/browser_commands.cc b/chrome/browser/ui/browser_commands.cc
index 445bd497eeac76baff7c614a1ad493c30dd703b9..abfbf1845cbcb8cc1d431c1e1ecc5c9510b50212 100644
--- a/chrome/browser/ui/browser_commands.cc
+++ b/chrome/browser/ui/browser_commands.cc
@@ -4,6 +4,9 @@
 
 #include "chrome/browser/ui/browser_commands.h"
 
+#include "ash/accelerators/accelerator_controller.h"
+#include "ash/screenshot_delegate.h"
+#include "ash/shell.h"

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

These three #includes should be inside an #if defined(OS_CHROMEOS)

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

Done.

 #include "base/command_line.h"
 #include "base/metrics/histogram.h"
 #include "base/prefs/pref_service.h"
@@ -942,6 +945,16 @@ void OpenTaskManager(Browser* browser) {
   chrome::ShowTaskManager(browser);
 }
 
+void TakeScreenshot() {

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

Surround the body of this function with an #if defined(OS_CHROMEOS) since this
is calls into ash-specific code. For non-CrOS this makes this function a noop.

A similar thing has been done for TogglePagePinnedToStartScreen() for Windows,
as an example. 

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

Done.

+  content::RecordAction(UserMetricsAction("Menu_Take_Screenshot"));
+  ash::ScreenshotDelegate* screenshot_delegate = ash::Shell::GetInstance()->
+      accelerator_controller()->screenshot_delegate();
+  if (screenshot_delegate &&
+      screenshot_delegate->CanTakeScreenshot()) {
+    screenshot_delegate->HandleTakeScreenshotForAllRootWindows();
+  }
+}
+
 void OpenFeedbackDialog(Browser* browser) {
   content::RecordAction(UserMetricsAction("Feedback"));
   chrome::ShowFeedbackPage(browser, std::string(), std::string());