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

Unified Diff: content/renderer/gpu/render_widget_compositor.cc

Issue 2769823002: Add decode() functionality to image elements. (Closed)
Patch Set: update Created 3 years, 8 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/renderer/gpu/render_widget_compositor.cc
diff --git a/content/renderer/gpu/render_widget_compositor.cc b/content/renderer/gpu/render_widget_compositor.cc
index 304fd16061c9a25a90b564896cf8e334770e2f7c..4ac1959079ed711b9fa0f80e0a8ffa3d92a5133d 100644
--- a/content/renderer/gpu/render_widget_compositor.cc
+++ b/content/renderer/gpu/render_widget_compositor.cc
@@ -66,6 +66,7 @@
#include "third_party/WebKit/public/web/WebKit.h"
#include "third_party/WebKit/public/web/WebRuntimeFeatures.h"
#include "third_party/WebKit/public/web/WebSelection.h"
+#include "third_party/skia/include/core/SkImage.h"
#include "ui/gl/gl_switches.h"
#include "ui/native_theme/native_theme_switches.h"
#include "ui/native_theme/overlay_scrollbar_constants_aura.h"
@@ -1024,6 +1025,23 @@ void RenderWidgetCompositor::setBottomControlsHeight(float height) {
layer_tree_host_->SetBottomControlsHeight(height);
}
+void RenderWidgetCompositor::requestDecode(
+ sk_sp<SkImage> image,
+ const base::Callback<void(bool)>& callback) {
+ layer_tree_host_->QueueImageDecode(std::move(image), callback);
+
+ // If we're compositing synchronously, the SetNeedsCommit call which will be
+ // issued by |layer_tree_host_| is not going to cause a commit, due to the
+ // fact that this would make layout tests slow and cause flakiness. However,
+ // in this case we actually need a commit to transfer the decode requests to
+ // the impl side. So, force a commit to happen.
+ if (CompositeIsSynchronous()) {
+ base::ThreadTaskRunnerHandle::Get()->PostTask(
+ FROM_HERE, base::Bind(&RenderWidgetCompositor::SynchronouslyComposite,
+ weak_factory_.GetWeakPtr()));
+ }
+}
+
void RenderWidgetCompositor::WillBeginMainFrame() {
delegate_->WillBeginCompositorFrame();
}

Powered by Google App Engine
This is Rietveld 408576698