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

Side by Side Diff: chrome/browser/themes/custom_theme_supplier.cc

Issue 1869693002: Theme suppliers: Avoid all Image copying. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase. Created 4 years, 7 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 | « chrome/browser/themes/custom_theme_supplier.h ('k') | chrome/browser/themes/theme_service.h » ('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 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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/themes/custom_theme_supplier.h" 5 #include "chrome/browser/themes/custom_theme_supplier.h"
6 6
7 #include "base/memory/ref_counted_memory.h" 7 #include "base/memory/ref_counted_memory.h"
8 #include "ui/gfx/color_utils.h" 8 #include "ui/gfx/color_utils.h"
9 #include "ui/gfx/image/image.h"
10 9
11 CustomThemeSupplier::CustomThemeSupplier(ThemeType theme_type) 10 CustomThemeSupplier::CustomThemeSupplier(ThemeType theme_type)
12 : theme_type_(theme_type) {} 11 : theme_type_(theme_type) {}
13 12
14 CustomThemeSupplier::~CustomThemeSupplier() {} 13 CustomThemeSupplier::~CustomThemeSupplier() {}
15 14
16 void CustomThemeSupplier::StartUsingTheme() {} 15 void CustomThemeSupplier::StartUsingTheme() {}
17 16
18 void CustomThemeSupplier::StopUsingTheme() {} 17 void CustomThemeSupplier::StopUsingTheme() {}
19 18
20 bool CustomThemeSupplier::GetTint(int id, color_utils::HSL* hsl) const { 19 bool CustomThemeSupplier::GetTint(int id, color_utils::HSL* hsl) const {
21 return false; 20 return false;
22 } 21 }
23 22
24 bool CustomThemeSupplier::GetColor(int id, SkColor* color) const { 23 bool CustomThemeSupplier::GetColor(int id, SkColor* color) const {
25 return false; 24 return false;
26 } 25 }
27 26
28 bool CustomThemeSupplier::GetDisplayProperty(int id, int* result) const { 27 bool CustomThemeSupplier::GetDisplayProperty(int id, int* result) const {
29 return false; 28 return false;
30 } 29 }
31 30
32 gfx::Image CustomThemeSupplier::GetImageNamed(int id) { 31 const gfx::Image& CustomThemeSupplier::GetImageNamed(int id) {
33 return gfx::Image(); 32 return empty_image();
34 } 33 }
35 34
36 base::RefCountedMemory* CustomThemeSupplier::GetRawData( 35 base::RefCountedMemory* CustomThemeSupplier::GetRawData(
37 int idr_id, 36 int idr_id,
38 ui::ScaleFactor scale_factor) const { 37 ui::ScaleFactor scale_factor) const {
39 return NULL; 38 return NULL;
40 } 39 }
41 40
42 bool CustomThemeSupplier::HasCustomImage(int id) const { 41 bool CustomThemeSupplier::HasCustomImage(int id) const {
43 return false; 42 return false;
44 } 43 }
OLDNEW
« no previous file with comments | « chrome/browser/themes/custom_theme_supplier.h ('k') | chrome/browser/themes/theme_service.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698