SetUploadData now throws an exception if contentType is null.

SetUploadData now throws an exception if contentType is null.
It used to crash in nativeSetUploadData instead.

BUG=409151
Committed: https://crrev.com/b459f829122c050a2c711099cc76610170edc5ab
Cr-Commit-Position: refs/heads/master@{#292729}

[mef, 2014-08-29 20:30:06]

mef@chromium.org changed reviewers:
+ clm@google.com, mmenke@chromium.org, xunjieli@chromium.org

[mmenke, 2014-08-29 20:34:05]

On 2014/08/29 20:30:54, mef wrote:

LGTM

[xunjieli, 2014-08-29 20:41:18]

lgtm. Thanks for keeping me in the loop.

https://codereview.chromium.org/526503002/diff/1/components/cronet/android/sa...
File
components/cronet/android/sample/javatests/src/org/chromium/cronet_sample_apk/CronetSampleUrlTest.java
(right):

https://codereview.chromium.org/526503002/diff/1/components/cronet/android/sa...
components/cronet/android/sample/javatests/src/org/chromium/cronet_sample_apk/CronetSampleUrlTest.java:156:
// Create request with null listener to trigger an exception.
This is not a null listener. Probably the comment is outdated.

[mef, 2014-08-29 20:46:12]

thanks!

https://codereview.chromium.org/526503002/diff/1/components/cronet/android/sa...
File
components/cronet/android/sample/javatests/src/org/chromium/cronet_sample_apk/CronetSampleUrlTest.java
(right):

https://codereview.chromium.org/526503002/diff/1/components/cronet/android/sa...
components/cronet/android/sample/javatests/src/org/chromium/cronet_sample_apk/CronetSampleUrlTest.java:156:
// Create request with null listener to trigger an exception.
On 2014/08/29 20:41:18, xunjieli1 wrote:
> This is not a null listener. Probably the comment is outdated.

Done.

[mef, 2014-08-29 20:46:21]

The CQ bit was checked by mef@chromium.org

[commit-bot: I haz the power, 2014-08-30 00:14:38]

Committed patchset #2 (id:20001) as 199613305f2f5146ed1a9390a225d269b69fdddd

[commit-bot: I haz the power, 2014-09-10 03:12:11]

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