Remove duplicate MoveReadyDelayedTasksToIncomingQueueLocked

content/child/scheduler/task_queue_manager.cc

 // Copyright 2014 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 "content/child/scheduler/task_queue_manager.h"
 
 #include <queue>
 #include <set>
 
 #include "base/bind.h"
@@ skipping 285 matching lines @@
   return true;
 }
 
 bool TaskQueue::UpdateWorkQueue(LazyNow* lazy_now,
                                 const base::PendingTask* previous_task) {
   if (!work_queue_.empty())
     return true;
 
   {
     base::AutoLock lock(lock_);
     MoveReadyDelayedTasksToIncomingQueueLocked(lazy_now);

[Sami, 2015/04/07 12:41:30]

Should we do this after the pump check instead? That way we don't do any extra
work for non-auto queues.

[alex clarke (OOO till 29th), 2015/04/08 10:00:30]

Done.

     if (!ShouldAutoPumpQueueLocked(previous_task))
       return false;
-    MoveReadyDelayedTasksToIncomingQueueLocked(lazy_now);
     work_queue_.Swap(&incoming_queue_);
     TraceQueueSize(true);
     return true;
   }
 }
 
 base::PendingTask TaskQueue::TakeTaskFromWorkQueue() {
   base::PendingTask pending_task = work_queue_.front();
   work_queue_.pop();
   TraceQueueSize(false);
@@ skipping 385 matching lines @@
   state->EndArray();
   state->BeginDictionary("selector");
   selector_->AsValueInto(state.get());
   state->EndDictionary();
   if (should_run)
     state->SetInteger("selected_queue", selected_queue);
   return state;
 }
 
 }  // namespace content