Enabling two tests that were disabled in app_shell_browsertests.

extensions/browser/guest_view/web_view/web_view_apitest.cc

Index: extensions/browser/guest_view/web_view/web_view_apitest.cc
diff --git a/extensions/browser/guest_view/web_view/web_view_apitest.cc b/extensions/browser/guest_view/web_view/web_view_apitest.cc
index 8aec95523f92c62cfcfd400f8c8ec4004fa1c5be..2d0a7e9b94ce6a1d1cd3f749b71e2b7c4ccfb6f0 100644
--- a/extensions/browser/guest_view/web_view/web_view_apitest.cc
+++ b/extensions/browser/guest_view/web_view/web_view_apitest.cc
@@ -216,10 +216,7 @@ 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");
 }
 
@@ -409,11 +406,9 @@ 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) {