cc: Remove unneeded methods from OutputSurface.

blimp/client/core/compositor/delegated_output_surface.h

 // Copyright 2016 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 BLIMP_CLIENT_CORE_COMPOSITOR_DELEGATED_OUTPUT_SURFACE_H_
 #define BLIMP_CLIENT_CORE_COMPOSITOR_DELEGATED_OUTPUT_SURFACE_H_
 
 #include <memory>
 
 #include "base/macros.h"
@@ skipping 31 matching lines @@
   bool BindToClient(cc::OutputSurfaceClient* client) override;
   void SwapBuffers(cc::CompositorFrame frame) override;
   void DetachFromClient() override;
 
   // BlimpOutputSurface implementation.
   void ReclaimCompositorResources(
       const cc::ReturnedResourceArray& resources) override;
 
  private:
   scoped_refptr<base::SingleThreadTaskRunner> main_task_runner_;
-  base::WeakPtr<BlimpOutputSurfaceClient> client_;
+  base::WeakPtr<BlimpOutputSurfaceClient> blimp_client_;

[danakj, 2016/09/10 00:58:24]

I renamed this because the base class has a client_ already and this was
shadowing it.

[Wez, 2016/09/12 00:06:44]

SGTM, although I think we should review this Blimp-specific code, since having
BlimpOutputSurface[Client] not actually be OutputSurface[Client] is super
confusing. CC'ing khushalsagar@ as an FYI on that.

 
   bool bound_to_client_;
 
   // This OutputSurface is responsible for providing the BeginFrameSource to
   // drive frame creation.  This will be built on the compositor impl thread at
   // BindToClient call time.
   std::unique_ptr<cc::BeginFrameSource> begin_frame_source_;
 
   base::WeakPtrFactory<DelegatedOutputSurface> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(DelegatedOutputSurface);
 };
 
 }  // namespace client
 }  // namespace blimp
 
 #endif  // BLIMP_CLIENT_CORE_COMPOSITOR_DELEGATED_OUTPUT_SURFACE_H_