Add video-rendering to mojo media pipeline.

media/base/pipeline_status.h

 // Copyright (c) 2012 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.
 
 #ifndef MEDIA_BASE_PIPELINE_STATUS_H_
 #define MEDIA_BASE_PIPELINE_STATUS_H_
 
-#include "base/callback.h"
-
 #include <stdint.h>
 
 #include <string>
 
+#include "base/callback.h"
+#include "ui/gfx/geometry/size.h"
+
 namespace media {
 
 // Status states for pipeline.  All codes except PIPELINE_OK indicate errors.
 // Logged to UMA, so never reuse a value, always add new/greater ones!
 enum PipelineStatus {
   PIPELINE_OK = 0,
   // Deprecated: PIPELINE_ERROR_URL_NOT_FOUND = 1,
   PIPELINE_ERROR_NETWORK = 2,
   PIPELINE_ERROR_DECODE = 3,
   // Deprecated: PIPELINE_ERROR_DECRYPT = 4,
@@ skipping 33 matching lines @@
   uint32_t video_frames_decoded = 0;
   uint32_t video_frames_dropped = 0;
   int64_t audio_memory_usage = 0;
   int64_t video_memory_usage = 0;
 };
 
 // Used for updating pipeline statistics; the passed value should be a delta
 // of all attributes since the last update.
 typedef base::Callback<void(const PipelineStatistics&)> StatisticsCB;
 
+// Used for updating video natural size; the passed value is the new natural
+// size of video frames being rendered.
+typedef base::Callback<void(const gfx::Size&)> NaturalSizeChangedCB;

[xhwang, 2016/04/12 19:40:37]

nit:

This seems an odd place to declare this callback. Maybe we should have a
separate file for this callback. Note that StatisticsCB doesn't quite belong to
this file either.

Or maybe we should rename this file.

Think about it twice I do prefer the former.

[alokp, 2016/04/19 00:16:07]

I thought about this for some time and could not come up with a good file name.
May be we should move all the callback definitions to renderer_callbacks.h or
pipeline_callbacks.h. WDYT?

+
 }  // namespace media
 
 #endif  // MEDIA_BASE_PIPELINE_STATUS_H_