base/threading: restrict to set only current thread priority

content/renderer/gpu/compositor_output_surface.h

 // Copyright (c) 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 CONTENT_RENDERER_GPU_COMPOSITOR_OUTPUT_SURFACE_H_
 #define CONTENT_RENDERER_GPU_COMPOSITOR_OUTPUT_SURFACE_H_
 
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
 #include "base/memory/ref_counted.h"
@@ skipping 36 matching lines @@
           worker_context_provider,
       scoped_ptr<cc::SoftwareOutputDevice> software,
       scoped_refptr<FrameSwapMessageQueue> swap_frame_message_queue,
       bool use_swap_compositor_frame_message);
   ~CompositorOutputSurface() override;
 
   // cc::OutputSurface implementation.
   bool BindToClient(cc::OutputSurfaceClient* client) override;
   void SwapBuffers(cc::CompositorFrame* frame) override;
 
+#if defined(OS_ANDROID)
   // TODO(epenner): This seems out of place here and would be a better fit
   // int CompositorThread after it is fully refactored (http://crbug/170828)
   void UpdateSmoothnessTakesPriority(bool prefer_smoothness) override;
+#endif
 
  protected:
   void ShortcutSwapAck(uint32 output_surface_id,
                        scoped_ptr<cc::GLFrameData> gl_frame_data,
                        scoped_ptr<cc::SoftwareFrameData> software_frame_data);
   virtual void OnSwapAck(uint32 output_surface_id,
                          const cc::CompositorFrameAck& ack);
   virtual void OnReclaimResources(uint32 output_surface_id,
                                   const cc::CompositorFrameAck& ack);
   uint32 output_surface_id_;
@@ skipping 25 matching lines @@
   bool Send(IPC::Message* message);
 
   bool use_swap_compositor_frame_message_;
 
   scoped_refptr<CompositorForwardingMessageFilter> output_surface_filter_;
   CompositorForwardingMessageFilter::Handler output_surface_filter_handler_;
   scoped_refptr<CompositorOutputSurfaceProxy> output_surface_proxy_;
   scoped_refptr<IPC::SyncMessageFilter> message_sender_;
   scoped_refptr<FrameSwapMessageQueue> frame_swap_message_queue_;
   int routing_id_;
+#if defined(OS_ANDROID)
   bool prefers_smoothness_;
-  base::PlatformThreadHandle main_thread_handle_;
+  scoped_refptr<base::SingleThreadTaskRunner> main_thread_runner_;

[gab, 2015/06/23 17:01:32]

Please keep logic changes to another CL here as well.

[Ken Russell (switch to Gerrit), 2015/06/23 21:08:40]

I agree with gab's assessment.

[Takashi Toyoshima, 2015/06/24 04:15:39]

Done.

+#endif
 
   // TODO(danakj): Remove this when crbug.com/311404
   bool layout_test_mode_;
   scoped_ptr<cc::CompositorFrameAck> layout_test_previous_frame_ack_;
   base::WeakPtrFactory<CompositorOutputSurface> weak_ptrs_;
 };
 
 }  // namespace content
 
 #endif  // CONTENT_RENDERER_GPU_COMPOSITOR_OUTPUT_SURFACE_H_