Integrate Surface requests with MediaPlayerRenderer

media/mojo/clients/mojo_renderer.h

Index: media/mojo/clients/mojo_renderer.h
diff --git a/media/mojo/clients/mojo_renderer.h b/media/mojo/clients/mojo_renderer.h
index e4a351e74602d9965aabe20039b73241d76bb824..3063dfc72966b4fae86efae872b688f004e08ce8 100644
--- a/media/mojo/clients/mojo_renderer.h
+++ b/media/mojo/clients/mojo_renderer.h
@@ -9,6 +9,7 @@
 
 #include "base/macros.h"
 #include "base/time/default_tick_clock.h"
+#include "base/unguessable_token.h"
 #include "media/base/demuxer_stream.h"
 #include "media/base/renderer.h"
 #include "media/base/time_delta_interpolator.h"
@@ -58,6 +59,14 @@ class MojoRenderer : public Renderer, public mojom::RendererClient {
   bool HasAudio() override;
   bool HasVideo() override;
 
+  using ReceiveSurfaceRequestTokenCB =
+      base::Callback<void(const base::Optional<base::UnguessableToken>&)>;
+
+  // Asks |remote_renderer_| to register a request in the browser's
+  // ScopedSurfaceRequestManager, and returns the request's token.
+  void InitiateScopedSurfaceRequest(
+      ReceiveSurfaceRequestTokenCB receive_request_token_cb);

[dcheng, 2016/09/26 17:41:51]

Callbacks are typically passed by const ref rather than by value (even though
copying it is relatively lightweight, it's not free).

(Strictly speaking, the preferred method going forward will be to pass by value
and use std::move() where transferring ownership, but that's not consistent with
how it's used in the codebase today. At some point, we're going to have a
migration plan and convert a lot of code to this new model though. Something to
think about...)

[tguilbert, 2016/09/26 19:46:35]

Ah yes! Done.

+
  private:
   // mojom::RendererClient implementation, dispatched on the
   // |task_runner_|.