Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2030)

Unified Diff: content/browser/browser_url_handler_impl_unittest.cc

Issue 11273049: Revert 164120 - content/browser: Move more files into the content namespace. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « content/browser/browser_url_handler_impl.cc ('k') | content/browser/cert_store_impl.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/browser_url_handler_impl_unittest.cc
===================================================================
--- content/browser/browser_url_handler_impl_unittest.cc (revision 164121)
+++ content/browser/browser_url_handler_impl_unittest.cc (working copy)
@@ -7,12 +7,11 @@
#include "googleurl/src/gurl.h"
#include "testing/gtest/include/gtest/gtest.h"
-namespace content {
+class BrowserURLHandlerImplTest : public testing::Test {
+};
-namespace {
-
// Test URL rewriter that rewrites all "foo://" URLs to "bar://bar".
-bool FooRewriter(GURL* url, BrowserContext* browser_context) {
+static bool FooRewriter(GURL* url, content::BrowserContext* browser_context) {
if (url->scheme() == "foo") {
*url = GURL("bar://bar");
return true;
@@ -21,7 +20,7 @@
}
// Test URL rewriter that rewrites all "bar://" URLs to "foo://foo".
-bool BarRewriter(GURL* url, BrowserContext* browser_context) {
+static bool BarRewriter(GURL* url, content::BrowserContext* browser_context) {
if (url->scheme() == "bar") {
*url = GURL("foo://foo");
return true;
@@ -29,10 +28,8 @@
return false;
}
-} // namespace
-
-TEST(BrowserURLHandlerImplTest, BasicRewriteAndReverse) {
- TestBrowserContext browser_context;
+TEST_F(BrowserURLHandlerImplTest, BasicRewriteAndReverse) {
+ content::TestBrowserContext browser_context;
BrowserURLHandlerImpl handler;
handler.AddHandlerPair(FooRewriter, BarRewriter);
@@ -60,8 +57,8 @@
ASSERT_EQ(saved_url, url);
}
-TEST(BrowserURLHandlerImplTest, NullHandlerReverse) {
- TestBrowserContext browser_context;
+TEST_F(BrowserURLHandlerImplTest, NullHandlerReverse) {
+ content::TestBrowserContext browser_context;
BrowserURLHandlerImpl handler;
GURL url("bar://foo");
@@ -79,5 +76,3 @@
ASSERT_TRUE(reversed);
ASSERT_EQ("foo://foo", url.spec());
}
-
-} // namespace content
« no previous file with comments | « content/browser/browser_url_handler_impl.cc ('k') | content/browser/cert_store_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698