Change snasphotting API to take in a bounds Rect

chrome/browser/ui/window_snapshot/window_snapshot.h

Index: chrome/browser/ui/window_snapshot/window_snapshot.h
diff --git a/chrome/browser/ui/window_snapshot/window_snapshot.h b/chrome/browser/ui/window_snapshot/window_snapshot.h
index 51786628ed7581962c8730b4c7bed2429e42be13..f8a7800b4e5b63c64474ecbfb20a467808a7ae3e 100644
--- a/chrome/browser/ui/window_snapshot/window_snapshot.h
+++ b/chrome/browser/ui/window_snapshot/window_snapshot.h
@@ -21,6 +21,12 @@ namespace browser {
 gfx::Rect GrabWindowSnapshot(gfx::NativeWindow window,
                              std::vector<unsigned char>* png_representation);
 
+// Grabs a snapshot of the rectangle area |snapshot_bounds| with respect to the
+// top left corner of the designated window and stores a PNG representation
+// into a byte vector. Returns the image bounds.

[jonathan.backer, 2011/11/11 14:41:20]

Document that an empty rect means grab the whole region?

Better yet... factor out the commonality between these two functions into a
function in an anonymous namespace in each of the platform specific impls.

+gfx::Rect GrabWindowSnapshot(gfx::NativeWindow window,
+                             std::vector<unsigned char>* png_representation,
+                             gfx::Rect snapshot_bounds);

[kkania, 2011/11/11 17:06:31]

const gfx::Rect& snapshot_bounds

 }  // namespace browser
 
 #endif  // CHROME_BROWSER_UI_WINDOW_SNAPSHOT_WINDOW_SNAPSHOT_H_