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

Side by Side Diff: ui/android/resources/ui_resource_provider.cc

Issue 755643004: Replace SystemUIResourceManager with ResourceManager (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Apply Jared's comments Created 6 years 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 "content/browser/android/ui_resource_provider_impl.h" 5 #include "ui/android/resources/ui_resource_provider.h"
6 6
7 #include "cc/resources/ui_resource_client.h" 7 #include "cc/resources/ui_resource_client.h"
8 #include "cc/trees/layer_tree_host.h" 8 #include "cc/trees/layer_tree_host.h"
9 #include "content/public/browser/android/ui_resource_client_android.h" 9 #include "ui/android/resources/ui_resource_client_android.h"
10 10
11 namespace content { 11 namespace ui {
12 12
13 UIResourceProviderImpl::UIResourceProviderImpl() 13 UIResourceProvider::UIResourceProvider()
14 : system_ui_resource_manager_(this), host_(NULL), 14 : host_(NULL), supports_etc1_npot_(false) {
15 supports_etc1_npot_(false) {
16
17 } 15 }
18 16
19 UIResourceProviderImpl::~UIResourceProviderImpl() { 17 UIResourceProvider::~UIResourceProvider() {
20 SetLayerTreeHost(NULL); 18 SetLayerTreeHost(NULL);
21 } 19 }
22 20
23 void UIResourceProviderImpl::SetLayerTreeHost(cc::LayerTreeHost* host) { 21 void UIResourceProvider::SetLayerTreeHost(cc::LayerTreeHost* host) {
24 if (host_ == host) 22 if (host_ == host)
25 return; 23 return;
26 host_ = host; 24 host_ = host;
27 UIResourcesAreInvalid(); 25 UIResourcesAreInvalid();
28 } 26 }
29 27
30 void UIResourceProviderImpl::UIResourcesAreInvalid() { 28 void UIResourceProvider::UIResourcesAreInvalid() {
31 UIResourceClientMap client_map = ui_resource_client_map_; 29 UIResourceClientMap client_map = ui_resource_client_map_;
jdduke (slow) 2014/12/10 19:19:27 Nit: This should really be (sorry, I know this was
Jaekyun Seok (inactive) 2014/12/10 21:42:38 Done.
32 ui_resource_client_map_.clear(); 30 ui_resource_client_map_.clear();
33 for (UIResourceClientMap::iterator iter = client_map.begin(); 31 for (UIResourceClientMap::iterator iter = client_map.begin();
34 iter != client_map.end(); 32 iter != client_map.end(); iter++) {
35 iter++) {
36 iter->second->UIResourceIsInvalid(); 33 iter->second->UIResourceIsInvalid();
37 } 34 }
38 } 35 }
39 36
40 cc::UIResourceId UIResourceProviderImpl::CreateUIResource( 37 cc::UIResourceId UIResourceProvider::CreateUIResource(
41 UIResourceClientAndroid* client) { 38 ui::UIResourceClientAndroid* client) {
42 if (!host_) 39 if (!host_)
43 return 0; 40 return 0;
44 cc::UIResourceId id = host_->CreateUIResource(client); 41 cc::UIResourceId id = host_->CreateUIResource(client);
45 DCHECK(ui_resource_client_map_.find(id) == ui_resource_client_map_.end()); 42 DCHECK(ui_resource_client_map_.find(id) == ui_resource_client_map_.end());
46 43
47 ui_resource_client_map_[id] = client; 44 ui_resource_client_map_[id] = client;
48 return id; 45 return id;
49 } 46 }
50 47
51 void UIResourceProviderImpl::DeleteUIResource(cc::UIResourceId ui_resource_id) { 48 void UIResourceProvider::DeleteUIResource(cc::UIResourceId ui_resource_id) {
52 UIResourceClientMap::iterator iter = 49 UIResourceClientMap::iterator iter =
53 ui_resource_client_map_.find(ui_resource_id); 50 ui_resource_client_map_.find(ui_resource_id);
54 DCHECK(iter != ui_resource_client_map_.end()); 51 DCHECK(iter != ui_resource_client_map_.end());
55 52
56 ui_resource_client_map_.erase(iter); 53 ui_resource_client_map_.erase(iter);
57 54
58 if (!host_) 55 if (!host_)
59 return; 56 return;
60 host_->DeleteUIResource(ui_resource_id); 57 host_->DeleteUIResource(ui_resource_id);
61 } 58 }
62 59
63 ui::SystemUIResourceManager& 60 bool UIResourceProvider::SupportsETC1NonPowerOfTwo() const {
64 UIResourceProviderImpl::GetSystemUIResourceManager() {
65 return system_ui_resource_manager_;
66 }
67
68 bool UIResourceProviderImpl::SupportsETC1NonPowerOfTwo() const {
69 return supports_etc1_npot_; 61 return supports_etc1_npot_;
70 } 62 }
71 63
72 } // namespace content 64 } // namespace ui
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698