Add is_fullscreen to video updates

ash/wm/video_detector.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 "ash/wm/video_detector.h"
 
 #include "ash/shell.h"
+#include "ash/wm/window_util.h"
 #include "ui/aura/env.h"
 #include "ui/aura/root_window.h"
 #include "ui/aura/window.h"
 #include "ui/gfx/rect.h"
 
 namespace ash {
 
 const int VideoDetector::kMinUpdateWidth = 300;
 const int VideoDetector::kMinUpdateHeight = 225;
 const int VideoDetector::kMinFramesPerSecond = 15;
@@ skipping 78 matching lines @@
       kNotifyIntervalSec)
     return;
 
   if (!window->IsVisible())
     return;
 
   gfx::Rect root_bounds = window->GetRootWindow()->bounds();
   if (!window->GetBoundsInRootWindow().Intersects(root_bounds))
     return;
 
-  FOR_EACH_OBSERVER(VideoDetectorObserver, observers_, OnVideoDetected());
+  bool is_fullscreen = wm::IsWindowFullscreen(window);

[Daniel Erat, 2012/09/07 17:07:09]

Does this method work differently on Windows than on Linux?  It looks like the
test is setting ui::SHOW_STATE_FULLSCREEN, so I'm unclear on why this would be.

Regardless of whether this code is needed on Windows, it's still surprising that
it doesn't work there.  Please file a bug to figure out why it's not working and
enable the test on Windows.

+  FOR_EACH_OBSERVER(VideoDetectorObserver,
+                    observers_,
+                    OnVideoDetected(is_fullscreen));
   last_observer_notification_time_ = now;
 }
 
 }  // namespace ash