Mac: Clean up ifdefs in output surface

content/browser/compositor/browser_compositor_output_surface.h

 // 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.
 
 #ifndef CONTENT_BROWSER_COMPOSITOR_BROWSER_COMPOSITOR_OUTPUT_SURFACE_H_
 #define CONTENT_BROWSER_COMPOSITOR_BROWSER_COMPOSITOR_OUTPUT_SURFACE_H_
 
 #include "base/macros.h"
 #include "base/threading/non_thread_safe.h"
 #include "build/build_config.h"
@@ skipping 53 matching lines @@
   // The argument |params_mac| is used to communicate parameters needed on Mac
   // to display the CALayer for the swap in the browser process.
   // TODO(ccameron): Remove |params_mac| when the CALayer tree is hosted in the
   // browser process.
   virtual void OnGpuSwapBuffersCompleted(
       const std::vector<ui::LatencyInfo>& latency_info,
       gfx::SwapResult result,
       const gpu::GpuProcessHostedCALayerTreeParamsMac* params_mac) = 0;
 
 #if defined(OS_MACOSX)
-  virtual void SetSurfaceSuspendedForRecycle(bool suspended) = 0;
+  virtual void SetSurfaceSuspendedForRecycle(bool suspended);

[danakj, 2016/05/26 21:33:51]

why not abstract?

[ccameron, 2016/05/26 22:46:16]

I wanted to get rid of all of the #if defined(OS_MACOSX) in
GpuBrowserCompositorOutputSurface. I changed this back.

 #endif
 
   cc::SyntheticBeginFrameSource* begin_frame_source() {
     return synthetic_begin_frame_source_.get();
   }
 
  protected:
   // Constructor used by the accelerated implementation.
   BrowserCompositorOutputSurface(
       scoped_refptr<cc::ContextProvider> context,
@@ skipping 29 matching lines @@
 
   std::unique_ptr<display_compositor::CompositorOverlayCandidateValidator>
       overlay_candidate_validator_;
 
   DISALLOW_COPY_AND_ASSIGN(BrowserCompositorOutputSurface);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_COMPOSITOR_BROWSER_COMPOSITOR_OUTPUT_SURFACE_H_