Refactor GrabWindowSnapshot and GrabWindowSnapshotImpl names to GrabWindowSnapshotForUser and GrabW…

chrome/browser/ui/window_snapshot/window_snapshot.cc

Index: chrome/browser/ui/window_snapshot/window_snapshot.cc
diff --git a/chrome/browser/ui/window_snapshot/window_snapshot.cc b/chrome/browser/ui/window_snapshot/window_snapshot.cc
index ff8759ca132f4556ba24c7b222a97760454200d4..257326353a5a40f1706e14beffceb24cd2f2c89b 100644
--- a/chrome/browser/ui/window_snapshot/window_snapshot.cc
+++ b/chrome/browser/ui/window_snapshot/window_snapshot.cc
@@ -12,17 +12,17 @@ namespace chrome {
 
 // Like GrabWindowSnapshot, but does not check if policy settings allow taking
 // screenshots. Implemented in a platform-specific way.
-bool GrabWindowSnapshotImpl(gfx::NativeWindow window,
-                            std::vector<unsigned char>* png_representation,
-                            const gfx::Rect& snapshot_bounds);
+bool GrabWindowSnapshot(gfx::NativeWindow window,

[grt (UTC plus 2), 2012/08/03 20:50:45]

is this not a duplication of the declaration in the header?  please remove if
so.

[qfel, 2012/08/06 08:49:10]

Oh my blind eyes. Done.

+                        std::vector<unsigned char>* png_representation,
+                        const gfx::Rect& snapshot_bounds);
 
-bool GrabWindowSnapshot(
+bool GrabWindowSnapshotForUser(
     gfx::NativeWindow window,
     std::vector<unsigned char>* png_representation,
     const gfx::Rect& snapshot_bounds) {
   if (g_browser_process->local_state()->GetBoolean(prefs::kDisableScreenshots))
     return false;
-  return GrabWindowSnapshotImpl(window, png_representation, snapshot_bounds);
+  return GrabWindowSnapshot(window, png_representation, snapshot_bounds);
 }
 
 void RegisterScreenshotPrefs(PrefService* service) {