Fix a crash when cameraInfo is NULL

Fix a crash when cameraInfo is NULL

It is possible that cameraInfo returns NULL.
However, the NULL case is not well handled and it will cause crashes.
This change also removed the unncessary JNI call to convert an int to string

BUG=424775
Committed: https://crrev.com/f3c57f9b4bb6c0e899a80737244ba1783b6691d2
Cr-Commit-Position: refs/heads/master@{#300578}

[qinmin, 2014-10-17 22:54:58]

qinmin@chromium.org changed reviewers:
+ mcasas@chromium.org

[qinmin, 2014-10-17 22:54:58]

PTAL

[mcasas, 2014-10-20 11:33:42]

LGTM with a nit.

https://codereview.chromium.org/667433003/diff/1/media/video/capture/android/...
File media/video/capture/android/video_capture_device_factory_android.cc
(right):

https://codereview.chromium.org/667433003/diff/1/media/video/capture/android/...
media/video/capture/android/video_capture_device_factory_android.cc:75:
base::android::ConvertJavaStringToUTF8(device_name), device_id);
nit: I would use here base::IntToString(camera_id) 
instead of creating a |device_id| explicitly.

[qinmin, 2014-10-20 17:46:23]

https://codereview.chromium.org/667433003/diff/1/media/video/capture/android/...
File media/video/capture/android/video_capture_device_factory_android.cc
(right):

https://codereview.chromium.org/667433003/diff/1/media/video/capture/android/...
media/video/capture/android/video_capture_device_factory_android.cc:75:
base::android::ConvertJavaStringToUTF8(device_name), device_id);
On 2014/10/20 11:33:42, mcasas wrote:
> nit: I would use here base::IntToString(camera_id) 
> instead of creating a |device_id| explicitly.

Done.

[qinmin, 2014-10-20 17:46:44]

The CQ bit was checked by qinmin@chromium.org

[commit-bot: I haz the power, 2014-10-20 17:48:37]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/667433003/20001

[commit-bot: I haz the power, 2014-10-20 17:56:36]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-10-20 17:56:37]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[qinmin, 2014-10-21 19:02:18]

The CQ bit was checked by qinmin@chromium.org

[commit-bot: I haz the power, 2014-10-21 19:03:51]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/667433003/20001

[commit-bot: I haz the power, 2014-10-21 19:12:51]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-10-21 19:12:52]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[qinmin, 2014-10-22 00:00:48]

The CQ bit was checked by qinmin@chromium.org

[commit-bot: I haz the power, 2014-10-22 00:05:19]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/667433003/20001

[commit-bot: I haz the power, 2014-10-22 00:12:03]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-10-22 00:13:14]

Patchset 2 (id:??) landed as
https://crrev.com/f3c57f9b4bb6c0e899a80737244ba1783b6691d2
Cr-Commit-Position: refs/heads/master@{#300578}