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

Side by Side Diff: chrome/browser/android/compositor/tab_content_manager.cc

Issue 2334613003: Re-write many calls to WrapUnique() with MakeUnique() (Closed)
Patch Set: Changes from review by sky Created 4 years, 3 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 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 "chrome/browser/android/compositor/tab_content_manager.h" 5 #include "chrome/browser/android/compositor/tab_content_manager.h"
6 6
7 #include <android/bitmap.h> 7 #include <android/bitmap.h>
8 #include <stddef.h> 8 #include <stddef.h>
9 9
10 #include <utility> 10 #include <utility>
(...skipping 94 matching lines...) Expand 10 before | Expand all | Expand 10 after
105 } 105 }
106 106
107 TabContentManager::TabContentManager(JNIEnv* env, 107 TabContentManager::TabContentManager(JNIEnv* env,
108 jobject obj, 108 jobject obj,
109 jint default_cache_size, 109 jint default_cache_size,
110 jint approximation_cache_size, 110 jint approximation_cache_size,
111 jint compression_queue_max_size, 111 jint compression_queue_max_size,
112 jint write_queue_max_size, 112 jint write_queue_max_size,
113 jboolean use_approximation_thumbnail) 113 jboolean use_approximation_thumbnail)
114 : weak_java_tab_content_manager_(env, obj), weak_factory_(this) { 114 : weak_java_tab_content_manager_(env, obj), weak_factory_(this) {
115 thumbnail_cache_ = base::WrapUnique(new ThumbnailCache( 115 thumbnail_cache_ = base::MakeUnique<ThumbnailCache>(
116 (size_t)default_cache_size, (size_t)approximation_cache_size, 116 static_cast<size_t>(default_cache_size),
117 (size_t)compression_queue_max_size, (size_t)write_queue_max_size, 117 static_cast<size_t>(approximation_cache_size),
118 use_approximation_thumbnail)); 118 static_cast<size_t>(compression_queue_max_size),
119 static_cast<size_t>(write_queue_max_size), use_approximation_thumbnail);
119 thumbnail_cache_->AddThumbnailCacheObserver(this); 120 thumbnail_cache_->AddThumbnailCacheObserver(this);
120 } 121 }
121 122
122 TabContentManager::~TabContentManager() { 123 TabContentManager::~TabContentManager() {
123 } 124 }
124 125
125 void TabContentManager::Destroy(JNIEnv* env, const JavaParamRef<jobject>& obj) { 126 void TabContentManager::Destroy(JNIEnv* env, const JavaParamRef<jobject>& obj) {
126 thumbnail_cache_->RemoveThumbnailCacheObserver(this); 127 thumbnail_cache_->RemoveThumbnailCacheObserver(this);
127 delete this; 128 delete this;
128 } 129 }
(...skipping 101 matching lines...) Expand 10 before | Expand all | Expand 10 after
230 pending_tab_readbacks_.find(tab_id) != pending_tab_readbacks_.end() || 231 pending_tab_readbacks_.find(tab_id) != pending_tab_readbacks_.end() ||
231 pending_tab_readbacks_.size() >= kMaxReadbacks) { 232 pending_tab_readbacks_.size() >= kMaxReadbacks) {
232 thumbnail_cache_->Remove(tab_id); 233 thumbnail_cache_->Remove(tab_id);
233 return; 234 return;
234 } 235 }
235 236
236 TabReadbackCallback readback_done_callback = 237 TabReadbackCallback readback_done_callback =
237 base::Bind(&TabContentManager::PutThumbnailIntoCache, 238 base::Bind(&TabContentManager::PutThumbnailIntoCache,
238 weak_factory_.GetWeakPtr(), tab_id); 239 weak_factory_.GetWeakPtr(), tab_id);
239 pending_tab_readbacks_.set( 240 pending_tab_readbacks_.set(
240 tab_id, base::WrapUnique(new TabReadbackRequest( 241 tab_id, base::MakeUnique<TabReadbackRequest>(
241 web_contents, thumbnail_scale, readback_done_callback))); 242 web_contents, thumbnail_scale, readback_done_callback));
242 } 243 }
243 } 244 }
244 245
245 void TabContentManager::CacheTabWithBitmap(JNIEnv* env, 246 void TabContentManager::CacheTabWithBitmap(JNIEnv* env,
246 const JavaParamRef<jobject>& obj, 247 const JavaParamRef<jobject>& obj,
247 const JavaParamRef<jobject>& tab, 248 const JavaParamRef<jobject>& tab,
248 const JavaParamRef<jobject>& bitmap, 249 const JavaParamRef<jobject>& bitmap,
249 jfloat thumbnail_scale) { 250 jfloat thumbnail_scale) {
250 TabAndroid* tab_android = TabAndroid::GetNativeTab(env, tab); 251 TabAndroid* tab_android = TabAndroid::GetNativeTab(env, tab);
251 DCHECK(tab_android); 252 DCHECK(tab_android);
(...skipping 91 matching lines...) Expand 10 before | Expand all | Expand 10 after
343 jboolean use_approximation_thumbnail) { 344 jboolean use_approximation_thumbnail) {
344 TabContentManager* manager = new TabContentManager( 345 TabContentManager* manager = new TabContentManager(
345 env, obj, default_cache_size, approximation_cache_size, 346 env, obj, default_cache_size, approximation_cache_size,
346 compression_queue_max_size, write_queue_max_size, 347 compression_queue_max_size, write_queue_max_size,
347 use_approximation_thumbnail); 348 use_approximation_thumbnail);
348 return reinterpret_cast<intptr_t>(manager); 349 return reinterpret_cast<intptr_t>(manager);
349 } 350 }
350 351
351 } // namespace android 352 } // namespace android
352 } // namespace chrome 353 } // namespace chrome
OLDNEW
« no previous file with comments | « chrome/browser/android/bottombar/overlay_panel_content.cc ('k') | chrome/browser/android/data_usage/data_use_matcher.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698