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

Side by Side Diff: chrome/browser/ui/gtk/theme_install_bubble_view_gtk.cc

Issue 7598010: Profiles: Mark the last of BROWSER_THEME_CHANGED AllSources() usages as OK. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 4 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/ui/gtk/theme_install_bubble_view_gtk.h" 5 #include "chrome/browser/ui/gtk/theme_install_bubble_view_gtk.h"
6 6
7 #include <math.h> 7 #include <math.h>
8 8
9 #include "chrome/browser/ui/gtk/gtk_util.h" 9 #include "chrome/browser/ui/gtk/gtk_util.h"
10 #include "chrome/browser/ui/gtk/rounded_window.h" 10 #include "chrome/browser/ui/gtk/rounded_window.h"
(...skipping 39 matching lines...) Expand 10 before | Expand all | Expand 10 after
50 InitWidgets(); 50 InitWidgets();
51 51
52 // Close when theme has been installed. 52 // Close when theme has been installed.
53 // 53 //
54 // TODO(erg): At least for version 1 of multiprofiles, we're still going to 54 // TODO(erg): At least for version 1 of multiprofiles, we're still going to
55 // listen to AllSources(). Installing a theme blocks the entire UI thread so 55 // listen to AllSources(). Installing a theme blocks the entire UI thread so
56 // we won't have another profile trying to install a theme. 56 // we won't have another profile trying to install a theme.
57 registrar_.Add( 57 registrar_.Add(
58 this, 58 this,
59 chrome::NOTIFICATION_BROWSER_THEME_CHANGED, 59 chrome::NOTIFICATION_BROWSER_THEME_CHANGED,
60 NotificationService::AllSources()); 60 NotificationService::AllBrowserContextsAndSources());
61 61
62 // Close when we are installing an extension, not a theme. 62 // Close when we are installing an extension, not a theme.
63 registrar_.Add( 63 registrar_.Add(
64 this, 64 this,
65 chrome::NOTIFICATION_NO_THEME_DETECTED, 65 chrome::NOTIFICATION_NO_THEME_DETECTED,
66 NotificationService::AllSources()); 66 NotificationService::AllSources());
67 registrar_.Add( 67 registrar_.Add(
68 this, 68 this,
69 chrome::NOTIFICATION_EXTENSION_INSTALLED, 69 chrome::NOTIFICATION_EXTENSION_INSTALLED,
70 NotificationService::AllSources()); 70 NotificationService::AllSources());
(...skipping 109 matching lines...) Expand 10 before | Expand all | Expand 10 after
180 0, M_PI_2); 180 0, M_PI_2);
181 cairo_arc(cr, inner_rect.x(), inner_rect.bottom(), inset, 181 cairo_arc(cr, inner_rect.x(), inner_rect.bottom(), inset,
182 M_PI_2, M_PI); 182 M_PI_2, M_PI);
183 183
184 cairo_set_source_rgba(cr, 0.0, 0.0, 0.0, kBubbleOpacity); 184 cairo_set_source_rgba(cr, 0.0, 0.0, 0.0, kBubbleOpacity);
185 cairo_fill(cr); 185 cairo_fill(cr);
186 cairo_destroy(cr); 186 cairo_destroy(cr);
187 187
188 return FALSE; 188 return FALSE;
189 } 189 }
OLDNEW
« no previous file with comments | « chrome/browser/ui/cocoa/theme_install_bubble_view.mm ('k') | chrome/browser/ui/views/theme_install_bubble_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698