Remove window_snapshot.*

chrome/browser/ui/cocoa/panels/panel_stack_window_cocoa.mm

Index: chrome/browser/ui/cocoa/panels/panel_stack_window_cocoa.mm
diff --git a/chrome/browser/ui/cocoa/panels/panel_stack_window_cocoa.mm b/chrome/browser/ui/cocoa/panels/panel_stack_window_cocoa.mm
index a4601eb2da3af5549163a89cec4071d1059c8f47..394281f75f5c3d41412df90b8534906f693485e2 100644
--- a/chrome/browser/ui/cocoa/panels/panel_stack_window_cocoa.mm
+++ b/chrome/browser/ui/cocoa/panels/panel_stack_window_cocoa.mm
@@ -290,6 +290,11 @@ void PanelStackWindowCocoa::BoundsUpdateAnimationEnded() {
 }
 
 void PanelStackWindowCocoa::Minimize() {
+  if (!g_browser_process->local_state()->GetBoolean(
+      prefs::kDisableScreenshots)) {
+    [window_ miniaturize:nil];
+    return;
+  }
   // Provide the custom miniwindow image since there is nothing painted for
   // the background stack window.
   gfx::Size stack_window_size = GetStackWindowBounds().size();
@@ -300,11 +305,8 @@ void PanelStackWindowCocoa::Minimize() {
     Panel* panel = *iter;
     gfx::Rect snapshot_bounds = gfx::Rect(panel->GetBounds().size());
     std::vector<unsigned char> png;
-    if (!chrome::GrabWindowSnapshotForUser(panel->GetNativeWindow(),
-                                           &png,
-                                           snapshot_bounds)) {
+    if (ui::GrabWindowSnapshot(panel->GetNativeWindow(), &png, snapshot_bounds))

[jianli, 2014/01/10 00:47:37]

Should we break out for loop when the function returns false?

[enne (OOO), 2014/01/10 01:29:38]

I don't know this code very well.  Breaking out of the loop preserves the
previous behavior.

       break;
-    }
     gfx::Image snapshot_image = gfx::Image::CreateFrom1xPNGBytes(
         &(png[0]), png.size());
     canvas.DrawImageInt(snapshot_image.AsImageSkia(), 0, y);