Back out r10487 too

chrome/browser/renderer_host/renderer_security_policy_unittest.cc

 // Copyright (c) 2006-2008 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 <string>
 
 #include "base/basictypes.h"
 #include "chrome/browser/renderer_host/renderer_security_policy.h"
 #include "chrome/common/url_constants.h"
 #include "net/url_request/url_request.h"
@@ skipping 70 matching lines @@
 TEST_F(RendererSecurityPolicyTest, AboutTest) {
   RendererSecurityPolicy* p = RendererSecurityPolicy::GetInstance();
 
   p->Add(kRendererID);
 
   EXPECT_TRUE(p->CanRequestURL(kRendererID, GURL("about:blank")));
   EXPECT_TRUE(p->CanRequestURL(kRendererID, GURL("about:BlAnK")));
   EXPECT_TRUE(p->CanRequestURL(kRendererID, GURL("aBouT:BlAnK")));
   EXPECT_TRUE(p->CanRequestURL(kRendererID, GURL("aBouT:blank")));
 
-  EXPECT_FALSE(p->CanRequestURL(kRendererID, GURL(chrome::kAboutMemoryURL)));
-  EXPECT_FALSE(p->CanRequestURL(kRendererID, GURL(chrome::kAboutCrashURL)));
-  EXPECT_FALSE(p->CanRequestURL(kRendererID, GURL(chrome::kAboutCacheURL)));
-  EXPECT_FALSE(p->CanRequestURL(kRendererID, GURL(chrome::kAboutHangURL)));
+  EXPECT_FALSE(p->CanRequestURL(kRendererID, GURL("about:memory")));
+  EXPECT_FALSE(p->CanRequestURL(kRendererID, GURL("about:crash")));
+  EXPECT_FALSE(p->CanRequestURL(kRendererID, GURL("about:cache")));
+  EXPECT_FALSE(p->CanRequestURL(kRendererID, GURL("about:hang")));
 
   EXPECT_FALSE(p->CanRequestURL(kRendererID, GURL("aBoUt:memory")));
   EXPECT_FALSE(p->CanRequestURL(kRendererID, GURL("about:CrASh")));
   EXPECT_FALSE(p->CanRequestURL(kRendererID, GURL("abOuT:cAChe")));
 
-  p->GrantRequestURL(kRendererID, GURL(chrome::kAboutMemoryURL));
-  EXPECT_FALSE(p->CanRequestURL(kRendererID, GURL(chrome::kAboutMemoryURL)));
+  p->GrantRequestURL(kRendererID, GURL("about:memory"));
+  EXPECT_FALSE(p->CanRequestURL(kRendererID, GURL("about:memory")));
 
-  p->GrantRequestURL(kRendererID, GURL(chrome::kAboutCrashURL));
-  EXPECT_FALSE(p->CanRequestURL(kRendererID, GURL(chrome::kAboutCrashURL)));
+  p->GrantRequestURL(kRendererID, GURL("about:crash"));
+  EXPECT_FALSE(p->CanRequestURL(kRendererID, GURL("about:crash")));
 
-  p->GrantRequestURL(kRendererID, GURL(chrome::kAboutCacheURL));
-  EXPECT_FALSE(p->CanRequestURL(kRendererID, GURL(chrome::kAboutCacheURL)));
+  p->GrantRequestURL(kRendererID, GURL("about:cache"));
+  EXPECT_FALSE(p->CanRequestURL(kRendererID, GURL("about:cache")));
 
-  p->GrantRequestURL(kRendererID, GURL(chrome::kAboutHangURL));
-  EXPECT_FALSE(p->CanRequestURL(kRendererID, GURL(chrome::kAboutHangURL)));
+  p->GrantRequestURL(kRendererID, GURL("about:hang"));
+  EXPECT_FALSE(p->CanRequestURL(kRendererID, GURL("about:hang")));
 
   p->Remove(kRendererID);
 }
 
 TEST_F(RendererSecurityPolicyTest, JavaScriptTest) {
   RendererSecurityPolicy* p = RendererSecurityPolicy::GetInstance();
 
   p->Add(kRendererID);
 
   EXPECT_FALSE(p->CanRequestURL(kRendererID, GURL("javascript:alert('xss')")));
@@ skipping 133 matching lines @@
   // Renderers are added and removed on the UI thread, but the policy can be
   // queried on the IO thread.  The RendererSecurityPolicy needs to be prepared
   // to answer policy questions about renderers who no longer exist.
 
   // In this case, we default to secure behavior.
   EXPECT_FALSE(p->CanRequestURL(kRendererID, url));
   EXPECT_FALSE(p->CanUploadFile(kRendererID, file));
   EXPECT_FALSE(p->HasDOMUIBindings(kRendererID));
 }