[Android] Filter the unknown type in function GetDeviceSupportedFormats

media/video/capture/android/video_capture_device_factory_android.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 "media/video/capture/android/video_capture_device_factory_android.h"
 
 #include "base/android/jni_string.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/stringprintf.h"
 #include "jni/VideoCaptureFactory_jni.h"
@@ skipping 98 matching lines @@
         env, format.obj())) {
       case VideoCaptureDeviceAndroid::ANDROID_IMAGE_FORMAT_YV12:
         pixel_format = media::PIXEL_FORMAT_YV12;
         break;
       case VideoCaptureDeviceAndroid::ANDROID_IMAGE_FORMAT_NV21:
         pixel_format = media::PIXEL_FORMAT_NV21;
         break;
       default:
         break;
     }
+    if (pixel_format == media::PIXEL_FORMAT_UNKNOWN)

[tommi (sloooow) - chröme, 2015/07/03 08:52:41]

could we continue on the default label of the switch()?

+      continue;
     VideoCaptureFormat capture_format(
         gfx::Size(media::Java_VideoCaptureFactory_getCaptureFormatWidth(env,
                       format.obj()),
                   media::Java_VideoCaptureFactory_getCaptureFormatHeight(env,
                       format.obj())),
         media::Java_VideoCaptureFactory_getCaptureFormatFramerate(env,
                                                                   format.obj()),
         pixel_format);
     capture_formats->push_back(capture_format);
     DVLOG(1) << device.name() << " "
              << VideoCaptureFormat::ToString(capture_format);
   }
 }
 
 // static
 VideoCaptureDeviceFactory*
 VideoCaptureDeviceFactory::CreateVideoCaptureDeviceFactory(
     scoped_refptr<base::SingleThreadTaskRunner> ui_task_runner) {
   return new VideoCaptureDeviceFactoryAndroid();
 }
 
 }  // namespace media