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

Unified Diff: chrome/browser/extensions/theme_installed_infobar_delegate.cc

Issue 499004: Try 2: Completely redo how themes are stored on disk and processed at install time. (Closed)
Patch Set: Created 11 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/extensions/theme_installed_infobar_delegate.cc
diff --git a/chrome/browser/extensions/theme_installed_infobar_delegate.cc b/chrome/browser/extensions/theme_installed_infobar_delegate.cc
index 879cbb224547ffe3cbfaea42c6082ee98caad49e..caea07d1835ac0ec97ae85f579896cc97076f5a5 100644
--- a/chrome/browser/extensions/theme_installed_infobar_delegate.cc
+++ b/chrome/browser/extensions/theme_installed_infobar_delegate.cc
@@ -27,7 +27,9 @@ ThemeInstalledInfoBarDelegate::ThemeInstalledInfoBarDelegate(
void ThemeInstalledInfoBarDelegate::InfoBarClosed() {
ExtensionsService* service = profile_->GetExtensionsService();
- if (service) {
+ // Only delete the theme if we've installed a new theme and not the same
+ // theme on top of the current one.
+ if (service && previous_theme_id_ != new_theme_id_) {
std::string uninstall_id;
if (was_canceled_)
uninstall_id = new_theme_id_;

Powered by Google App Engine
This is Rietveld 408576698