Change snasphotting API to take in a bounds Rect

chrome/browser/ui/webui/bug_report_ui.cc

Index: chrome/browser/ui/webui/bug_report_ui.cc
diff --git a/chrome/browser/ui/webui/bug_report_ui.cc b/chrome/browser/ui/webui/bug_report_ui.cc
index d9abc7e72259f7849e5f585d1a9b37e337fb25e0..e079ee179543b60caf0097a9734bc97b7df724b2 100644
--- a/chrome/browser/ui/webui/bug_report_ui.cc
+++ b/chrome/browser/ui/webui/bug_report_ui.cc
@@ -139,8 +139,11 @@ void ShowHtmlBugReportView(Browser* browser,
   last_screenshot_png->clear();
 
   gfx::NativeWindow native_window = browser->window()->GetNativeHandle();
-  BugReportUtil::SetScreenshotSize(
-      browser::GrabWindowSnapshot(native_window, last_screenshot_png));

[Nico, 2011/11/15 17:10:09]

nit: I liked the old interface better for the use case in this file. I would've
left the old interface intact, added a new function for your use case, and then
implemented the old one in terms of the new one. That's just me, though :-)

+  gfx::Rect window_bounds = browser->window()->GetBounds();
+  bool success = browser::GrabWindowSnapshot(native_window,
+                                             last_screenshot_png,
+                                             window_bounds);

[sky, 2011/11/15 16:58:08]

Don't you need to convert window_bounds to have an origin of 0x0?

+  BugReportUtil::SetScreenshotSize(success ? window_bounds : gfx::Rect());
 
   std::string bug_report_url = std::string(chrome::kChromeUIBugReportURL) +
       "#" + base::IntToString(browser->active_index()) +