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

Side by Side Diff: ui/app_list/app_list_item_view.cc

Issue 10386206: RefCounted types should not have public destructors, chromeos edition (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase to r143931 Created 8 years, 6 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 | Annotate | Revision Log
« no previous file with comments | « content/browser/renderer_host/image_transport_factory.cc ('k') | ui/aura/bench/bench_main.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) 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 #include "ui/app_list/app_list_item_view.h" 5 #include "ui/app_list/app_list_item_view.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/message_loop.h" 10 #include "base/message_loop.h"
(...skipping 104 matching lines...) Expand 10 before | Expand all | Expand 10 after
115 void Cancel() { 115 void Cancel() {
116 cancel_flag_.Set(); 116 cancel_flag_.Set();
117 } 117 }
118 118
119 const SkBitmap& bitmap() const { 119 const SkBitmap& bitmap() const {
120 return bitmap_; 120 return bitmap_;
121 } 121 }
122 122
123 private: 123 private:
124 friend class base::RefCountedThreadSafe<AppListItemView::IconOperation>; 124 friend class base::RefCountedThreadSafe<AppListItemView::IconOperation>;
125 ~IconOperation() {}
125 126
126 base::CancellationFlag cancel_flag_; 127 base::CancellationFlag cancel_flag_;
127 128
128 SkBitmap bitmap_; 129 SkBitmap bitmap_;
129 const gfx::Size size_; 130 const gfx::Size size_;
130 const gfx::ShadowValues shadows_; 131 const gfx::ShadowValues shadows_;
131 132
132 DISALLOW_COPY_AND_ASSIGN(IconOperation); 133 DISALLOW_COPY_AND_ASSIGN(IconOperation);
133 }; 134 };
134 135
(...skipping 168 matching lines...) Expand 10 before | Expand all | Expand 10 after
303 apps_grid_view_->SetSelectedItem(this); 304 apps_grid_view_->SetSelectedItem(this);
304 title_->SetEnabledColor(kTitleHoverColor); 305 title_->SetEnabledColor(kTitleHoverColor);
305 } else { 306 } else {
306 apps_grid_view_->ClearSelectedItem(this); 307 apps_grid_view_->ClearSelectedItem(this);
307 model_->SetHighlighted(false); 308 model_->SetHighlighted(false);
308 title_->SetEnabledColor(kTitleColor); 309 title_->SetEnabledColor(kTitleColor);
309 } 310 }
310 } 311 }
311 312
312 } // namespace app_list 313 } // namespace app_list
OLDNEW
« no previous file with comments | « content/browser/renderer_host/image_transport_factory.cc ('k') | ui/aura/bench/bench_main.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698