linux_aura: Implement the fallback path in IsX11WindowFullScreen.

ui/base/x/x11_util.cc

Index: ui/base/x/x11_util.cc
diff --git a/ui/base/x/x11_util.cc b/ui/base/x/x11_util.cc
index cf5f2a8582919894c93b3150cc7dda38635fde3f..ff1de58b0d6cd34b9091dc9f160ffb31da337ad6 100644
--- a/ui/base/x/x11_util.cc
+++ b/ui/base/x/x11_util.cc
@@ -1320,23 +1320,28 @@ bool IsX11WindowFullScreen(XID window) {
           != atom_properties.end())
     return true;
 
+  gfx::Rect window_rect;
+  if (!ui::GetWindowRect(window, &window_rect))
+    return false;
+
 #if defined(TOOLKIT_GTK)
   // As the last resort, check if the window size is as large as the main
   // screen.
   GdkRectangle monitor_rect;
   gdk_screen_get_monitor_geometry(gdk_screen_get_default(), 0, &monitor_rect);
 
-  gfx::Rect window_rect;
-  if (!ui::GetWindowRect(window, &window_rect))
-    return false;
-
   return monitor_rect.x == window_rect.x() &&
          monitor_rect.y == window_rect.y() &&
          monitor_rect.width == window_rect.width() &&
          monitor_rect.height == window_rect.height();
 #else
-  NOTIMPLEMENTED();
-  return false;
+  // We can't use gfx::Screen here because we don't have an aura::Window. So
+  // instead just look at the size of the default display.
+  ::Display* display = ui::GetXDisplay();

[Daniel Erat, 2013/06/01 03:05:59]

this will probably break in the multi-monitor case -- presumably you'd need to
use xrandr or xinerama to get the bounds of the monitor where the window is
being displayed. at least, i'm assuming that most window managers won't
fullscreen windows across multiple monitors. if you implement the above
suggestion, this is probably enough of a corner case that i wouldn't worry about
it (e.g. i don't think that ion3 sets the fullscreen property, but it also
doesn't do per-monitor fullscreening, so whatever).

+  ::Screen* screen = DefaultScreenOfDisplay(display);
+  int width = WidthOfScreen(screen);
+  int height = HeightOfScreen(screen);
+  return window_rect.size() == gfx::Size(width, height);
 #endif
 }