Remove dead code paths around PIXEL_STORAGE_TEXTURE in capture pipeline.

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/memory/scoped_vector.h"
 #include "base/stl_util.h"
@@ skipping 243 matching lines @@
   }
   NOTREACHED();
   return true;
 }
 
 // static
 scoped_ptr<VideoCaptureBufferPool::Tracker>
 VideoCaptureBufferPool::Tracker::CreateTracker(
     media::VideoPixelStorage storage) {
   DCHECK(storage == media::PIXEL_STORAGE_GPUMEMORYBUFFER ||
-         storage == media::PIXEL_STORAGE_CPU ||
-         storage == media::PIXEL_STORAGE_TEXTURE);
+         storage == media::PIXEL_STORAGE_CPU);

[mcasas, 2015/11/13 19:13:57]

Actually, these are all the cases now, remove this DCHECK()?

[miu, 2015/11/14 03:43:47]

Done.  Replaced with switch statement to ensure compiles will error out if the
enum is ever changed.

 
   if (storage == media::PIXEL_STORAGE_GPUMEMORYBUFFER)
     return make_scoped_ptr(new GpuMemoryBufferTracker());
   else
     return make_scoped_ptr(new SharedMemTracker());
 }
 
 VideoCaptureBufferPool::Tracker::~Tracker() {}
 
 VideoCaptureBufferPool::VideoCaptureBufferPool(int count)
@@ skipping 179 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