Cleanup: Remove some LOG(ERROR) in ServiceWorkerContextWrapper

Cleanup: Remove some LOG(ERROR) in ServiceWorkerContextWrapper

We haven't found these useful and other code that bails due to no context
doesn't log anything.

Committed: https://crrev.com/69a8f6885cc7af0e4eec2914ed0bd07f8802b87c
Cr-Commit-Position: refs/heads/master@{#346862}

[falken, 2015-09-02 06:10:47]

falken@chromium.org changed reviewers:
+ nhiroki@chromium.org

[falken, 2015-09-02 06:10:47]

ptal (low priority)

[nhiroki, 2015-09-02 06:15:38]

Thank you for cleaning up! LGTM

https://codereview.chromium.org/1327763002/diff/1/content/browser/service_wor...
File content/browser/service_worker/service_worker_context_wrapper.cc (right):

https://codereview.chromium.org/1327763002/diff/1/content/browser/service_wor...
content/browser/service_worker/service_worker_context_wrapper.cc:197: if
(!context_core_.get()) {
nit: How about removing ".get()" like line 149?

https://codereview.chromium.org/1327763002/diff/1/content/browser/service_wor...
content/browser/service_worker/service_worker_context_wrapper.cc:290: if
(!context_core_.get()) {
nit: Can you remove {}?

[nhiroki, 2015-09-02 06:16:23]

nit: s/anthing/anything in CL description

[falken, 2015-09-02 06:28:09]

Thanks, fixed Cl desc

https://codereview.chromium.org/1327763002/diff/1/content/browser/service_wor...
File content/browser/service_worker/service_worker_context_wrapper.cc (right):

https://codereview.chromium.org/1327763002/diff/1/content/browser/service_wor...
content/browser/service_worker/service_worker_context_wrapper.cc:197: if
(!context_core_.get()) {
On 2015/09/02 06:15:38, nhiroki wrote:
> nit: How about removing ".get()" like line 149?

Done.

https://codereview.chromium.org/1327763002/diff/1/content/browser/service_wor...
content/browser/service_worker/service_worker_context_wrapper.cc:290: if
(!context_core_.get()) {
On 2015/09/02 06:15:38, nhiroki wrote:
> nit: Can you remove {}?

Done.

[falken, 2015-09-02 06:28:14]

The CQ bit was checked by falken@chromium.org

[falken, 2015-09-02 06:28:14]

The patchset sent to the CQ was uploaded after l-g-t-m from nhiroki@chromium.org
Link to the patchset: https://codereview.chromium.org/1327763002/#ps40001
(title: "and rm more .get()")

[commit-bot: I haz the power, 2015-09-02 06:28:19]

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

[commit-bot: I haz the power, 2015-09-02 07:27:56]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-09-02 07:28:34]

Patchset 3 (id:??) landed as
https://crrev.com/69a8f6885cc7af0e4eec2914ed0bd07f8802b87c
Cr-Commit-Position: refs/heads/master@{#346862}