Rework FlushSync to return early if commands have been processed since the last update

gpu/command_buffer/service/gpu_scheduler.h

 // Copyright (c) 2011 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 GPU_COMMAND_BUFFER_SERVICE_GPU_SCHEDULER_H_
 #define GPU_COMMAND_BUFFER_SERVICE_GPU_SCHEDULER_H_
 
 #include <queue>
 #include <vector>
 
@@ skipping 43 matching lines @@
                   const gfx::Size& size,
                   const gles2::DisallowedExtensions& disallowed_extensions,
                   const char* allowed_extensions,
                   const std::vector<int32>& attribs,
                   GpuScheduler* parent,
                   uint32 parent_texture_id);
 
   void Destroy();
   void DestroyCommon();
 
-  virtual void ProcessCommands();
+  void PutChanged(bool sync);
 
   // Helper which causes a call to ProcessCommands to be scheduled later.
   void ScheduleProcessCommands();
 
   // Implementation of CommandBufferEngine.
   virtual Buffer GetSharedMemoryBuffer(int32 shm_id);
   virtual void set_token(int32 token);
   virtual bool SetGetOffset(int32 offset);
   virtual int32 GetGetOffset();
 
@@ skipping 23 matching lines @@
   // renderer and GPU processes.
   uint64 swap_buffers_count() const;
   void set_acknowledged_swap_buffers_count(
       uint64 acknowledged_swap_buffers_count);
 
   void DidDestroySurface();
 #endif
 
   // Sets a callback that is called when a glResizeCHROMIUM command
   // is processed.
-  virtual void SetResizeCallback(Callback1<gfx::Size>::Type* callback);
+  void SetResizeCallback(Callback1<gfx::Size>::Type* callback);
 
   // Sets a callback which is called when a SwapBuffers command is processed.
   // Must be called after Initialize().
   // It is not defined on which thread this callback is called.
-  virtual void SetSwapBuffersCallback(Callback0::Type* callback);
+  void SetSwapBuffersCallback(Callback0::Type* callback);
 
-  virtual void SetCommandProcessedCallback(Callback0::Type* callback);
+  void SetCommandProcessedCallback(Callback0::Type* callback);
 
   // Sets a callback which is called after a Set/WaitLatch command is processed.
   // The bool parameter will be true for SetLatch, and false for a WaitLatch
   // that is blocked. An unblocked WaitLatch will not trigger a callback.
   void SetLatchCallback(const base::Callback<void(bool)>& callback) {
     decoder_->SetLatchCallback(callback);
   }
 
   // Get the GLES2Decoder associated with this scheduler.
   gles2::GLES2Decoder* decoder() const { return decoder_.get(); }
 
  protected:
   // Perform common initialization. Takes ownership of GLContext.
   bool InitializeCommon(
       gfx::GLContext* context,
       const gfx::Size& size,
       const gles2::DisallowedExtensions& disallowed_extensions,
       const char* allowed_extensions,
       const std::vector<int32>& attribs,
       gles2::GLES2Decoder* parent_decoder,
       uint32 parent_texture_id);
 
 
  private:
   // Called via a callback just before we are supposed to call the
   // user's swap buffers callback.
-  virtual void WillSwapBuffers();
+  void WillSwapBuffers();
+  void ProcessCommands();
 
   // The GpuScheduler holds a weak reference to the CommandBuffer. The
   // CommandBuffer owns the GpuScheduler and holds a strong reference to it
   // through the ProcessCommands callback.
   CommandBuffer* command_buffer_;
 
   int commands_per_update_;
 
   scoped_ptr<gles2::GLES2Decoder> decoder_;
   scoped_ptr<CommandParser> parser_;
 
 #if defined(OS_MACOSX)
   scoped_ptr<AcceleratedSurface> surface_;
   uint64 swap_buffers_count_;
   uint64 acknowledged_swap_buffers_count_;
 #endif
 
   ScopedRunnableMethodFactory<GpuScheduler> method_factory_;
   scoped_ptr<Callback0::Type> wrapped_swap_buffers_callback_;
   scoped_ptr<Callback0::Type> command_processed_callback_;
 };
 
 }  // namespace gpu
 
 #endif  // GPU_COMMAND_BUFFER_SERVICE_GPU_SCHEDULER_H_