Fixing WebViewVisibilityTests for OOPIF-<webview>.

components/guest_view/browser/guest_view_base.cc

Index: components/guest_view/browser/guest_view_base.cc
diff --git a/components/guest_view/browser/guest_view_base.cc b/components/guest_view/browser/guest_view_base.cc
index 634b1f7900b7eb7b5798e865f6de57b17037a76e..c9725cceda7e839fd9ba22df1ea5092337835db9 100644
--- a/components/guest_view/browser/guest_view_base.cc
+++ b/components/guest_view/browser/guest_view_base.cc
@@ -599,13 +599,6 @@ void GuestViewBase::DidNavigateMainFrame(
     const content::FrameNavigateParams& params) {
   if (attached() && ZoomPropagatesFromEmbedderToGuest())
     SetGuestZoomLevelToMatchEmbedder();
-
-  // TODO(lazyboy): This breaks guest visibility in --site-per-process because

[lazyboy, 2016/01/16 00:46:47]

Great.
Does this take care of all the visibility related issues?
Also, I don't see any tests being enabled in this CL.

[EhsanK, 2016/01/20 00:00:03]

We had three visibility tests where the first two (<webivew>/parent going
invisible) passed after the plumbing. These lines were breaking the third test
which was testHiddenBeforeNavigate in shim/main.

-  // we do not take the widget's visibility into account.  We need to also
-  // stay hidden during "visibility:none" state.
-  if (content::BrowserPluginGuestMode::UseCrossProcessFramesForGuests()) {
-    web_contents()->WasShown();
-  }
 }
 
 void GuestViewBase::ActivateContents(WebContents* web_contents) {