Tab/Desktop Capture: Use requests instead of timer-based refreshing.

content/browser/renderer_host/media/video_capture_buffer_pool.cc

 // Copyright (c) 2013 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/browser/renderer_host/media/video_capture_buffer_pool.h"
 
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/stl_util.h"
 #include "build/build_config.h"
@@ skipping 356 matching lines @@
   // potentially modify the content). Check that the expected dimensions,
   // format, and storage match.
   TrackerMap::iterator it = trackers_.find(last_relinquished_buffer_id_);
   DCHECK(it != trackers_.end());
   DCHECK(!it->second->held_by_producer());
   if (it->second->consumer_hold_count() == 0 &&
       it->second->dimensions() == dimensions &&
       it->second->pixel_format() == format &&
       it->second->storage_type() == storage) {
     it->second->set_held_by_producer(true);
-    return last_relinquished_buffer_id_;
+    const int resurrected_buffer_id = last_relinquished_buffer_id_;
+    last_relinquished_buffer_id_ = kInvalidId;
+    return resurrected_buffer_id;
   }
 
   return kInvalidId;
 }
 
 double VideoCaptureBufferPool::GetBufferPoolUtilization() const {
   base::AutoLock lock(lock_);
   int num_buffers_held = 0;
   for (const auto& entry : trackers_) {
     Tracker* const tracker = entry.second;
@@ skipping 81 matching lines @@
   return buffer_id;
 }
 
 VideoCaptureBufferPool::Tracker* VideoCaptureBufferPool::GetTracker(
     int buffer_id) {
   TrackerMap::const_iterator it = trackers_.find(buffer_id);
   return (it == trackers_.end()) ? NULL : it->second;
 }
 
 }  // namespace content