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

Unified Diff: chrome/browser/translate/translate_manager.cc

Issue 8956059: Rename NavigationController to NavigationControllerImpl and put it into the content namespace. Al... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 9 years 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
Index: chrome/browser/translate/translate_manager.cc
===================================================================
--- chrome/browser/translate/translate_manager.cc (revision 115900)
+++ chrome/browser/translate/translate_manager.cc (working copy)
@@ -48,6 +48,7 @@
#include "net/url_request/url_request_status.h"
#include "ui/base/resource/resource_bundle.h"
+using content::NavigationEntry;
using content::WebContents;
namespace {
@@ -262,7 +263,7 @@
content::Source<NavigationController>(source).ptr();
content::LoadCommittedDetails* load_details =
content::Details<content::LoadCommittedDetails>(details).ptr();
- content::NavigationEntry* entry = controller->GetActiveEntry();
+ NavigationEntry* entry = controller->GetActiveEntry();
if (!entry) {
NOTREACHED();
return;
@@ -402,7 +403,7 @@
// The tab went away while we were retrieving the script.
continue;
}
- content::NavigationEntry* entry = tab->GetController().GetActiveEntry();
+ NavigationEntry* entry = tab->GetController().GetActiveEntry();
if (!entry || entry->GetPageID() != request.page_id) {
// We navigated away from the page the translation was triggered on.
continue;
@@ -468,7 +469,7 @@
if (CommandLine::ForCurrentProcess()->HasSwitch(switches::kDisableTranslate))
return;
- content::NavigationEntry* entry = tab->GetController().GetActiveEntry();
+ NavigationEntry* entry = tab->GetController().GetActiveEntry();
if (!entry) {
// This can happen for popups created with window.open("").
return;
@@ -553,8 +554,7 @@
void TranslateManager::TranslatePage(WebContents* web_contents,
const std::string& source_lang,
const std::string& target_lang) {
- content::NavigationEntry* entry =
- web_contents->GetController().GetActiveEntry();
+ NavigationEntry* entry = web_contents->GetController().GetActiveEntry();
if (!entry) {
NOTREACHED();
return;
@@ -586,8 +586,7 @@
}
void TranslateManager::RevertTranslation(WebContents* web_contents) {
- content::NavigationEntry* entry =
- web_contents->GetController().GetActiveEntry();
+ NavigationEntry* entry = web_contents->GetController().GetActiveEntry();
if (!entry) {
NOTREACHED();
return;
@@ -633,7 +632,7 @@
const std::string& translate_script,
const std::string& source_lang,
const std::string& target_lang) {
- content::NavigationEntry* entry = tab->GetController().GetActiveEntry();
+ NavigationEntry* entry = tab->GetController().GetActiveEntry();
if (!entry) {
NOTREACHED();
return;

Powered by Google App Engine
This is Rietveld 408576698