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

Unified Diff: content/browser/android/overscroll_glow.cc

Issue 454863003: Revert of [Android] Use UIResource for overscroll glow (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 4 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 side-by-side diff with in-line comments
Download patch
Index: content/browser/android/overscroll_glow.cc
diff --git a/content/browser/android/overscroll_glow.cc b/content/browser/android/overscroll_glow.cc
index 3f316d8aee62236969e084b2072c555a2e7b8859..fb491f3ebaacfbd5b4364601e56e10de80b23dbb 100644
--- a/content/browser/android/overscroll_glow.cc
+++ b/content/browser/android/overscroll_glow.cc
@@ -6,8 +6,11 @@
#include "base/debug/trace_event.h"
#include "base/lazy_instance.h"
-#include "cc/layers/layer.h"
+#include "base/threading/worker_pool.h"
+#include "cc/layers/image_layer.h"
#include "content/browser/android/edge_effect.h"
+#include "skia/ext/image_operations.h"
+#include "ui/gfx/android/java_bitmap.h"
using std::max;
using std::min;
@@ -17,8 +20,56 @@
namespace {
const float kEpsilon = 1e-3f;
+const int kScaledEdgeHeight = 12;
+const int kScaledGlowHeight = 64;
const float kEdgeHeightAtMdpi = 12.f;
const float kGlowHeightAtMdpi = 128.f;
+
+SkBitmap CreateSkBitmapFromAndroidResource(const char* name, gfx::Size size) {
+ base::android::ScopedJavaLocalRef<jobject> jobj =
+ gfx::CreateJavaBitmapFromAndroidResource(name, size);
+ if (jobj.is_null())
+ return SkBitmap();
+
+ SkBitmap bitmap = CreateSkBitmapFromJavaBitmap(gfx::JavaBitmap(jobj.obj()));
+ if (bitmap.isNull())
+ return bitmap;
+
+ return skia::ImageOperations::Resize(
+ bitmap, skia::ImageOperations::RESIZE_BOX, size.width(), size.height());
+}
+
+class OverscrollResources {
+ public:
+ OverscrollResources() {
+ TRACE_EVENT0("browser", "OverscrollResources::Create");
+ edge_bitmap_ =
+ CreateSkBitmapFromAndroidResource("android:drawable/overscroll_edge",
+ gfx::Size(128, kScaledEdgeHeight));
+ glow_bitmap_ =
+ CreateSkBitmapFromAndroidResource("android:drawable/overscroll_glow",
+ gfx::Size(128, kScaledGlowHeight));
+ }
+
+ const SkBitmap& edge_bitmap() const { return edge_bitmap_; }
+ const SkBitmap& glow_bitmap() const { return glow_bitmap_; }
+
+ private:
+ SkBitmap edge_bitmap_;
+ SkBitmap glow_bitmap_;
+
+ DISALLOW_COPY_AND_ASSIGN(OverscrollResources);
+};
+
+// Leaky to allow access from a worker thread.
+base::LazyInstance<OverscrollResources>::Leaky g_overscroll_resources =
+ LAZY_INSTANCE_INITIALIZER;
+
+scoped_refptr<cc::Layer> CreateImageLayer(const SkBitmap& bitmap) {
+ scoped_refptr<cc::ImageLayer> layer = cc::ImageLayer::Create();
+ layer->SetBitmap(bitmap);
+ return layer;
+}
bool IsApproxZero(float value) {
return std::abs(value) < kEpsilon;
@@ -32,18 +83,25 @@
return vector;
}
-} // namespace
-
-scoped_ptr<OverscrollGlow> OverscrollGlow::Create(
- ui::SystemUIResourceManager* resource_manager) {
- return make_scoped_ptr(new OverscrollGlow(resource_manager));
-}
-
-OverscrollGlow::OverscrollGlow(ui::SystemUIResourceManager* resource_manager)
- : enabled_(true), initialized_(false), resource_manager_(resource_manager) {
- DCHECK(resource_manager_);
- EdgeEffect::PreloadResources(resource_manager_);
-}
+// Force loading of any necessary resources. This function is thread-safe.
+void EnsureResources() {
+ g_overscroll_resources.Get();
+}
+
+} // namespace
+
+scoped_ptr<OverscrollGlow> OverscrollGlow::Create(bool enabled) {
+ // Don't block the main thread with effect resource loading during creation.
+ // Effect instantiation is deferred until the effect overscrolls, in which
+ // case the main thread may block until the resource has loaded.
+ if (enabled && g_overscroll_resources == NULL)
+ base::WorkerPool::PostTask(FROM_HERE, base::Bind(EnsureResources), true);
+
+ return make_scoped_ptr(new OverscrollGlow(enabled));
+}
+
+OverscrollGlow::OverscrollGlow(bool enabled)
+ : enabled_(enabled), initialized_(false) {}
OverscrollGlow::~OverscrollGlow() {
Detach();
@@ -164,9 +222,6 @@
if (root_layer_->parent() != parent)
parent->AddChild(root_layer_);
-
- for (size_t i = 0; i < EdgeEffect::EDGE_COUNT; ++i)
- edge_effects_[i]->SetParent(root_layer_);
}
void OverscrollGlow::Detach() {
@@ -179,10 +234,22 @@
if (initialized_)
return true;
+ const SkBitmap& edge = g_overscroll_resources.Get().edge_bitmap();
+ const SkBitmap& glow = g_overscroll_resources.Get().glow_bitmap();
+ if (edge.isNull() || glow.isNull()) {
+ Disable();
+ return false;
+ }
+
DCHECK(!root_layer_);
root_layer_ = cc::Layer::Create();
- for (size_t i = 0; i < EdgeEffect::EDGE_COUNT; ++i)
- edge_effects_[i] = make_scoped_ptr(new EdgeEffect(resource_manager_));
+ for (size_t i = 0; i < EdgeEffect::EDGE_COUNT; ++i) {
+ scoped_refptr<cc::Layer> edge_layer = CreateImageLayer(edge);
+ scoped_refptr<cc::Layer> glow_layer = CreateImageLayer(glow);
+ root_layer_->AddChild(edge_layer);
+ root_layer_->AddChild(glow_layer);
+ edge_effects_[i] = make_scoped_ptr(new EdgeEffect(edge_layer, glow_layer));
+ }
initialized_ = true;
return true;
« no previous file with comments | « content/browser/android/overscroll_glow.h ('k') | content/browser/android/system_ui_resource_manager_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698