Re-land: cc: Implement shared worker contexts.

cc/output/output_surface.h

 // Copyright 2012 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_OUTPUT_OUTPUT_SURFACE_H_
 #define CC_OUTPUT_OUTPUT_SURFACE_H_
 
 #include <deque>
 
 #include "base/basictypes.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/weak_ptr.h"
+#include "base/threading/thread_checker.h"
 #include "cc/base/cc_export.h"
 #include "cc/output/context_provider.h"
 #include "cc/output/overlay_candidate_validator.h"
 #include "cc/output/software_output_device.h"
 
 namespace base { class SingleThreadTaskRunner; }
 
 namespace ui {
 class LatencyInfo;
 }
@@ skipping 81 matching lines @@
   ContextProvider* context_provider() const { return context_provider_.get(); }
   ContextProvider* worker_context_provider() const {
     return worker_context_provider_.get();
   }
   SoftwareOutputDevice* software_device() const {
     return software_device_.get();
   }
 
   // Called by the compositor on the compositor thread. This is a place where
   // thread-specific data for the output surface can be initialized, since from
-  // this point on the output surface will only be used on the compositor
-  // thread.
+  // this point to when DetachFromClient() is called the output surface will
+  // only be used on the compositor thread.
   virtual bool BindToClient(OutputSurfaceClient* client);
+  // Called by the compositor on the compositor thread. This is a place where

[danakj, 2015/09/12 00:09:14]

nit: whitespace before the comment please

[reveman, 2015/09/12 12:49:57]

Done.

+  // thread-specific data for the output surface can be uninitialized.
+  virtual void DetachFromClient();
+
+  bool CalledOnValidClientThread();

[danakj, 2015/09/12 00:09:14]

Do we need a public method?

[reveman, 2015/09/12 12:49:57]

Good point. Removed.

 
   virtual void EnsureBackbuffer();
   virtual void DiscardBackbuffer();
 
   virtual void Reshape(const gfx::Size& size, float scale_factor);
   gfx::Size SurfaceSize() const { return surface_size_; }
   float device_scale_factor() const { return device_scale_factor_; }
 
   // If supported, this causes a ReclaimResources for all resources that are
   // currently in use.
@@ skipping 58 matching lines @@
   void SetExternalDrawConstraints(
       const gfx::Transform& transform,
       const gfx::Rect& viewport,
       const gfx::Rect& clip,
       const gfx::Rect& viewport_rect_for_tile_priority,
       const gfx::Transform& transform_for_tile_priority,
       bool resourceless_software_draw);
 
  private:
   bool external_stencil_test_enabled_;
+  base::ThreadChecker client_thread_checker_;

[danakj, 2015/09/12 00:09:14]

just make this protected?

[reveman, 2015/09/12 12:49:57]

Done.

 
   base::WeakPtrFactory<OutputSurface> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(OutputSurface);
 };
 
 }  // namespace cc
 
 #endif  // CC_OUTPUT_OUTPUT_SURFACE_H_