Chromium Code Reviews| OLD | NEW |
|---|---|
| 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 "ui/android/resources/ui_resource_android.h" | 5 #include "ui/android/resources/ui_resource_android.h" |
| 6 | 6 |
| 7 #include "base/logging.h" | 7 #include "base/logging.h" |
| 8 #include "ui/android/resources/ui_resource_provider.h" | 8 #include "ui/android/resources/ui_resource_provider.h" |
| 9 | 9 |
| 10 namespace ui { | 10 namespace ui { |
| 11 | 11 |
| 12 scoped_ptr<UIResourceAndroid> UIResourceAndroid::CreateFromJavaBitmap( | 12 scoped_ptr<UIResourceAndroid> UIResourceAndroid::CreateFromJavaBitmap( |
| 13 ui::UIResourceProvider* provider, | 13 ui::UIResourceProvider* provider, |
| 14 const gfx::JavaBitmap& java_bitmap) { | 14 const gfx::JavaBitmap& java_bitmap) { |
| 15 SkBitmap skbitmap = gfx::CreateSkBitmapFromJavaBitmap(java_bitmap); | 15 SkBitmap skbitmap = gfx::CreateSkBitmapFromJavaBitmap(java_bitmap); |
| 16 skbitmap.setImmutable(); | 16 skbitmap.setImmutable(); |
| 17 | 17 |
| 18 return make_scoped_ptr(new UIResourceAndroid(provider, skbitmap)); | 18 return make_scoped_ptr(new UIResourceAndroid(provider, skbitmap)); |
| 19 } | 19 } |
| 20 | 20 |
| 21 UIResourceAndroid::~UIResourceAndroid() { | 21 UIResourceAndroid::~UIResourceAndroid() { |
| 22 if (id_ && provider_) | 22 if (id_ && provider_) |
| 23 provider_->DeleteUIResource(id_); | 23 provider_->DeleteUIResource(id_); |
| 24 } | 24 } |
| 25 | 25 |
| 26 const SkBitmap& UIResourceAndroid::GetSkBitmap() { | |
| 27 // TODO(twellington): Would it be better to use CreateSkBitmapFromJavaBitmap | |
| 28 // directly rather than exposing this method? I mostly did it this way because | |
|
pedro (no code reviews)
2015/09/29 01:14:40
I don't know the answer to this question.
Theresa
2015/10/01 01:57:31
Since this is going to undergo another major refac
| |
| 29 // it was easy/quick to implement w/ our current contextual search code. | |
| 30 DCHECK(!bitmap_.empty()); | |
| 31 return bitmap_; | |
| 32 } | |
| 33 | |
| 26 cc::UIResourceBitmap UIResourceAndroid::GetBitmap(cc::UIResourceId uid, | 34 cc::UIResourceBitmap UIResourceAndroid::GetBitmap(cc::UIResourceId uid, |
| 27 bool resource_lost) { | 35 bool resource_lost) { |
| 28 DCHECK(!bitmap_.empty()); | 36 DCHECK(!bitmap_.empty()); |
| 29 return cc::UIResourceBitmap(bitmap_); | 37 return cc::UIResourceBitmap(bitmap_); |
| 30 } | 38 } |
| 31 | 39 |
| 32 cc::UIResourceId UIResourceAndroid::id() { | 40 cc::UIResourceId UIResourceAndroid::id() { |
| 33 if (id_) | 41 if (id_) |
| 34 return id_; | 42 return id_; |
| 35 if (!provider_ || bitmap_.empty()) | 43 if (!provider_ || bitmap_.empty()) |
| 36 return 0; | 44 return 0; |
| 37 id_ = provider_->CreateUIResource(this); | 45 id_ = provider_->CreateUIResource(this); |
| 38 return id_; | 46 return id_; |
| 39 } | 47 } |
| 40 | 48 |
| 41 void UIResourceAndroid::UIResourceIsInvalid() { | 49 void UIResourceAndroid::UIResourceIsInvalid() { |
| 42 id_ = 0; | 50 id_ = 0; |
| 43 } | 51 } |
| 44 | 52 |
| 45 UIResourceAndroid::UIResourceAndroid(ui::UIResourceProvider* provider, | 53 UIResourceAndroid::UIResourceAndroid(ui::UIResourceProvider* provider, |
| 46 const SkBitmap& skbitmap) | 54 const SkBitmap& skbitmap) |
| 47 : provider_(provider), bitmap_(skbitmap), id_(0) { | 55 : provider_(provider), bitmap_(skbitmap), id_(0) { |
| 48 } | 56 } |
| 49 | 57 |
| 50 } // namespace ui | 58 } // namespace ui |
| OLD | NEW |