[Android] Stop hiding the RWHV layer subtree when hiding the widget

content/browser/renderer_host/delegated_frame_evictor.cc

 // Copyright 2013 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/delegated_frame_evictor.h"
 
 #include "base/logging.h"
 
 namespace content {
 
 DelegatedFrameEvictor::DelegatedFrameEvictor(
     DelegatedFrameEvictorClient* client)
-    : client_(client), has_frame_(false) {}
+    : client_(client), has_frame_(false), visible_(false) {}
 
 DelegatedFrameEvictor::~DelegatedFrameEvictor() { DiscardedFrame(); }
 
 void DelegatedFrameEvictor::SwappedFrame(bool visible) {
   has_frame_ = true;

[no sievers, 2015/03/20 21:48:24]

Should this DCHECK_EQ(visible, visible_)?

[jdduke (slow), 2015/03/20 22:48:18]

Good catch, I think so, yes. Done.

   RendererFrameManager::GetInstance()->AddFrame(this, visible);
 }
 
 void DelegatedFrameEvictor::DiscardedFrame() {
   RendererFrameManager::GetInstance()->RemoveFrame(this);
   has_frame_ = false;
 }
 
 void DelegatedFrameEvictor::SetVisible(bool visible) {
+  if (visible_ == visible)
+    return;
+  visible_ = visible;
   if (has_frame_) {
     if (visible) {
-      RendererFrameManager::GetInstance()->LockFrame(this);
+      LockFrame();
     } else {
-      RendererFrameManager::GetInstance()->UnlockFrame(this);
+      UnlockFrame();
     }
   }
 }
 
 void DelegatedFrameEvictor::LockFrame() {
   DCHECK(has_frame_);
   RendererFrameManager::GetInstance()->LockFrame(this);
 }
 
 void DelegatedFrameEvictor::UnlockFrame() {
   DCHECK(has_frame_);
   RendererFrameManager::GetInstance()->UnlockFrame(this);
 }
 
 void DelegatedFrameEvictor::EvictCurrentFrame() {
   client_->EvictDelegatedFrame();
 }
 
 }  // namespace content