12-bit vp9 video support

cc/resources/video_resource_updater.h

 // Copyright 2013 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 CC_RESOURCES_VIDEO_RESOURCE_UPDATER_H_
 #define CC_RESOURCES_VIDEO_RESOURCE_UPDATER_H_
 
 #include <stddef.h>
 #include <stdint.h>
 
@@ skipping 62 matching lines @@
 class CC_EXPORT VideoResourceUpdater
     : public base::SupportsWeakPtr<VideoResourceUpdater> {
  public:
   VideoResourceUpdater(ContextProvider* context_provider,
                        ResourceProvider* resource_provider);
   ~VideoResourceUpdater();
 
   VideoFrameExternalResources CreateExternalResourcesFromVideoFrame(
       scoped_refptr<media::VideoFrame> video_frame);
 
+  // Public for testing only.
+  static void MakeHalfFloats(const uint16_t* src,

[danakj, 2016/09/26 20:35:04]

There are 4 parameters, all are shortened names, and there's 0 comments
explaining what this function does or what the parameters are.

Also, maybe this shouldn't be on VideoResourceUpdater? Though I'd have to
reverse engineer some documentation for this function to suggest anywhere else.

[hubbe, 2016/09/26 21:19:36]

Comment added.

+                             float mult,
+                             size_t num,
+                             uint16_t* dst);
+
  private:
   class PlaneResource {
    public:
     PlaneResource(unsigned resource_id,
                   const gfx::Size& resource_size,
                   ResourceFormat resource_format,
                   gpu::Mailbox mailbox);
     PlaneResource(const PlaneResource& other);
 
     // Returns true if this resource matches the unique identifiers of another
@@ skipping 87 matching lines @@
   // Recycle resources so that we can reduce the number of allocations and
   // data transfers.
   ResourceList all_resources_;
 
   DISALLOW_COPY_AND_ASSIGN(VideoResourceUpdater);
 };
 
 }  // namespace cc
 
 #endif  // CC_RESOURCES_VIDEO_RESOURCE_UPDATER_H_