diff options
author | jam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-12-28 21:04:23 +0000 |
---|---|---|
committer | jam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-12-28 21:04:23 +0000 |
commit | 10f417c53722348c4452dc22e1a0dd18c0e4b140 (patch) | |
tree | 470c5574d9d2ef9c7ba62951868149e70169b1a5 /chrome/browser/rlz | |
parent | a47fccd356705a7d92cc83b03150c9782167f8c4 (diff) | |
download | chromium_src-10f417c53722348c4452dc22e1a0dd18c0e4b140.zip chromium_src-10f417c53722348c4452dc22e1a0dd18c0e4b140.tar.gz chromium_src-10f417c53722348c4452dc22e1a0dd18c0e4b140.tar.bz2 |
Rename NavigationController to NavigationControllerImpl and put it into the content namespace. Also make all users of content::NavigationController interface use the "using" keyword so they don't have to put content:: everywhere (similar to other Content API and WebKit API classes).
BUG=98716
TBR=joi
Review URL: http://codereview.chromium.org/8956059
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115916 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/rlz')
-rw-r--r-- | chrome/browser/rlz/rlz.cc | 5 | ||||
-rw-r--r-- | chrome/browser/rlz/rlz_unittest.cc | 13 |
2 files changed, 9 insertions, 9 deletions
diff --git a/chrome/browser/rlz/rlz.cc b/chrome/browser/rlz/rlz.cc index a191790..49f8087 100644 --- a/chrome/browser/rlz/rlz.cc +++ b/chrome/browser/rlz/rlz.cc @@ -37,6 +37,7 @@ #include "content/public/browser/notification_service.h" using content::BrowserThread; +using content::NavigationEntry; namespace { @@ -305,8 +306,8 @@ void RLZTracker::Observe(int type, content::NotificationService::AllSources()); break; case content::NOTIFICATION_NAV_ENTRY_PENDING: { - const content::NavigationEntry* entry = - content::Details<content::NavigationEntry>(details).ptr(); + const NavigationEntry* entry = + content::Details<NavigationEntry>(details).ptr(); if (entry != NULL && ((entry->GetTransitionType() & content::PAGE_TRANSITION_HOME_PAGE) != 0)) { diff --git a/chrome/browser/rlz/rlz_unittest.cc b/chrome/browser/rlz/rlz_unittest.cc index 1e28437..990c207 100644 --- a/chrome/browser/rlz/rlz_unittest.cc +++ b/chrome/browser/rlz/rlz_unittest.cc @@ -24,6 +24,7 @@ #include "testing/gtest/include/gtest/gtest.h" using base::win::RegKey; +using content::NavigationEntry; using registry_util::RegistryOverrideManager; using testing::AssertionResult; using testing::AssertionSuccess; @@ -237,13 +238,12 @@ void RlzLibTest::SimulateOmniboxUsage() { } void RlzLibTest::SimulateHomepageUsage() { - scoped_ptr<content::NavigationEntry> entry( - content::NavigationEntry::Create()); + scoped_ptr<NavigationEntry> entry(NavigationEntry::Create()); entry->SetPageID(0); entry->SetTransitionType(content::PAGE_TRANSITION_HOME_PAGE); tracker_.Observe(content::NOTIFICATION_NAV_ENTRY_PENDING, content::NotificationService::AllSources(), - content::Details<content::NavigationEntry>(entry.get())); + content::Details<NavigationEntry>(entry.get())); } void RlzLibTest::InvokeDelayedInit() { @@ -582,16 +582,15 @@ TEST_F(RlzLibTest, PingUpdatesRlzCache) { } TEST_F(RlzLibTest, ObserveHandlesBadArgs) { - scoped_ptr<content::NavigationEntry> entry( - content::NavigationEntry::Create()); + scoped_ptr<NavigationEntry> entry(NavigationEntry::Create()); entry->SetPageID(0); entry->SetTransitionType(content::PAGE_TRANSITION_LINK); tracker_.Observe(content::NOTIFICATION_NAV_ENTRY_PENDING, content::NotificationService::AllSources(), - content::Details<content::NavigationEntry>(NULL)); + content::Details<NavigationEntry>(NULL)); tracker_.Observe(content::NOTIFICATION_NAV_ENTRY_PENDING, content::NotificationService::AllSources(), - content::Details<content::NavigationEntry>(entry.get())); + content::Details<NavigationEntry>(entry.get())); } TEST_F(RlzLibTest, ReactivationNonOrganicNonOrganic) { |