add default constructor to PendingTask; compilation fix for VS2012

base/pending_task.cc

 // 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.
 
 #include "base/pending_task.h"
 
 #include "base/tracked_objects.h"
 
 namespace base {
 
+PendingTask::PendingTask() {
+}

[Ryan Sleevi, 2012/10/08 23:08:42]

drive by: Isn't sequence_num uninitialized because you're providing a
user-defined constructor, even though it's POD? Should you be initializing it to
zero, "just in case"?

As for the default ctor - is that just because the lack of a move ctor?

+
 PendingTask::PendingTask(const tracked_objects::Location& posted_from,
                          const base::Closure& task)
     : base::TrackingInfo(posted_from, TimeTicks()),
       task(task),
       posted_from(posted_from),
       sequence_num(0),
       nestable(true) {
 }
 
 PendingTask::PendingTask(const tracked_objects::Location& posted_from,
@@ skipping 24 matching lines @@
   // If the times happen to match, then we use the sequence number to decide.
   // Compare the difference to support integer roll-over.
   return (sequence_num - other.sequence_num) > 0;
 }
 
 void TaskQueue::Swap(TaskQueue* queue) {
   c.swap(queue->c);  // Calls std::deque::swap.
 }
 
 }  // namespace base