Refactor GrabWindowSnapshot and GrabWindowSnapshotImpl names to GrabWindowSnapshotForUser and GrabW…

chrome/browser/ui/window_snapshot/window_snapshot.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 "chrome/browser/ui/window_snapshot/window_snapshot.h"
 
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/prefs/pref_service.h"
 #include "chrome/common/pref_names.h"
 
 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) {
   service->RegisterBooleanPref(prefs::kDisableScreenshots, false);
 }
 
 }  // namespace browser