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

Side by Side Diff: chrome/browser/themes/browser_theme_pack.h

Issue 1771033003: gfx::Image: Added thread checker. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove UpdateShortcutWorker changes; instead just disable thread checking. Created 3 years, 5 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
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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_THEMES_BROWSER_THEME_PACK_H_ 5 #ifndef CHROME_BROWSER_THEMES_BROWSER_THEME_PACK_H_
6 #define CHROME_BROWSER_THEMES_BROWSER_THEME_PACK_H_ 6 #define CHROME_BROWSER_THEMES_BROWSER_THEME_PACK_H_
7 7
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <map> 10 #include <map>
(...skipping 243 matching lines...) Expand 10 before | Expand all | Expand 10 after
254 // non-NULL; |image_memory_| is only filled during BuildFromExtension(). Any 254 // non-NULL; |image_memory_| is only filled during BuildFromExtension(). Any
255 // image data that needs to be written to the DataPack during WriteToDisk() 255 // image data that needs to be written to the DataPack during WriteToDisk()
256 // needs to be in |image_memory_|. 256 // needs to be in |image_memory_|.
257 RawImages image_memory_; 257 RawImages image_memory_;
258 258
259 // Loaded images. These are loaded from |image_memory_|, from |data_pack_|, 259 // Loaded images. These are loaded from |image_memory_|, from |data_pack_|,
260 // and by BuildFromExtension(). 260 // and by BuildFromExtension().
261 ImageCache images_; 261 ImageCache images_;
262 262
263 // Cache of images created in BuildFromExtension(). Once the theme pack is 263 // Cache of images created in BuildFromExtension(). Once the theme pack is
264 // created, this cache should only be accessed on the file thread. There 264 // created, this cache should only be accessed on the FILE thread (or whatever
265 // should be no IDs in |image_memory_| that are in |images_on_file_thread_| 265 // thread WriteToDisk is called from*). There should be no IDs in
266 // or vice versa. 266 // |image_memory_| that are in |images_on_file_thread_| or vice versa.
267 //
268 // *NOTE: These are usually accessed on the FILE thread, but as the
269 // documentation for WriteToDisk notes, it is not always the FILE thread; just
270 // some thread other than UI.
267 ImageCache images_on_file_thread_; 271 ImageCache images_on_file_thread_;
268 272
269 // Whether the theme pack has been succesfully initialized and is ready to 273 // Whether the theme pack has been succesfully initialized and is ready to
270 // use. 274 // use.
271 bool is_valid_ = false; 275 bool is_valid_ = false;
272 276
273 DISALLOW_COPY_AND_ASSIGN(BrowserThemePack); 277 DISALLOW_COPY_AND_ASSIGN(BrowserThemePack);
274 }; 278 };
275 279
276 #endif // CHROME_BROWSER_THEMES_BROWSER_THEME_PACK_H_ 280 #endif // CHROME_BROWSER_THEMES_BROWSER_THEME_PACK_H_
OLDNEW
« no previous file with comments | « chrome/browser/profiles/profile_info_cache.cc ('k') | chrome/browser/themes/browser_theme_pack.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698