Mac: Clean up visibility state tracking

content/browser/renderer_host/render_widget_host_view_mac.mm

 // 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 "content/browser/renderer_host/render_widget_host_view_mac.h"
 
 #import <objc/runtime.h>
 #include <OpenGL/gl.h>
 #include <QuartzCore/QuartzCore.h>
 #include <stdint.h>
@@ skipping 390 matching lines @@
 
 namespace content {
 
 ////////////////////////////////////////////////////////////////////////////////
 // BrowserCompositorMacClient, public:
 
 NSView* RenderWidgetHostViewMac::BrowserCompositorMacGetNSView() const {
   return cocoa_view_;
 }
 
-bool RenderWidgetHostViewMac::BrowserCompositorMacIsVisible() const {
-  return !render_widget_host_->is_hidden();
-}
-
 SkColor RenderWidgetHostViewMac::BrowserCompositorMacGetGutterColor(
     SkColor color) const {
   // When making an element on the page fullscreen the element's background
   // may not match the page's, so use black as the gutter color to avoid
   // flashes of brighter colors during the transition.
   if (render_widget_host_->delegate() &&
       render_widget_host_->delegate()->IsFullscreenForCurrentTab()) {
     return SK_ColorBLACK;
   }
   return color;
@@ skipping 873 matching lines @@
 
 void RenderWidgetHostViewMac::OnSwapCompositorFrame(uint32_t output_surface_id,
                                                     cc::CompositorFrame frame) {
   TRACE_EVENT0("browser", "RenderWidgetHostViewMac::OnSwapCompositorFrame");
 
   last_scroll_offset_ = frame.metadata.root_scroll_offset;
 
   page_at_minimum_scale_ =
       frame.metadata.page_scale_factor == frame.metadata.min_page_scale_factor;
   if (frame.delegated_frame_data) {
-    // TODO(ccameron): This would not be needed if BrowserCompostiorMac were to
-    // correctly subscribe to the show and hide notifications for
-    // RenderWidgetHostImpl. We do not correctly subscribe to these
-    // notifications because we want to set the hide property property only
-    // after all notifications (the thumbnailer in particular) have all
-    // completed.
-    browser_compositor_->SetRenderWidgetHostIsHidden(
-        render_widget_host_->is_hidden());
     browser_compositor_->SwapCompositorFrame(output_surface_id,
                                              std::move(frame));
     SendVSyncParametersToRenderer();
   } else {
     DLOG(ERROR) << "Received unexpected frame type.";
     bad_message::ReceivedBadMessage(render_widget_host_->GetProcess(),
                                     bad_message::RWHVM_UNEXPECTED_FRAME_TYPE);
   }
 }
 
@@ skipping 1919 matching lines @@
 
 // "-webkit-app-region: drag | no-drag" is implemented on Mac by excluding
 // regions that are not draggable. (See ControlRegionView in
 // native_app_window_cocoa.mm). This requires the render host view to be
 // draggable by default.
 - (BOOL)mouseDownCanMoveWindow {
   return YES;
 }
 
 @end