Adding Null check in onAuthRequired() for handling FTP requests

Adding Null check in onAuthRequired() for handling FTP requests

statusCode was implemented in:
https://code.google.com/p/chromium/issues/detail?id=447180

BUG=521296
Committed: https://crrev.com/1e571d6050004a98dc506f95f2faaa76f6045c74
Cr-Commit-Position: refs/heads/master@{#344433}

[shiva.jm, 2015-08-19 10:26:23]

shiva.jm@samsung.com changed reviewers:
+ battre@chromium.org, kalman@chromium.org, rockot@chromium.org

[shiva.jm, 2015-08-19 10:26:23]

pls have a look.

[not at google - send to devlin, 2015-08-19 14:43:25]

lgtm

https://codereview.chromium.org/1297503004/diff/1/extensions/browser/api/web_...
File extensions/browser/api/web_request/web_request_api.cc (right):

https://codereview.chromium.org/1297503004/diff/1/extensions/browser/api/web_...
extensions/browser/api/web_request/web_request_api.cc:1002:
request->response_headers()->response_code());
Make sure to run "git cl format" before submitting this CL.

[shiva.jm, 2015-08-20 04:07:37]

updated with comments. pls have a look.

https://codereview.chromium.org/1297503004/diff/1/extensions/browser/api/web_...
File extensions/browser/api/web_request/web_request_api.cc (right):

https://codereview.chromium.org/1297503004/diff/1/extensions/browser/api/web_...
extensions/browser/api/web_request/web_request_api.cc:1002:
request->response_headers()->response_code());
On 2015/08/19 14:43:25, kalman wrote:
> Make sure to run "git cl format" before submitting this CL.

Done.

[shiva.jm, 2015-08-20 04:07:52]

The CQ bit was checked by shiva.jm@samsung.com

[shiva.jm, 2015-08-20 04:07:52]

The patchset sent to the CQ was uploaded after l-g-t-m from kalman@chromium.org
Link to the patchset: https://codereview.chromium.org/1297503004/#ps20001
(title: " ")

[commit-bot: I haz the power, 2015-08-20 04:08:06]

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

[commit-bot: I haz the power, 2015-08-20 04:33:37]

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

[commit-bot: I haz the power, 2015-08-20 04:33:38]

Try jobs failed on following builders:
  chromeos_daisy_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)

[shiva.jm, 2015-08-20 05:50:43]

The CQ bit was checked by shiva.jm@samsung.com

[commit-bot: I haz the power, 2015-08-20 05:50:52]

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

[commit-bot: I haz the power, 2015-08-20 06:12:51]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-08-20 06:13:30]

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

[cbentzel, 2015-08-20 09:50:53]

On 2015/08/20 06:13:30, commit-bot: I haz the power wrote:
> Patchset 2 (id:??) landed as
> https://crrev.com/1e571d6050004a98dc506f95f2faaa76f6045c74
> Cr-Commit-Position: refs/heads/master@{#344433}

[cbentzel, 2015-08-20 09:50:54]

On 2015/08/20 06:13:30, commit-bot: I haz the power wrote:
> Patchset 2 (id:??) landed as
> https://crrev.com/1e571d6050004a98dc506f95f2faaa76f6045c74
> Cr-Commit-Position: refs/heads/master@{#344433}

[cbentzel, 2015-08-20 09:51:24]

On 2015/08/20 06:13:30, commit-bot: I haz the power wrote:
> Patchset 2 (id:??) landed as
> https://crrev.com/1e571d6050004a98dc506f95f2faaa76f6045c74
> Cr-Commit-Position: refs/heads/master@{#344433}

post-commit request: could you add a test which exercises this behavior?

[shiva.jm, 2015-08-27 11:17:54]

Patchset #3 (id:40001) has been deleted