Index: chrome/browser/extensions/extension_resource_request_policy_apitest.cc |
diff --git a/chrome/browser/extensions/extension_resource_request_policy_apitest.cc b/chrome/browser/extensions/extension_resource_request_policy_apitest.cc |
index db5398f9dd560753d4587404af6db57d3cfd925a..bebeab6fa67cd3ee2cab23e218c847abd030102f 100644 |
--- a/chrome/browser/extensions/extension_resource_request_policy_apitest.cc |
+++ b/chrome/browser/extensions/extension_resource_request_policy_apitest.cc |
@@ -349,11 +349,8 @@ IN_PROC_BROWSER_TEST_F(ExtensionResourceRequestPolicyTest, |
// The iframe should not load |private_page|, which is not web-accessible. |
// |
- // TODO(alexmos): The failure mode differs on whether or not |
- // --isolate-extensions is used: if it is on, the request is canceled and we |
- // stay on public.html (see https://crbug.com/656752), and if it's off, the |
- // request is blocked in ExtensionNavigationThrottle, which loads an error |
- // page into the iframe. This check handles both cases, but we should make |
- // the check stricter once --isolate-extensions is on by default. |
+ // TODO(alexmos): Make this check stricter, as extensions are now fully |
+ // isolated. The failure mode is that the request is canceled and we stay on |
+ // public.html (see https://crbug.com/656752). |
EXPECT_NE("Private", content); |
} |