Make cc::CompositorFrames movable [Part 1 of 2]

cc/output/renderer_unittest.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 "cc/output/delegating_renderer.h"
 #include "cc/output/gl_renderer.h"
 #include "cc/output/output_surface.h"
 #include "cc/test/fake_output_surface_client.h"
 #include "cc/test/fake_renderer_client.h"
 #include "cc/test/fake_resource_provider.h"
 #include "cc/test/test_context_provider.h"
 #include "cc/test/test_web_graphics_context_3d.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace cc {
 namespace {
 
 class TestOutputSurface : public OutputSurface {
  public:
   explicit TestOutputSurface(scoped_refptr<ContextProvider> context_provider);
   ~TestOutputSurface() override;
 
-  // OutputSurface implementation.
-  void SwapBuffers(CompositorFrame* frame) override;
+  // OutputSurface implementation

[danakj, 2016/06/24 18:35:11]

put the period back please :)

[Fady Samuel, 2016/06/24 20:00:24]

Done.

+  void SwapBuffers(CompositorFrame frame) override;
   uint32_t GetFramebufferCopyTextureFormat() override { return GL_RGB; }
 };
 
 TestOutputSurface::TestOutputSurface(
     scoped_refptr<ContextProvider> context_provider)
     : OutputSurface(std::move(context_provider), nullptr, nullptr) {}
 
 TestOutputSurface::~TestOutputSurface() {
 }
 
-void TestOutputSurface::SwapBuffers(CompositorFrame* frame) {
+void TestOutputSurface::SwapBuffers(CompositorFrame frame) {
   client_->DidSwapBuffers();
   client_->DidSwapBuffersComplete();
 }
 
 class MockContextProvider : public TestContextProvider {
  public:
   explicit MockContextProvider(
       std::unique_ptr<TestWebGraphicsContext3D> context)
       : TestContextProvider(std::move(context)) {}
   MOCK_METHOD0(DeleteCachedResources, void());
@@ skipping 60 matching lines @@
   EXPECT_CALL(*(this->context_provider_.get()), DeleteCachedResources())
       .Times(1);
 
   EXPECT_TRUE(this->renderer_->visible());
   this->renderer_->SetVisible(false);
   EXPECT_FALSE(this->renderer_->visible());
 }
 
 }  // namespace
 }  // namespace cc