DevTools: FrameRecorder: add cancelRecordingFrames command

content/browser/devtools/protocol/frame_recorder.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/devtools/protocol/frame_recorder.h"
 
 #include "base/base64.h"
 #include "base/bind.h"
 #include "base/macros.h"
 #include "base/task_runner_util.h"
 #include "base/threading/worker_pool.h"
 #include "content/browser/renderer_host/render_view_host_impl.h"
 #include "content/browser/renderer_host/render_widget_host_view_base.h"
 #include "third_party/skia/include/core/SkBitmap.h"
 #include "ui/gfx/codec/png_codec.h"
 #include "ui/gfx/geometry/size.h"
 
 namespace content {
 namespace devtools {
 namespace page {
 
 namespace {
 
 static int kMaxRecordFrameCount = 180;
 
-std::string EncodeFrame(const SkBitmap& bitmap) {
+scoped_ptr<EncodedFrame> EncodeFrame(
+    const SkBitmap& bitmap, double timestamp) {
   std::vector<unsigned char> data;
   SkAutoLockPixels lock_image(bitmap);
   bool encoded = gfx::PNGCodec::Encode(
       reinterpret_cast<unsigned char*>(bitmap.getAddr32(0, 0)),
       gfx::PNGCodec::FORMAT_SkBitmap,
       gfx::Size(bitmap.width(), bitmap.height()),
       bitmap.width() * bitmap.bytesPerPixel(),
       false, std::vector<gfx::PNGCodec::Comment>(), &data);
 
+  scoped_ptr<EncodedFrame> result(new EncodedFrame(std::string(), timestamp));
+
   if (!encoded)
-    return std::string();
+    return result.Pass();
 
   std::string base_64_data;
   base::Base64Encode(
       base::StringPiece(reinterpret_cast<char*>(&data[0]), data.size()),
-      &base_64_data);
+      &result->first);
 
-  return base_64_data;
+  return result.Pass();
 }
 } // namespace
 
 typedef DevToolsProtocolClient::Response Response;
 
 FrameRecorder::FrameRecorder()
     : host_(nullptr),
       state_(Ready),
       inflight_requests_count_(0),
       max_frame_count_(0),
@@ skipping 28 matching lines @@
 Response FrameRecorder::StopRecordingFrames(
     StopRecordingFramesCallback callback) {
   if (state_ != Recording)
     return Response::InternalError("Not recording");
   state_ = Encoding;
   callback_ = callback;
   MaybeSendResponse();
   return Response::OK();
 }
 
+Response FrameRecorder::CancelRecordingFrames() {
+  frame_encoded_callback_.Cancel();

[dgozman, 2015/01/30 10:06:23]

Where do you initialize this field?

[eustas, 2015/01/30 12:08:16]

In StartRecoding... Removed too much during cleanup =)

+  std::vector<scoped_refptr<devtools::page::RecordedFrame>> no_frames;
+  frames_.swap(no_frames);
+  if (state_ == Encoding)
+    callback_.Run(StopRecordingFramesResponse::Create()->set_frames(frames_));
+  state_ = Ready;
+  return Response::OK();
+}
+
 void FrameRecorder::OnSwapCompositorFrame() {
   if (!host_ || state_ != Recording)
     return;
   if (captured_frames_count_ >= max_frame_count_)
     return;
   RenderWidgetHostViewBase* view =
       static_cast<RenderWidgetHostViewBase*>(host_->GetView());
   if (!view)
     return;
 
@@ skipping 10 matching lines @@
   inflight_requests_count_--;
   base::Time timestamp = last_captured_frame_timestamp_;
   last_captured_frame_timestamp_ = base::Time::Now();
   if (timestamp.is_null() || response != READBACK_SUCCESS)
     return;
 
   captured_frames_count_++;
   base::PostTaskAndReplyWithResult(
       base::WorkerPool::GetTaskRunner(true).get(),
       FROM_HERE,
-      base::Bind(&EncodeFrame, bitmap),
-      base::Bind(&FrameRecorder::FrameEncoded, weak_factory_.GetWeakPtr(),
-                 timestamp.ToDoubleT()));
+      base::Bind(&EncodeFrame, bitmap, timestamp.ToDoubleT()),
+      frame_encoded_callback_.callback());
 }
 
 void FrameRecorder::FrameEncoded(
-    double timestamp, const std::string& encoded_frame) {
+  const scoped_ptr<EncodedFrame>& encoded_frame) {
   frames_.push_back(RecordedFrame::Create()
-      ->set_data(encoded_frame)
-      ->set_timestamp(timestamp));
+      ->set_data(encoded_frame->first)
+      ->set_timestamp(encoded_frame->second));
   MaybeSendResponse();
 }
 
 void FrameRecorder::MaybeSendResponse() {
   if (state_ != Encoding)
     return;
   if (inflight_requests_count_ || frames_.size() != captured_frames_count_)
     return;
   callback_.Run(StopRecordingFramesResponse::Create()->set_frames(frames_));
   std::vector<scoped_refptr<devtools::page::RecordedFrame>> frames;
   frames_.swap(frames);
   state_ = Ready;
 }
 
 }  // namespace page
 }  // namespace devtools
 }  // namespace content