cc: Make OutputSurface::BindToClient pure virtual and not return bool

services/ui/surfaces/direct_output_surface.cc

Index: services/ui/surfaces/direct_output_surface.cc
diff --git a/services/ui/surfaces/direct_output_surface.cc b/services/ui/surfaces/direct_output_surface.cc
index 02274dfc8967a6fadd16fe2f3affb44eb4fe4f42..d3adce0f1ddc2a57940f26e638582b36f4539408 100644
--- a/services/ui/surfaces/direct_output_surface.cc
+++ b/services/ui/surfaces/direct_output_surface.cc
@@ -23,20 +23,17 @@ DirectOutputSurface::DirectOutputSurface(
     : cc::OutputSurface(context_provider),
       synthetic_begin_frame_source_(synthetic_begin_frame_source),
       weak_ptr_factory_(this) {
+  capabilities_.flipped_output_surface =
+      context_provider->ContextCapabilities().flips_vertically;
   context_provider->SetDelegate(this);
 }
 
-DirectOutputSurface::~DirectOutputSurface() = default;
+DirectOutputSurface::~DirectOutputSurface() {}
 
-bool DirectOutputSurface::BindToClient(cc::OutputSurfaceClient* client) {
-  if (!cc::OutputSurface::BindToClient(client))
-    return false;
-
-  if (capabilities_.uses_default_gl_framebuffer) {
-    capabilities_.flipped_output_surface =
-        context_provider()->ContextCapabilities().flips_vertically;
-  }
-  return true;
+void DirectOutputSurface::BindToClient(cc::OutputSurfaceClient* client) {
+  DCHECK(client);
+  DCHECK(!client_);
+  client_ = client;
 }
 
 void DirectOutputSurface::EnsureBackbuffer() {}