Add IsVisible to TestBrowserWindowAura.

This is for testing Adaptive Screen Brightness code.

Also changes the Adaptive Screen Brightness code to
use this implementation of IsVisible.

Bug: 820883
Change-Id: Ieb46e1b6cc8e9036c4e510872ee053017426943b
Reviewed-on: https://chromium-review.googlesource.com/1004837
Commit-Queue: Paul Dyson <pdyson@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Michael Giuffrida <michaelpg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549969}

chrome/browser/chromeos/power/ml/adaptive_screen_brightness_manager.cc

@@ -37,7 +37,6 @@
 #include "services/metrics/public/cpp/ukm_source_id.h"
 #include "services/viz/public/interfaces/compositing/video_detector_observer.mojom.h"
 #include "ui/aura/env.h"
-#include "ui/aura/window.h"
 #include "ui/base/user_activity/user_activity_detector.h"
 
 namespace chromeos {
@@ -69,8 +68,7 @@ Browser* GetFocusedOrTopmostVisibleBrowser() {
        browser_iterator != browser_list->end_last_active();
        ++browser_iterator) {
     browser = *browser_iterator;
-    if (browser->profile()->IsOffTheRecord() ||
-        !browser->window()->GetNativeWindow()->IsVisible())
+    if (browser->profile()->IsOffTheRecord() || !browser->window()->IsVisible())
       continue;
 
     if (browser->window()->IsActive())

chrome/test/base/test_browser_window_aura.cc

@@ -48,6 +48,10 @@ void TestBrowserWindowAura::Hide() {
   native_window_->Hide();
 }
 
+bool TestBrowserWindowAura::IsVisible() const {
+  return native_window_->IsVisible();
+}
+
 void TestBrowserWindowAura::Activate() {
   CHECK(native_window_->GetRootWindow())
       << "A TestBrowserWindowAura must have a root window to be activated.";

chrome/test/base/test_browser_window_aura.h

@@ -19,6 +19,7 @@ class TestBrowserWindowAura : public TestBrowserWindow {
   gfx::NativeWindow GetNativeWindow() const override;
   void Show() override;
   void Hide() override;
+  bool IsVisible() const override;
   void Activate() override;
   bool IsActive() const override;
   gfx::Rect GetBounds() const override;