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

Unified Diff: content/renderer/skia_benchmarking_extension.cc

Issue 15967010: Add skiaBenchmarking.getOps() (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Helper test function instead of macro. Created 7 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « content/content_tests.gypi ('k') | content/renderer/skia_benchmarking_extension_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/skia_benchmarking_extension.cc
diff --git a/content/renderer/skia_benchmarking_extension.cc b/content/renderer/skia_benchmarking_extension.cc
index c916c3942d24a7bf8ba854abda0de777f19ce079..0a1c6086e67a16950a291c842804eb348c3079bd 100644
--- a/content/renderer/skia_benchmarking_extension.cc
+++ b/content/renderer/skia_benchmarking_extension.cc
@@ -12,6 +12,8 @@
#include "third_party/skia/include/core/SkCanvas.h"
#include "third_party/skia/include/core/SkColorPriv.h"
#include "third_party/skia/include/core/SkGraphics.h"
+#include "third_party/skia/src/utils/debugger/SkDebugCanvas.h"
+#include "third_party/skia/src/utils/debugger/SkDrawCommand.h"
#include "ui/gfx/rect_conversions.h"
#include "v8/include/v8.h"
@@ -21,6 +23,19 @@ namespace {
const char kSkiaBenchmarkingExtensionName[] = "v8/SkiaBenchmarking";
+static scoped_refptr<cc::Picture> ParsePictureArg(v8::Handle<v8::Value> arg) {
+ scoped_ptr<content::V8ValueConverter> converter(
+ content::V8ValueConverter::create());
+
+ v8::String::Value v8_picture(arg);
+ scoped_ptr<base::Value> picture_value(
+ converter->FromV8Value(arg, v8::Context::GetCurrent()));
+ if (!picture_value)
+ return NULL;
+
+ return cc::Picture::CreateFromValue(picture_value.get());
+}
+
class SkiaBenchmarkingWrapper : public v8::Extension {
public:
SkiaBenchmarkingWrapper() :
@@ -39,10 +54,23 @@ class SkiaBenchmarkingWrapper : public v8::Extension {
" @param [Number, Number, Number, Number] clip_rect (optional)."
" @returns { 'width': {Number}, 'height': {Number},"
" 'data': {ArrayBuffer} }"
+ " @returns undefined if the arguments are invalid or the picture"
+ " version is not supported."
" */"
" native function Rasterize();"
" return Rasterize(picture, scale, rect);"
"};"
+ "chrome.skiaBenchmarking.getOps = function(picture) {"
+ " /* "
+ " Extracts the Skia draw commands from a JSON-encoded cc::Picture"
+ " @param {Object} picture A json-encoded cc::Picture."
+ " @returns [{ 'cmd': {String}, 'info': [String, ...] }, ...]"
+ " @returns undefined if the arguments are invalid or the picture"
+ " version is not supported."
+ " */"
+ " native function GetOps();"
+ " return GetOps(picture);"
+ "};"
) {
content::SkiaBenchmarkingExtension::InitSkGraphics();
}
@@ -51,6 +79,8 @@ class SkiaBenchmarkingWrapper : public v8::Extension {
v8::Handle<v8::String> name) OVERRIDE {
if (name->Equals(v8::String::New("Rasterize")))
return v8::FunctionTemplate::New(Rasterize);
+ if (name->Equals(v8::String::New("GetOps")))
+ return v8::FunctionTemplate::New(GetOps);
return v8::Handle<v8::FunctionTemplate>();
}
@@ -59,22 +89,7 @@ class SkiaBenchmarkingWrapper : public v8::Extension {
if (args.Length() < 1)
return;
- WebFrame* web_frame = WebFrame::frameForCurrentContext();
- if (!web_frame)
- return;
-
- scoped_ptr<content::V8ValueConverter> converter(
- content::V8ValueConverter::create());
-
- v8::String::Value v8_picture(args[0]);
- scoped_ptr<base::Value> picture_value(
- converter->FromV8Value(
- args[0], v8::Context::GetCurrent()));
- if (!picture_value)
- return;
-
- scoped_refptr<cc::Picture> picture =
- cc::Picture::CreateFromValue(picture_value.get());
+ scoped_refptr<cc::Picture> picture = ParsePictureArg(args[0]);
if (!picture.get())
return;
@@ -140,6 +155,44 @@ class SkiaBenchmarkingWrapper : public v8::Extension {
args.GetReturnValue().Set(result);
}
+
+ static void GetOps(const v8::FunctionCallbackInfo<v8::Value>& args) {
+ if (args.Length() != 1)
+ return;
+
+ scoped_refptr<cc::Picture> picture = ParsePictureArg(args[0]);
+ if (!picture.get())
+ return;
+
+ gfx::Rect bounds = picture->LayerRect();
+ SkDebugCanvas canvas(bounds.width(), bounds.height());
+ picture->Raster(&canvas, NULL, bounds, 1.0f);
+
+ v8::Local<v8::Array> result = v8::Array::New(canvas.getSize());
+ for (int i = 0; i < canvas.getSize(); ++i) {
+ DrawType cmd_type = canvas.getDrawCommandAt(i)->getType();
+ v8::Handle<v8::Object> cmd = v8::Object::New();
+ cmd->Set(v8::String::New("cmd_type"), v8::Integer::New(cmd_type));
+ cmd->Set(v8::String::New("cmd_string"), v8::String::New(
+ SkDrawCommand::GetCommandString(cmd_type)));
+
+ SkTDArray<SkString*>* info = canvas.getCommandInfo(i);
+ DCHECK(info);
+
+ v8::Local<v8::Array> v8_info = v8::Array::New(info->count());
+ for (int j = 0; j < info->count(); ++j) {
+ const SkString* info_str = (*info)[j];
+ DCHECK(info_str);
+ v8_info->Set(j, v8::String::New(info_str->c_str()));
+ }
+
+ cmd->Set(v8::String::New("info"), v8_info);
+
+ result->Set(i, cmd);
+ }
+
+ args.GetReturnValue().Set(result);
+ }
};
} // namespace
« no previous file with comments | « content/content_tests.gypi ('k') | content/renderer/skia_benchmarking_extension_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698