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

chrome/browser/ui/browser_commands.cc

 // Copyright 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/browser_commands.h"
 
 #include "base/command_line.h"
 #include "base/metrics/histogram.h"
 #include "base/prefs/pref_service.h"
 #include "base/strings/utf_string_conversions.h"
@@ skipping 58 matching lines @@
 #include "content/public/browser/render_view_host.h"
 #include "content/public/browser/user_metrics.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/browser/web_contents_view.h"
 #include "content/public/common/renderer_preferences.h"
 #include "content/public/common/url_constants.h"
 #include "content/public/common/url_utils.h"
 #include "net/base/escape.h"
 #include "webkit/common/user_agent/user_agent_util.h"
 
+#if defined(OS_CHROMEOS)
+#include "ash/accelerators/accelerator_controller.h"
+#include "ash/screenshot_delegate.h"
+#include "ash/shell.h"
+#endif
+
 #if defined(OS_WIN)
 #include "chrome/browser/ui/metro_pin_tab_helper_win.h"
 #include "win8/util/win8_util.h"
 #endif
 
 #if defined(ENABLE_PRINTING)
 #if defined(ENABLE_FULL_PRINTING)
 #include "chrome/browser/printing/print_preview_dialog_controller.h"
 #include "chrome/browser/printing/print_view_manager.h"
 #else
@@ skipping 846 matching lines @@
 #else
   return true;
 #endif
 }
 
 void OpenTaskManager(Browser* browser) {
   content::RecordAction(UserMetricsAction("TaskManager"));
   chrome::ShowTaskManager(browser);
 }
 
+void TakeScreenshot() {

[sky, 2014/01/10 00:02:52]

If this is chromeos specific, then move to browser_commands_chromeos.

[mfomitchev, 2014/01/10 18:06:50]

Done.

+#if defined(OS_CHROMEOS)

[sadrul, 2014/01/09 21:39:33]

I was going to suggest using USE_ASH instead of OS_CHROMEOS. But looks like the
accelerators for taking screenshot are also available on chromeos only (I wonder
why). So this is OK

[mfomitchev, 2014/01/09 22:07:23]

I think that's because on other OSes there are other means to take a screenshot,
like 'Print Scrn' on Windows. I guess this is seen as OS functionality rather
than browser functionality (which makes it weird to put the menu item within the
browser menu, but we don't really have a better place to put it now).

+  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();
+  }
+#endif
+}
+
 void OpenFeedbackDialog(Browser* browser) {
   content::RecordAction(UserMetricsAction("Feedback"));
   chrome::ShowFeedbackPage(browser, std::string(), std::string());
 }
 
 void ToggleBookmarkBar(Browser* browser) {
   content::RecordAction(UserMetricsAction("ShowBookmarksBar"));
   ToggleBookmarkBarWhenVisible(browser->profile());
 }
 
@@ skipping 208 matching lines @@
           Browser::TYPE_POPUP, app_name, gfx::Rect(), browser->profile(),
           browser->host_desktop_type()));
   app_browser->tab_strip_model()->AppendWebContents(contents, true);
 
   contents->GetMutableRendererPrefs()->can_accept_load_drops = false;
   contents->GetRenderViewHost()->SyncRendererPrefs();
   app_browser->window()->Show();
 }
 
 }  // namespace chrome