Take screenshot of active window

ash/accelerators/accelerator_controller.cc

 // Copyright (c) 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 "ash/accelerators/accelerator_controller.h"
 
 #include <algorithm>
 #include <cmath>
 #include <string>
 #include <utility>
@@ skipping 441 matching lines @@
   return ime_control_delegate &&
       ime_control_delegate->CanSwitchIme(accelerator);
 }
 
 void HandleSwitchIme(ImeControlDelegate* ime_control_delegate,
                      const ui::Accelerator& accelerator) {
   base::RecordAction(UserMetricsAction("Accel_Switch_Ime"));
   ime_control_delegate->HandleSwitchIme(accelerator);
 }
 
+void HandleTakeActiveWindowScreenshot(ScreenshotDelegate* screenshot_delegate) {
+  base::RecordAction(UserMetricsAction("Accel_Take_Window_Screenshot"));
+  DCHECK(screenshot_delegate);
+  aura::Window* current_window = wm::GetActiveWindow();
+  if (screenshot_delegate->CanTakeScreenshot())

[sky, 2016/03/25 15:27:53]

Early out if current_window is null? Also, current_window is a bit misleading as
it's more of active_window. Although just window is fine by me too.

[Qiang(Joe) Xu, 2016/03/25 19:09:40]

I added:
if (!active_window)
  return;

once this is added, AcceleratorControllerTest.GlobalAccelerators
EXPECT_EQ(1, delegate->handle_take_window_screenshot_count()); --> will fail
(unable to terminate process group), I temporarily remove this test.

+    screenshot_delegate->HandleTakeWindowScreenshot(current_window);
+}
+
 void HandleTakePartialScreenshot(ScreenshotDelegate* screenshot_delegate) {
   base::RecordAction(UserMetricsAction("Accel_Take_Partial_Screenshot"));
   DCHECK(screenshot_delegate);
   Shell::GetInstance()
       ->partial_screenshot_controller()
       ->StartPartialScreenshotSession(screenshot_delegate);
 }
 
 void HandleTakeScreenshot(ScreenshotDelegate* screenshot_delegate) {
   base::RecordAction(UserMetricsAction("Accel_Take_Screenshot"));
@@ skipping 602 matching lines @@
     case NEW_TAB:
     case NEW_WINDOW:
     case OPEN_FEEDBACK_PAGE:
     case PRINT_UI_HIERARCHIES:
     case RESTORE_TAB:
     case ROTATE_SCREEN:
     case ROTATE_WINDOW:
     case SHOW_KEYBOARD_OVERLAY:
     case SHOW_SYSTEM_TRAY_BUBBLE:
     case SHOW_TASK_MANAGER:
+    case TAKE_ACTIVE_WINDOW_SCREENSHOT:
     case TAKE_PARTIAL_SCREENSHOT:
     case TAKE_SCREENSHOT:
     case TOGGLE_FULLSCREEN:
     case TOGGLE_MAXIMIZED:
     case TOGGLE_OVERVIEW:
     case WINDOW_MINIMIZE:
 #if defined(OS_CHROMEOS)
     case BRIGHTNESS_DOWN:
     case BRIGHTNESS_UP:
     case DISABLE_GPU_WATCHDOG:
@@ skipping 151 matching lines @@
       break;
     case SHOW_SYSTEM_TRAY_BUBBLE:
       HandleShowSystemTrayBubble();
       break;
     case SHOW_TASK_MANAGER:
       HandleShowTaskManager();
       break;
     case SWITCH_IME:
       HandleSwitchIme(ime_control_delegate_.get(), accelerator);
       break;
+    case TAKE_ACTIVE_WINDOW_SCREENSHOT:
+      HandleTakeActiveWindowScreenshot(screenshot_delegate_.get());
+      break;
     case TAKE_PARTIAL_SCREENSHOT:
       HandleTakePartialScreenshot(screenshot_delegate_.get());
       break;
     case TAKE_SCREENSHOT:
       HandleTakeScreenshot(screenshot_delegate_.get());
       break;
     case TOGGLE_APP_LIST:
       HandleToggleAppList(accelerator);
       break;
     case TOGGLE_FULLSCREEN:
@@ skipping 165 matching lines @@
 }
 
 void AcceleratorController::SetKeyboardBrightnessControlDelegate(
     scoped_ptr<KeyboardBrightnessControlDelegate>
     keyboard_brightness_control_delegate) {
   keyboard_brightness_control_delegate_ =
       std::move(keyboard_brightness_control_delegate);
 }
 
 }  // namespace ash