ASH: Use virtual screen coordinates in Display::bounds()

chrome/browser/ui/webui/options2/chromeos/display_options_handler.cc

Index: chrome/browser/ui/webui/options2/chromeos/display_options_handler.cc
diff --git a/chrome/browser/ui/webui/options2/chromeos/display_options_handler.cc b/chrome/browser/ui/webui/options2/chromeos/display_options_handler.cc
index 835b742fc43befd89b2a2f936b4022101522a83d..730c845003df688a81093c3d0cfe9c44240401fc 100644
--- a/chrome/browser/ui/webui/options2/chromeos/display_options_handler.cc
+++ b/chrome/browser/ui/webui/options2/chromeos/display_options_handler.cc
@@ -104,10 +104,10 @@ void DisplayOptionsHandler::SendDisplayInfo() {
 
   base::ListValue displays;
   for (size_t i = 0; i < monitor_manager->GetNumDisplays(); ++i) {
-    const gfx::Display& display = monitor_manager->GetDisplayAt(i);
-    const gfx::Rect& bounds = display.bounds();
+    const gfx::Display* display = monitor_manager->GetDisplayAt(i);

[Ben Goodger (Google), 2012/06/27 15:32:52]

i get confused by the various ways you return displays...

sometimes gfx::Display, sometimes const gfx::Display&, sometimes
gfx::Display*...

[oshima, 2012/06/27 21:00:06]

crbug.com/122863

I'm planning to consolidate gfx::Display and DisplaySettingsProvider (
crbug.com/122863), and most of them will be just "const gfx::Dislay*",
except for the case that display has to be modifiable.

+    const gfx::Rect& bounds = display->bounds();
     base::DictionaryValue* js_display = new base::DictionaryValue();
-    js_display->SetDouble("id", display.id());
+    js_display->SetDouble("id", display->id());
     js_display->SetDouble("x", bounds.x());
     js_display->SetDouble("y", bounds.y());
     js_display->SetDouble("width", bounds.width());