summaryrefslogtreecommitdiffstats
path: root/chrome/browser/ui/gtk/browser_toolbar_gtk.cc
diff options
context:
space:
mode:
authorerg@google.com <erg@google.com@0039d316-1c4b-4281-b951-d872f2087c98>2011-03-23 17:36:42 +0000
committererg@google.com <erg@google.com@0039d316-1c4b-4281-b951-d872f2087c98>2011-03-23 17:36:42 +0000
commita0ea76cbf036408912e337653500aaefd51586b9 (patch)
treeed1bf3f0a50025df8c2d4b7e083593ded4d2e01a /chrome/browser/ui/gtk/browser_toolbar_gtk.cc
parent58492209d54a3af3e40fa0a876c9d6381d943188 (diff)
downloadchromium_src-a0ea76cbf036408912e337653500aaefd51586b9.zip
chromium_src-a0ea76cbf036408912e337653500aaefd51586b9.tar.gz
chromium_src-a0ea76cbf036408912e337653500aaefd51586b9.tar.bz2
Theme system cleanup
- Removes useless dependencies on themeing from content/ - Renames BrowserThemeProvider to ThemeService, along with name variants such as GtkThemeService. BrowserThemeProvider has grown by a lot to be more than an implementation of the ui::ThemeProvider interface. This change touches everything. - Rename theme_service.{h,cc} (which defined ThemeServiceFactory) to theme_service_factory.{h,cc}. BUG=77155 TEST=compiles. Review URL: http://codereview.chromium.org/6727005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@79145 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/ui/gtk/browser_toolbar_gtk.cc')
-rw-r--r--chrome/browser/ui/gtk/browser_toolbar_gtk.cc40
1 files changed, 20 insertions, 20 deletions
diff --git a/chrome/browser/ui/gtk/browser_toolbar_gtk.cc b/chrome/browser/ui/gtk/browser_toolbar_gtk.cc
index f5c1735..4c057b0 100644
--- a/chrome/browser/ui/gtk/browser_toolbar_gtk.cc
+++ b/chrome/browser/ui/gtk/browser_toolbar_gtk.cc
@@ -19,7 +19,7 @@
#include "chrome/browser/net/url_fixer_upper.h"
#include "chrome/browser/prefs/pref_service.h"
#include "chrome/browser/profiles/profile.h"
-#include "chrome/browser/themes/browser_theme_provider.h"
+#include "chrome/browser/themes/theme_service.h"
#include "chrome/browser/ui/browser.h"
#include "chrome/browser/ui/gtk/accelerators_gtk.h"
#include "chrome/browser/ui/gtk/back_forward_button_gtk.h"
@@ -28,7 +28,7 @@
#include "chrome/browser/ui/gtk/cairo_cached_surface.h"
#include "chrome/browser/ui/gtk/custom_button.h"
#include "chrome/browser/ui/gtk/gtk_chrome_button.h"
-#include "chrome/browser/ui/gtk/gtk_theme_provider.h"
+#include "chrome/browser/ui/gtk/gtk_theme_service.h"
#include "chrome/browser/ui/gtk/gtk_util.h"
#include "chrome/browser/ui/gtk/location_bar_view_gtk.h"
#include "chrome/browser/ui/gtk/reload_button_gtk.h"
@@ -119,7 +119,7 @@ void BrowserToolbarGtk::Init(Profile* profile,
// Make sure to tell the location bar the profile before calling its Init.
SetProfile(profile);
- theme_provider_ = GtkThemeProvider::GetFrom(profile);
+ theme_service_ = GtkThemeService::GetFrom(profile);
offscreen_entry_.Own(gtk_entry_new());
show_home_button_.Init(prefs::kShowHomeButton, profile->GetPrefs(), this);
@@ -130,7 +130,7 @@ void BrowserToolbarGtk::Init(Profile* profile,
event_box_ = gtk_event_box_new();
// Make the event box transparent so themes can use transparent toolbar
// backgrounds.
- if (!theme_provider_->UseGtkTheme())
+ if (!theme_service_->UseGtkTheme())
gtk_event_box_set_visible_window(GTK_EVENT_BOX(event_box_), FALSE);
toolbar_ = gtk_hbox_new(FALSE, 0);
@@ -159,7 +159,7 @@ void BrowserToolbarGtk::Init(Profile* profile,
gtk_box_pack_start(GTK_BOX(toolbar_left_), reload_->widget(), FALSE, FALSE,
0);
- home_.reset(new CustomDrawButton(GtkThemeProvider::GetFrom(profile_),
+ home_.reset(new CustomDrawButton(GtkThemeService::GetFrom(profile_),
IDR_HOME, IDR_HOME_P, IDR_HOME_H, 0, GTK_STOCK_HOME,
GTK_ICON_SIZE_SMALL_TOOLBAR));
gtk_widget_set_tooltip_text(home_->widget(),
@@ -190,9 +190,9 @@ void BrowserToolbarGtk::Init(Profile* profile,
}
wrench_menu_image_ = gtk_image_new_from_pixbuf(
- theme_provider_->GetRTLEnabledPixbufNamed(IDR_TOOLS));
+ theme_service_->GetRTLEnabledPixbufNamed(IDR_TOOLS));
wrench_menu_button_.reset(new CustomDrawButton(
- GtkThemeProvider::GetFrom(profile_),
+ GtkThemeService::GetFrom(profile_),
IDR_TOOLS, IDR_TOOLS_P, IDR_TOOLS_H, 0,
wrench_menu_image_));
GtkWidget* wrench_button = wrench_menu_button_->widget();
@@ -236,7 +236,7 @@ void BrowserToolbarGtk::Init(Profile* profile,
location_bar_->UpdateContentSettingsIcons();
SetViewIDs();
- theme_provider_->InitThemesFor(this);
+ theme_service_->InitThemesFor(this);
}
void BrowserToolbarGtk::SetViewIDs() {
@@ -315,7 +315,7 @@ void BrowserToolbarGtk::StoppedShowing() {
}
GtkIconSet* BrowserToolbarGtk::GetIconSetForId(int idr) {
- return theme_provider_->GetIconSetForId(idr);
+ return theme_service_->GetIconSetForId(idr);
}
// Always show images because we desire that some icons always show
@@ -345,7 +345,7 @@ void BrowserToolbarGtk::Observe(NotificationType type,
NotifyPrefChanged(Details<std::string>(details).ptr());
} else if (type == NotificationType::BROWSER_THEME_CHANGED) {
// Update the spacing around the menu buttons
- bool use_gtk = theme_provider_->UseGtkTheme();
+ bool use_gtk = theme_service_->UseGtkTheme();
int border = use_gtk ? 0 : 2;
gtk_container_set_border_width(
GTK_CONTAINER(wrench_menu_button_->widget()), border);
@@ -372,7 +372,7 @@ void BrowserToolbarGtk::Observe(NotificationType type,
// we set the initial value in Init()).
gtk_image_set_from_pixbuf(
GTK_IMAGE(wrench_menu_image_),
- theme_provider_->GetRTLEnabledPixbufNamed(IDR_TOOLS));
+ theme_service_->GetRTLEnabledPixbufNamed(IDR_TOOLS));
}
UpdateRoundedness();
@@ -428,7 +428,7 @@ bool BrowserToolbarGtk::UpdateRoundedness() {
// We still round the corners if we are in chrome theme mode, but we do it by
// drawing theme resources rather than changing the physical shape of the
// widget.
- bool should_be_rounded = theme_provider_->UseGtkTheme() &&
+ bool should_be_rounded = theme_service_->UseGtkTheme() &&
window_->ShouldDrawContentDropShadow();
if (should_be_rounded == gtk_util::IsActingAsRoundedWindow(alignment_))
@@ -453,7 +453,7 @@ gboolean BrowserToolbarGtk::OnAlignmentExpose(GtkWidget* widget,
return TRUE;
// We don't need to render the toolbar image in GTK mode.
- if (theme_provider_->UseGtkTheme())
+ if (theme_service_->UseGtkTheme())
return FALSE;
cairo_t* cr = gdk_cairo_create(GDK_DRAWABLE(widget->window));
@@ -463,8 +463,8 @@ gboolean BrowserToolbarGtk::OnAlignmentExpose(GtkWidget* widget,
gfx::Point tabstrip_origin =
window_->tabstrip()->GetTabStripOriginForWidget(widget);
// Fill the entire region with the toolbar color.
- GdkColor color = theme_provider_->GetGdkColor(
- BrowserThemeProvider::COLOR_TOOLBAR);
+ GdkColor color = theme_service_->GetGdkColor(
+ ThemeService::COLOR_TOOLBAR);
gdk_cairo_set_source_color(cr, &color);
cairo_fill(cr);
@@ -490,7 +490,7 @@ gboolean BrowserToolbarGtk::OnAlignmentExpose(GtkWidget* widget,
area = area.Subtract(right).Subtract(left);
}
- CairoCachedSurface* background = theme_provider_->GetSurfaceNamed(
+ CairoCachedSurface* background = theme_service_->GetSurfaceNamed(
IDR_THEME_TOOLBAR, widget);
background->SetSource(cr, tabstrip_origin.x(), tabstrip_origin.y());
cairo_pattern_set_extend(cairo_get_source(cr), CAIRO_EXTEND_REPEAT);
@@ -517,13 +517,13 @@ gboolean BrowserToolbarGtk::OnAlignmentExpose(GtkWidget* widget,
cairo_set_operator(copy_cr, CAIRO_OPERATOR_SOURCE);
if (draw_left_corner) {
- CairoCachedSurface* left_corner = theme_provider_->GetSurfaceNamed(
+ CairoCachedSurface* left_corner = theme_service_->GetSurfaceNamed(
IDR_CONTENT_TOP_LEFT_CORNER_MASK, widget);
left_corner->SetSource(copy_cr, left.x(), left.y());
cairo_paint(copy_cr);
}
if (draw_right_corner) {
- CairoCachedSurface* right_corner = theme_provider_->GetSurfaceNamed(
+ CairoCachedSurface* right_corner = theme_service_->GetSurfaceNamed(
IDR_CONTENT_TOP_RIGHT_CORNER_MASK, widget);
right_corner->SetSource(copy_cr, right.x(), right.y());
// We fill a path rather than just painting because we don't want to
@@ -554,7 +554,7 @@ gboolean BrowserToolbarGtk::OnAlignmentExpose(GtkWidget* widget,
gboolean BrowserToolbarGtk::OnLocationHboxExpose(GtkWidget* location_hbox,
GdkEventExpose* e) {
- if (theme_provider_->UseGtkTheme()) {
+ if (theme_service_->UseGtkTheme()) {
gtk_util::DrawTextEntryBackground(offscreen_entry_.get(),
location_hbox, &e->area,
&location_hbox->allocation);
@@ -629,7 +629,7 @@ gboolean BrowserToolbarGtk::OnWrenchMenuButtonExpose(GtkWidget* sender,
GdkEventExpose* expose) {
const SkBitmap* badge = NULL;
if (UpgradeDetector::GetInstance()->notify_upgrade()) {
- badge = theme_provider_->GetBitmapNamed(IDR_UPDATE_BADGE);
+ badge = theme_service_->GetBitmapNamed(IDR_UPDATE_BADGE);
} else {
return FALSE;
}