Move media library AutoTaskRunner to base and rename ScopedTaskRunner.

remoting/client/rectangle_update_decoder.cc

 // Copyright (c) 2010 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.
 
 #include "remoting/client/rectangle_update_decoder.h"
 
 #include "base/logging.h"
 #include "base/message_loop.h"
-#include "media/base/callback.h"
 #include "remoting/base/decoder.h"
 #include "remoting/base/decoder_row_based.h"
 #include "remoting/base/decoder_vp8.h"
 #include "remoting/base/tracer.h"
 #include "remoting/base/util.h"
 #include "remoting/client/frame_consumer.h"
 #include "remoting/protocol/session_config.h"
 
-using media::AutoTaskRunner;
-using remoting::protocol::ChannelConfig;
-using remoting::protocol::SessionConfig;
+using ::remoting::protocol::ChannelConfig;

[brettw, 2011/05/26 17:17:22]

Are the leading colons here necessary? Usually we don't use them in cases like
this.

[Wez, 2011/05/26 18:48:21]

Reverted.

The C++ Style Guide implied it was preferred that they be there, but perhaps I
mis-read it.

+using ::remoting::protocol::SessionConfig;
 
 namespace remoting {
 
 namespace {
 
 class PartialFrameCleanup : public Task {
  public:
   PartialFrameCleanup(media::VideoFrame* frame, UpdatedRects* rects)
       : frame_(frame), rects_(rects) {
   }
@@ skipping 46 matching lines @@
 void RectangleUpdateDecoder::DecodePacket(const VideoPacket* packet,
                                           Task* done) {
   if (message_loop_ != MessageLoop::current()) {
     message_loop_->PostTask(
         FROM_HERE,
         NewTracedMethod(this,
                         &RectangleUpdateDecoder::DecodePacket, packet,
                         done));
     return;
   }
-  AutoTaskRunner done_runner(done);
+  ScopedTaskRunner done_runner(done);
 
   TraceContext::tracer()->PrintString("Decode Packet called.");
 
   AllocateFrame(packet, done_runner.release());
 }
 
 void RectangleUpdateDecoder::AllocateFrame(const VideoPacket* packet,
                                            Task* done) {
   if (message_loop_ != MessageLoop::current()) {
     message_loop_->PostTask(
         FROM_HERE,
         NewTracedMethod(this,
                         &RectangleUpdateDecoder::AllocateFrame, packet, done));
     return;
   }
-  AutoTaskRunner done_runner(done);
+  ScopedTaskRunner done_runner(done);
 
   TraceContext::tracer()->PrintString("AllocateFrame called.");
 
   // Find the required frame size.
   bool has_screen_size = packet->format().has_screen_width() &&
                          packet->format().has_screen_height();
   gfx::Size screen_size(packet->format().screen_width(),
                         packet->format().screen_height());
   if (!has_screen_size)
     screen_size = initial_screen_size_;
@@ skipping 29 matching lines @@
 void RectangleUpdateDecoder::ProcessPacketData(
     const VideoPacket* packet, Task* done) {
   if (message_loop_ != MessageLoop::current()) {
     message_loop_->PostTask(
         FROM_HERE,
         NewTracedMethod(this,
                         &RectangleUpdateDecoder::ProcessPacketData, packet,
                         done));
     return;
   }
-  AutoTaskRunner done_runner(done);
+  ScopedTaskRunner done_runner(done);
 
   if (frame_is_new_) {
     decoder_->Reset();
     decoder_->Initialize(frame_);
     frame_is_new_ = false;
   }
 
   if (!decoder_->IsReadyForData()) {
     // TODO(ajwong): This whole thing should move into an invalid state.
     LOG(ERROR) << "Decoder is unable to process data. Dropping packet.";
     return;
   }
 
   TraceContext::tracer()->PrintString("Executing Decode.");
 
   if (decoder_->DecodePacket(packet) == Decoder::DECODE_DONE) {
     UpdatedRects* rects = new UpdatedRects();
     decoder_->GetUpdatedRects(rects);
     consumer_->OnPartialFrameOutput(frame_, rects,
                                     new PartialFrameCleanup(frame_, rects));
   }
 }
 
 }  // namespace remoting