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

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

Issue 190693002: Migrate Telemetry from beginWindowSnapshotPNG to Page.captureScreenshot (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase Created 6 years, 7 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/public/browser/render_widget_host.h ('k') | content/renderer/render_view_impl.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/gpu/gpu_benchmarking_extension.cc
diff --git a/content/renderer/gpu/gpu_benchmarking_extension.cc b/content/renderer/gpu/gpu_benchmarking_extension.cc
index c3e5427b2afc4270f71bb23d7391889700736548..1b09c99390de469623c4dd8b1880b3d4aebf0c55 100644
--- a/content/renderer/gpu/gpu_benchmarking_extension.cc
+++ b/content/renderer/gpu/gpu_benchmarking_extension.cc
@@ -310,10 +310,6 @@ class GpuBenchmarkingWrapper : public v8::Extension {
" return BeginTap(position_x, position_y, callback, duration_ms,"
" gesture_source_type);"
"};"
- "chrome.gpuBenchmarking.beginWindowSnapshotPNG = function(callback) {"
- " native function BeginWindowSnapshotPNG();"
- " BeginWindowSnapshotPNG(callback);"
- "};"
"chrome.gpuBenchmarking.clearImageCache = function() {"
" native function ClearImageCache();"
" ClearImageCache();"
@@ -351,9 +347,6 @@ class GpuBenchmarkingWrapper : public v8::Extension {
return v8::FunctionTemplate::New(isolate, BeginPinch);
if (name->Equals(v8::String::NewFromUtf8(isolate, "BeginTap")))
return v8::FunctionTemplate::New(isolate, BeginTap);
- if (name->Equals(
- v8::String::NewFromUtf8(isolate, "BeginWindowSnapshotPNG")))
- return v8::FunctionTemplate::New(isolate, BeginWindowSnapshotPNG);
if (name->Equals(v8::String::NewFromUtf8(isolate, "ClearImageCache")))
return v8::FunctionTemplate::New(isolate, ClearImageCache);
if (name->Equals(v8::String::NewFromUtf8(isolate, "RunMicroBenchmark")))
@@ -790,27 +783,6 @@ class GpuBenchmarkingWrapper : public v8::Extension {
}
}
- static void BeginWindowSnapshotPNG(
- const v8::FunctionCallbackInfo<v8::Value>& args) {
- GpuBenchmarkingContext context;
- if (!context.Init(false))
- return;
-
- if (!args[0]->IsFunction())
- return;
-
- v8::Local<v8::Function> callback_local =
- v8::Local<v8::Function>::Cast(args[0]);
-
- scoped_refptr<CallbackAndContext> callback_and_context =
- new CallbackAndContext(args.GetIsolate(),
- callback_local,
- context.web_frame()->mainWorldScriptContext());
-
- context.render_view_impl()->GetWindowSnapshot(
- base::Bind(&OnSnapshotCompleted, callback_and_context));
- }
-
static void ClearImageCache(
const v8::FunctionCallbackInfo<v8::Value>& args) {
WebImageCache::clear();
« no previous file with comments | « content/public/browser/render_widget_host.h ('k') | content/renderer/render_view_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698