Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2757)

Unified Diff: ash/accelerators/accelerator_controller.cc

Issue 10827193: Brushes up the code of screenshot filename. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ash/accelerators/accelerator_controller.cc
diff --git a/ash/accelerators/accelerator_controller.cc b/ash/accelerators/accelerator_controller.cc
index f5598a2b42bd95412f72dbeb0452cb45e4e6fded..0383ef74b2d7169c73cf5c9772fc7ac0fb9ce41b 100644
--- a/ash/accelerators/accelerator_controller.cc
+++ b/ash/accelerators/accelerator_controller.cc
@@ -440,11 +440,8 @@ bool AcceleratorController::PerformAction(int action,
case TAKE_SCREENSHOT:
case TAKE_SCREENSHOT_BY_PRTSCN_KEY:
if (screenshot_delegate_.get() &&
- screenshot_delegate_->CanTakeScreenshot()) {
- Shell::RootWindowList root_windows = Shell::GetAllRootWindows();
- for (size_t i = 0; i < root_windows.size(); ++i)
- screenshot_delegate_->HandleTakeScreenshot(root_windows[i]);
- }
+ screenshot_delegate_->CanTakeScreenshot())
+ screenshot_delegate_->HandleTakeScreenshotForAllRootWindows();
// Return true to prevent propagation of the key event.
return true;
case TAKE_PARTIAL_SCREENSHOT:
« no previous file with comments | « no previous file | ash/accelerators/accelerator_controller_unittest.cc » ('j') | chrome/browser/ui/ash/screenshot_taker.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698