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

Side by Side Diff: chrome/browser/translate/translate_tab_helper.cc

Issue 9030010: Move most of the remaining users of WebContentsObserver::tab_contents() to use web_contents(). (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 8 years, 12 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/translate/translate_tab_helper.h" 5 #include "chrome/browser/translate/translate_tab_helper.h"
6 6
7 #include "chrome/browser/translate/page_translated_details.h" 7 #include "chrome/browser/translate/page_translated_details.h"
8 #include "chrome/common/chrome_notification_types.h" 8 #include "chrome/common/chrome_notification_types.h"
9 #include "chrome/common/render_messages.h" 9 #include "chrome/common/render_messages.h"
10 #include "content/browser/tab_contents/tab_contents.h" 10 #include "content/browser/tab_contents/tab_contents.h"
11 #include "content/public/browser/notification_service.h" 11 #include "content/public/browser/notification_service.h"
12 12
13 using content::WebContents;
14
13 TranslateTabHelper::TranslateTabHelper(TabContents* tab_contents) 15 TranslateTabHelper::TranslateTabHelper(TabContents* tab_contents)
14 : content::WebContentsObserver(tab_contents), 16 : content::WebContentsObserver(tab_contents),
15 language_state_(&tab_contents->GetController()) { 17 language_state_(&tab_contents->GetController()) {
16 } 18 }
17 19
18 TranslateTabHelper::~TranslateTabHelper() { 20 TranslateTabHelper::~TranslateTabHelper() {
19 } 21 }
20 22
21 bool TranslateTabHelper::OnMessageReceived(const IPC::Message& message) { 23 bool TranslateTabHelper::OnMessageReceived(const IPC::Message& message) {
22 bool handled = true; 24 bool handled = true;
(...skipping 14 matching lines...) Expand all
37 language_state_.DidNavigate(details); 39 language_state_.DidNavigate(details);
38 } 40 }
39 41
40 void TranslateTabHelper::OnLanguageDetermined(const std::string& language, 42 void TranslateTabHelper::OnLanguageDetermined(const std::string& language,
41 bool page_translatable) { 43 bool page_translatable) {
42 language_state_.LanguageDetermined(language, page_translatable); 44 language_state_.LanguageDetermined(language, page_translatable);
43 45
44 std::string lang = language; 46 std::string lang = language;
45 content::NotificationService::current()->Notify( 47 content::NotificationService::current()->Notify(
46 chrome::NOTIFICATION_TAB_LANGUAGE_DETERMINED, 48 chrome::NOTIFICATION_TAB_LANGUAGE_DETERMINED,
47 content::Source<TabContents>(tab_contents()), 49 content::Source<WebContents>(web_contents()),
48 content::Details<std::string>(&lang)); 50 content::Details<std::string>(&lang));
49 } 51 }
50 52
51 void TranslateTabHelper::OnPageTranslated(int32 page_id, 53 void TranslateTabHelper::OnPageTranslated(int32 page_id,
52 const std::string& original_lang, 54 const std::string& original_lang,
53 const std::string& translated_lang, 55 const std::string& translated_lang,
54 TranslateErrors::Type error_type) { 56 TranslateErrors::Type error_type) {
55 language_state_.set_current_language(translated_lang); 57 language_state_.set_current_language(translated_lang);
56 language_state_.set_translation_pending(false); 58 language_state_.set_translation_pending(false);
57 PageTranslatedDetails details(original_lang, translated_lang, error_type); 59 PageTranslatedDetails details(original_lang, translated_lang, error_type);
58 content::NotificationService::current()->Notify( 60 content::NotificationService::current()->Notify(
59 chrome::NOTIFICATION_PAGE_TRANSLATED, 61 chrome::NOTIFICATION_PAGE_TRANSLATED,
60 content::Source<TabContents>(tab_contents()), 62 content::Source<WebContents>(web_contents()),
61 content::Details<PageTranslatedDetails>(&details)); 63 content::Details<PageTranslatedDetails>(&details));
62 } 64 }
OLDNEW
« no previous file with comments | « chrome/browser/translate/translate_manager.cc ('k') | chrome/browser/ui/alternate_error_tab_observer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698