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

Side by Side Diff: trunk/src/chrome/browser/extensions/theme_installed_infobar_delegate.cc

Issue 102163002: Revert 238283 "Infobar system refactor." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 7 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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/extensions/theme_installed_infobar_delegate.h" 5 #include "chrome/browser/extensions/theme_installed_infobar_delegate.h"
6 6
7 #include <string> 7 #include <string>
8 8
9 #include "base/strings/utf_string_conversions.h" 9 #include "base/strings/utf_string_conversions.h"
10 #include "chrome/browser/chrome_notification_types.h" 10 #include "chrome/browser/chrome_notification_types.h"
11 #include "chrome/browser/extensions/extension_service.h" 11 #include "chrome/browser/extensions/extension_service.h"
12 #include "chrome/browser/infobars/infobar.h"
13 #include "chrome/browser/infobars/infobar_service.h" 12 #include "chrome/browser/infobars/infobar_service.h"
14 #include "chrome/browser/profiles/profile.h" 13 #include "chrome/browser/profiles/profile.h"
15 #include "chrome/browser/themes/theme_service.h" 14 #include "chrome/browser/themes/theme_service.h"
16 #include "chrome/browser/themes/theme_service_factory.h" 15 #include "chrome/browser/themes/theme_service_factory.h"
17 #include "chrome/browser/ui/browser_finder.h" 16 #include "chrome/browser/ui/browser_finder.h"
18 #include "chrome/browser/ui/tabs/tab_strip_model.h" 17 #include "chrome/browser/ui/tabs/tab_strip_model.h"
19 #include "content/public/browser/notification_source.h" 18 #include "content/public/browser/notification_source.h"
20 #include "extensions/common/extension.h" 19 #include "extensions/common/extension.h"
21 #include "grit/generated_resources.h" 20 #include "grit/generated_resources.h"
22 #include "grit/theme_resources.h" 21 #include "grit/theme_resources.h"
(...skipping 18 matching lines...) Expand all
41 chrome::FindTabbedBrowser(profile, true, chrome::GetActiveDesktop()); 40 chrome::FindTabbedBrowser(profile, true, chrome::GetActiveDesktop());
42 if (!browser) 41 if (!browser)
43 return; 42 return;
44 content::WebContents* web_contents = 43 content::WebContents* web_contents =
45 browser->tab_strip_model()->GetActiveWebContents(); 44 browser->tab_strip_model()->GetActiveWebContents();
46 if (!web_contents) 45 if (!web_contents)
47 return; 46 return;
48 InfoBarService* infobar_service = 47 InfoBarService* infobar_service =
49 InfoBarService::FromWebContents(web_contents); 48 InfoBarService::FromWebContents(web_contents);
50 ThemeService* theme_service = ThemeServiceFactory::GetForProfile(profile); 49 ThemeService* theme_service = ThemeServiceFactory::GetForProfile(profile);
51 scoped_ptr<InfoBar> new_infobar(ConfirmInfoBarDelegate::CreateInfoBar( 50 scoped_ptr<InfoBarDelegate> new_infobar(new ThemeInstalledInfoBarDelegate(
52 scoped_ptr<ConfirmInfoBarDelegate>(new ThemeInstalledInfoBarDelegate( 51 infobar_service, profile->GetExtensionService(), theme_service, new_theme,
53 profile->GetExtensionService(), theme_service, new_theme, 52 previous_theme_id, previous_using_native_theme));
54 previous_theme_id, previous_using_native_theme))));
55 53
56 // If there's a previous theme infobar, just replace that instead of adding a 54 // If there's a previous theme infobar, just replace that instead of adding a
57 // new one. 55 // new one.
58 for (size_t i = 0; i < infobar_service->infobar_count(); ++i) { 56 for (size_t i = 0; i < infobar_service->infobar_count(); ++i) {
59 InfoBar* old_infobar = infobar_service->infobar_at(i); 57 InfoBarDelegate* old_infobar = infobar_service->infobar_at(i);
60 ThemeInstalledInfoBarDelegate* theme_infobar = 58 ThemeInstalledInfoBarDelegate* theme_infobar =
61 old_infobar->delegate()->AsThemePreviewInfobarDelegate(); 59 old_infobar->AsThemePreviewInfobarDelegate();
62 if (theme_infobar) { 60 if (theme_infobar) {
63 // If the user installed the same theme twice, ignore the second install 61 // If the user installed the same theme twice, ignore the second install
64 // and keep the first install info bar, so that they can easily undo to 62 // and keep the first install info bar, so that they can easily undo to
65 // get back the previous theme. 63 // get back the previous theme.
66 if (theme_infobar->theme_id_ != new_theme->id()) { 64 if (theme_infobar->theme_id_ != new_theme->id()) {
67 infobar_service->ReplaceInfoBar(old_infobar, new_infobar.Pass()); 65 infobar_service->ReplaceInfoBar(old_infobar, new_infobar.Pass());
68 theme_service->OnInfobarDisplayed(); 66 theme_service->OnInfobarDisplayed();
69 } 67 }
70 return; 68 return;
71 } 69 }
72 } 70 }
73 71
74 // No previous theme infobar, so add this. 72 // No previous theme infobar, so add this.
75 infobar_service->AddInfoBar(new_infobar.Pass()); 73 infobar_service->AddInfoBar(new_infobar.Pass());
76 theme_service->OnInfobarDisplayed(); 74 theme_service->OnInfobarDisplayed();
77 } 75 }
78 76
79 ThemeInstalledInfoBarDelegate::ThemeInstalledInfoBarDelegate( 77 ThemeInstalledInfoBarDelegate::ThemeInstalledInfoBarDelegate(
78 InfoBarService* infobar_service,
80 ExtensionService* extension_service, 79 ExtensionService* extension_service,
81 ThemeService* theme_service, 80 ThemeService* theme_service,
82 const extensions::Extension* new_theme, 81 const extensions::Extension* new_theme,
83 const std::string& previous_theme_id, 82 const std::string& previous_theme_id,
84 bool previous_using_native_theme) 83 bool previous_using_native_theme)
85 : ConfirmInfoBarDelegate(), 84 : ConfirmInfoBarDelegate(infobar_service),
86 extension_service_(extension_service), 85 extension_service_(extension_service),
87 theme_service_(theme_service), 86 theme_service_(theme_service),
88 name_(new_theme->name()), 87 name_(new_theme->name()),
89 theme_id_(new_theme->id()), 88 theme_id_(new_theme->id()),
90 previous_theme_id_(previous_theme_id), 89 previous_theme_id_(previous_theme_id),
91 previous_using_native_theme_(previous_using_native_theme) { 90 previous_using_native_theme_(previous_using_native_theme) {
92 registrar_.Add(this, chrome::NOTIFICATION_BROWSER_THEME_CHANGED, 91 registrar_.Add(this, chrome::NOTIFICATION_BROWSER_THEME_CHANGED,
93 content::Source<ThemeService>(theme_service_)); 92 content::Source<ThemeService>(theme_service_));
94 } 93 }
95 94
(...skipping 52 matching lines...) Expand 10 before | Expand all | Expand 10 after
148 } 147 }
149 148
150 void ThemeInstalledInfoBarDelegate::Observe( 149 void ThemeInstalledInfoBarDelegate::Observe(
151 int type, 150 int type,
152 const content::NotificationSource& source, 151 const content::NotificationSource& source,
153 const content::NotificationDetails& details) { 152 const content::NotificationDetails& details) {
154 DCHECK_EQ(chrome::NOTIFICATION_BROWSER_THEME_CHANGED, type); 153 DCHECK_EQ(chrome::NOTIFICATION_BROWSER_THEME_CHANGED, type);
155 // If the new theme is different from what this info bar is associated with, 154 // If the new theme is different from what this info bar is associated with,
156 // close this info bar since it is no longer relevant. 155 // close this info bar since it is no longer relevant.
157 if (theme_id_ != theme_service_->GetThemeID()) 156 if (theme_id_ != theme_service_->GetThemeID())
158 infobar()->RemoveSelf(); 157 RemoveSelf();
159 } 158 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698