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

Side by Side Diff: chrome/browser/browser_theme_provider.h

Issue 1075014: Linux: Fix some GdkPixbuf leaks. (Closed)
Patch Set: move BrowserThemeProvider comment to the public functions Created 10 years, 9 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
« no previous file with comments | « no previous file | chrome/browser/gtk/extension_install_prompt_gtk.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2009 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2009 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 #ifndef CHROME_BROWSER_BROWSER_THEME_PROVIDER_H_ 5 #ifndef CHROME_BROWSER_BROWSER_THEME_PROVIDER_H_
6 #define CHROME_BROWSER_BROWSER_THEME_PROVIDER_H_ 6 #define CHROME_BROWSER_BROWSER_THEME_PROVIDER_H_
7 7
8 #include <map> 8 #include <map>
9 #include <set> 9 #include <set>
10 #include <string> 10 #include <string>
(...skipping 112 matching lines...) Expand 10 before | Expand all | Expand 10 after
123 123
124 // ThemeProvider implementation. 124 // ThemeProvider implementation.
125 virtual void Init(Profile* profile); 125 virtual void Init(Profile* profile);
126 virtual SkBitmap* GetBitmapNamed(int id) const; 126 virtual SkBitmap* GetBitmapNamed(int id) const;
127 virtual SkColor GetColor(int id) const; 127 virtual SkColor GetColor(int id) const;
128 virtual bool GetDisplayProperty(int id, int* result) const; 128 virtual bool GetDisplayProperty(int id, int* result) const;
129 virtual bool ShouldUseNativeFrame() const; 129 virtual bool ShouldUseNativeFrame() const;
130 virtual bool HasCustomImage(int id) const; 130 virtual bool HasCustomImage(int id) const;
131 virtual RefCountedMemory* GetRawData(int id) const; 131 virtual RefCountedMemory* GetRawData(int id) const;
132 #if defined(OS_LINUX) 132 #if defined(OS_LINUX)
133 // GdkPixbufs returned by GetPixbufNamed and GetRTLEnabledPixbufNamed are
134 // shared instances owned by the theme provider and should not be freed.
133 virtual GdkPixbuf* GetPixbufNamed(int id) const; 135 virtual GdkPixbuf* GetPixbufNamed(int id) const;
134 virtual GdkPixbuf* GetRTLEnabledPixbufNamed(int id) const; 136 virtual GdkPixbuf* GetRTLEnabledPixbufNamed(int id) const;
135 #elif defined(OS_MACOSX) 137 #elif defined(OS_MACOSX)
136 virtual NSImage* GetNSImageNamed(int id, bool allow_default) const; 138 virtual NSImage* GetNSImageNamed(int id, bool allow_default) const;
137 virtual NSColor* GetNSImageColorNamed(int id, bool allow_default) const; 139 virtual NSColor* GetNSImageColorNamed(int id, bool allow_default) const;
138 virtual NSColor* GetNSColor(int id, bool allow_default) const; 140 virtual NSColor* GetNSColor(int id, bool allow_default) const;
139 virtual NSColor* GetNSColorTint(int id, bool allow_default) const; 141 virtual NSColor* GetNSColorTint(int id, bool allow_default) const;
140 virtual NSGradient* GetNSGradient(int id) const; 142 virtual NSGradient* GetNSGradient(int id) const;
141 #endif 143 #endif
142 144
(...skipping 115 matching lines...) Expand 10 before | Expand all | Expand 10 after
258 260
259 scoped_refptr<BrowserThemePack> theme_pack_; 261 scoped_refptr<BrowserThemePack> theme_pack_;
260 262
261 // The number of infobars currently displayed. 263 // The number of infobars currently displayed.
262 int number_of_infobars_; 264 int number_of_infobars_;
263 265
264 DISALLOW_COPY_AND_ASSIGN(BrowserThemeProvider); 266 DISALLOW_COPY_AND_ASSIGN(BrowserThemeProvider);
265 }; 267 };
266 268
267 #endif // CHROME_BROWSER_BROWSER_THEME_PROVIDER_H_ 269 #endif // CHROME_BROWSER_BROWSER_THEME_PROVIDER_H_
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/gtk/extension_install_prompt_gtk.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698