Enabling two tests that were disabled in app_shell_browsertests.

extensions/browser/guest_view/web_view/web_view_apitest.cc

 // Copyright 2014 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 "extensions/browser/guest_view/web_view/web_view_apitest.h"
 
 #include "base/path_service.h"
 #include "base/strings/string_util.h"
 #include "base/strings/stringprintf.h"
 #include "content/public/browser/render_process_host.h"
@@ skipping 198 matching lines @@
 }
 
 IN_PROC_BROWSER_TEST_F(WebViewAPITest, TestDestroyOnEventListener) {
   RunTest("testDestroyOnEventListener", "web_view/apitest");
 }
 
 IN_PROC_BROWSER_TEST_F(WebViewAPITest, TestDisplayNoneWebviewLoad) {
   RunTest("testDisplayNoneWebviewLoad", "web_view/apitest");
 }
 
-// TODO(lfg): Re-enable this test once the fix for webview leaking
-// RenderProcessHost has landed. See http://crbug.com/419020 .
-IN_PROC_BROWSER_TEST_F(WebViewAPITest,
-                       DISABLED_TestDisplayNoneWebviewRemoveChild) {
+IN_PROC_BROWSER_TEST_F(WebViewAPITest, TestDisplayNoneWebviewRemoveChild) {
   RunTest("testDisplayNoneWebviewRemoveChild", "web_view/apitest");
 }
 
 IN_PROC_BROWSER_TEST_F(WebViewAPITest, TestEventName) {
   RunTest("testEventName", "web_view/apitest");
 }
 
 IN_PROC_BROWSER_TEST_F(WebViewAPITest, TestExecuteScript) {
   RunTest("testExecuteScript", "web_view/apitest");
 }
@@ skipping 169 matching lines @@
 }
 
 IN_PROC_BROWSER_TEST_F(WebViewAPITest, TestReloadAfterTerminate) {
   RunTest("testReloadAfterTerminate", "web_view/apitest");
 }
 
 IN_PROC_BROWSER_TEST_F(WebViewAPITest, TestRemoveSrcAttribute) {
   RunTest("testRemoveSrcAttribute", "web_view/apitest");
 }
 
-// TODO(lfg): Re-enable this test once the fix for webview leaking
-// RenderProcessHost has landed. See http://crbug.com/419020 .
-IN_PROC_BROWSER_TEST_F(WebViewAPITest,
-                       DISABLED_TestRemoveWebviewAfterNavigation) {
+IN_PROC_BROWSER_TEST_F(WebViewAPITest, TestRemoveWebviewAfterNavigation) {
   RunTest("testRemoveWebviewAfterNavigation", "web_view/apitest");
+  GetGuestViewManager()->WaitForGuestDeleted();

[Fady Samuel, 2014/10/03 20:51:36]

Can we move this to the bottom of RunTest?

[Fady Samuel, 2014/10/03 20:59:28]

LGTM. I like the idea of observing that every guest created gets destroyed in a
future CL. That seems like a great extra way to test for leaks on all tests.

 }
 
 IN_PROC_BROWSER_TEST_F(WebViewAPITest, TestResizeWebviewResizesContent) {
   RunTest("testResizeWebviewResizesContent", "web_view/apitest");
 }
 
 IN_PROC_BROWSER_TEST_F(WebViewAPITest, TestTerminateAfterExit) {
   RunTest("testTerminateAfterExit", "web_view/apitest");
 }
 
 }  // namespace extensions