Verify that ComponentExtensionResourceManager exists before using it.

Verify that ComponentExtensionResourceManager exists before using it.

App_shell doesn't have a ComponentExtensionResourceManager implementation, so we need to check if it's available before using it.

BUG=352293
Committed: https://crrev.com/4f8f50d244424ca4fe8765e8727581bd8dbdb23e
Cr-Commit-Position: refs/heads/master@{#297299}

[lfg, 2014-09-29 21:53:12]

lfg@chromium.org changed reviewers:
+ rockot@chromium.org

[Ken Rockot(use gerrit already), 2014-09-29 21:56:33]

lgtm

https://codereview.chromium.org/615703002/diff/1/extensions/browser/api/execu...
File extensions/browser/api/execute_code_function.cc (right):

https://codereview.chromium.org/615703002/diff/1/extensions/browser/api/execu...
extensions/browser/api/execute_code_function.cc:209: if
(ExtensionsBrowserClient::Get()->GetComponentExtensionResourceManager() &&
nit: Retain the value of
ExtensionsBrowserClient::Get()->GetComponentExtensionResourceManager() first and
then test (foo && foo->IsEtcEtc..) instead of this verbose expression.

[lfg, 2014-09-29 22:01:25]

The CQ bit was checked by lfg@chromium.org

[lfg, 2014-09-29 22:02:23]

The CQ bit was unchecked by lfg@chromium.org

[lfg, 2014-09-29 22:12:44]

https://codereview.chromium.org/615703002/diff/1/extensions/browser/api/execu...
File extensions/browser/api/execute_code_function.cc (right):

https://codereview.chromium.org/615703002/diff/1/extensions/browser/api/execu...
extensions/browser/api/execute_code_function.cc:209: if
(ExtensionsBrowserClient::Get()->GetComponentExtensionResourceManager() &&
On 2014/09/29 21:56:32, Ken Rockot wrote:
> nit: Retain the value of
> ExtensionsBrowserClient::Get()->GetComponentExtensionResourceManager() first
and
> then test (foo && foo->IsEtcEtc..) instead of this verbose expression.

Done.

[lfg, 2014-09-29 22:54:28]

The CQ bit was checked by lfg@chromium.org

[commit-bot: I haz the power, 2014-09-29 22:56:15]

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

[commit-bot: I haz the power, 2014-09-29 23:20:19]

Committed patchset #2 (id:20001) as f4a4b009bee9c5b6f02bd4f0a533be8aaccc5028

[commit-bot: I haz the power, 2014-09-29 23:21:42]

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