Fix extension->id() nullptr access at TabCaptureAccessHandler::HandleRequest

Fix extension->id() nullptr access at TabCaptureAccessHandler::HandleRequest

There is a check for null pointer of |extension| parameter,
but it is then accessed via a call extension->id().

R=changbin.shao, dcheng
BUG=709421
Review-Url: https://codereview.chromium.org/2805213002
Cr-Commit-Position: refs/heads/master@{#463209}
Committed: https://chromium.googlesource.com/chromium/src/+/5d7042a87c4f1901ef1ed03d29f3e31e8af1724d

[dcheng, 2017-04-07 17:26:38]

dcheng@chromium.org changed reviewers:
+ sergeyu@chromium.org

[dcheng, 2017-04-07 17:26:38]

LGTM, but I am not an OWNER.

+sergeyu

[Sergey Ulanov, 2017-04-07 18:06:58]

lgtm

[pavor, 2017-04-10 09:07:06]

The CQ bit was checked by pavor@yandex-team.ru to run a CQ dry run

[commit-bot: I haz the power, 2017-04-10 09:07:27]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-10 09:50:31]

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

[commit-bot: I haz the power, 2017-04-10 09:50:32]

Dry run: This issue passed the CQ dry run.

[pavor, 2017-04-10 09:59:06]

The CQ bit was checked by pavor@yandex-team.ru

[commit-bot: I haz the power, 2017-04-10 09:59:23]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-10 10:04:04]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1491818346354410, "parent_rev":
"8c484a3fff11a18c722b1690e1090effd1978a97", "commit_rev":
"5d7042a87c4f1901ef1ed03d29f3e31e8af1724d"}

[commit-bot: I haz the power, 2017-04-10 10:04:55]

Description was changed from

==========
Fix extension->id() nullptr access at TabCaptureAccessHandler::HandleRequest

There is a check for null pointer of |extension| parameter,
but it is then accessed via a call extension->id().

R=changbin.shao, dcheng
BUG=709421
==========

to

==========
Fix extension->id() nullptr access at TabCaptureAccessHandler::HandleRequest

There is a check for null pointer of |extension| parameter,
but it is then accessed via a call extension->id().

R=changbin.shao, dcheng
BUG=709421

Review-Url: https://codereview.chromium.org/2805213002
Cr-Commit-Position: refs/heads/master@{#463209}
Committed:
https://chromium.googlesource.com/chromium/src/+/5d7042a87c4f1901ef1ed03d29f3...
==========

[commit-bot: I haz the power, 2017-04-10 10:04:55]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/5d7042a87c4f1901ef1ed03d29f3...