Allow transitions to WebUI pages which are extension urls (new tab page is the relevant example).

chrome/browser/extensions/extension_resource_request_policy_apitest.cc

Index: chrome/browser/extensions/extension_resource_request_policy_apitest.cc
===================================================================
--- chrome/browser/extensions/extension_resource_request_policy_apitest.cc	(revision 149870)
+++ chrome/browser/extensions/extension_resource_request_policy_apitest.cc	(working copy)
@@ -206,6 +206,20 @@
       L"window.domAutomationController.send(document.title)",
     &result));
   EXPECT_EQ("Loading CER:// failed.", result);
+
+  GURL newtab_page("chrome://newtab");
+  GURL accessible_newtab_override(
+      test_server()->GetURL(
+          "files/extensions/api_test/extension_resource_request_policy/"
+          "web_accessible/accessible_newtab_override.html"));

[Mihai Parparita -not on Chrome, 2012/08/07 01:13:12]

Can you name this file something else? It doesn't actually override the NTP, so
the name was confusing.

[Cris Neckar, 2012/08/07 18:45:53]

Done.

+  ui_test_utils::NavigateToURL(browser(), newtab_page);
+  ui_test_utils::NavigateToURLBlockUntilNavigationsComplete(
+      browser(), accessible_newtab_override, 2);
+  ASSERT_TRUE(content::ExecuteJavaScriptAndExtractString(
+    chrome::GetActiveWebContents(browser())->GetRenderViewHost(), L"",
+      L"window.domAutomationController.send(document.title)",
+    &result));
+  EXPECT_EQ("New Tab Page Loaded Successfully", result);

[Mihai Parparita -not on Chrome, 2012/08/07 01:13:12]

This tests that reported bug was fixed, but I'm not seeing a test that
navigations with the PAGE_TRANSITION_LINK transition aren't allowed.

[Mihai Parparita -not on Chrome, 2012/08/07 18:33:37]

Cris explained that this CL doesn't actually disallow those navigations.

 }
 
 IN_PROC_BROWSER_TEST_F(ExtensionResourceRequestPolicyTest,