Implement "hole" video frame.

content/browser/renderer_host/media/video_capture_controller.cc

 // Copyright (c) 2012 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_controller.h"
 
 #include <set>
 
 #include "base/bind.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 403 matching lines @@
           base::TimeDelta(),
           base::Bind(&base::DoNothing)));
 
   const int kYPlane = media::VideoFrame::kYPlane;
   const int kUPlane = media::VideoFrame::kUPlane;
   const int kVPlane = media::VideoFrame::kVPlane;
   const int kRGBPlane = media::VideoFrame::kRGBPlane;
 
   // Do color conversion from the camera format to I420.
   switch (frame->format()) {
+#if defined(GOOGLE_TV)
+    case media::VideoFrame::HOLE: {
+      NOTREACHED();

[scherkus (not reviewing), 2013/02/27 06:54:20]

do you need to duplicate the { NOTREACHED(); break; } block?

what about letting it fall through to the one below?

[wonsik2, 2013/02/27 10:19:29]

Done. Added a short comment for a little safeguard against possible mistakes
around ifdef's and switch fall-through.

+      break;
+    }
+#endif
     case media::VideoFrame::INVALID:
     case media::VideoFrame::YV16:
     case media::VideoFrame::EMPTY:
     case media::VideoFrame::NATIVE_TEXTURE: {
       NOTREACHED();
       break;
     }
     case media::VideoFrame::I420:
     case media::VideoFrame::YV12: {
       DCHECK(!chopped_width_ && !chopped_height_);
@@ skipping 265 matching lines @@
   base::AutoLock lock(lock_);
   for (DIBMap::iterator dib_it = owned_dibs_.begin();
        dib_it != owned_dibs_.end(); dib_it++) {
     if (dib_it->second->references > 0)
       return true;
   }
   return false;
 }
 
 }  // namespace content