ServiceWorker: Re-enable FetchEvent_respondWithRejection

ServiceWorker: Re-enable FetchEvent_respondWithRejection

Memory sanitizer shows an error of uninitialized read in
service_worker_browsertest.cc. This patch adds a check if
WaitForConsoleMessages has been already called, so the variable will be read
only after WaitForConsoleMessages initializes it.

BUG=635599
TEST=./out/msan/content_browsertests --no-sandbox --gtest_filter="ServiceWorkerVersionBrowserTest.FetchEvent_respondWithRejection"

Committed: https://crrev.com/4d4c3711a48ca7006ecf7e0a860994eaa8fd006e
Cr-Commit-Position: refs/heads/master@{#410584}

[shimazu, 2016-08-09 03:16:25]

Description was changed from

==========
ServiceWorker: Re-enable FetchEvent_respondWithRejection

Memory sanitizer shows an error of uninitialized read in
service_worker_browsertest.cc. This patch adds a check if
WaitForConsoleMessages has been already called, and the variable will be read
only when WaitForConsoleMessages initializes it.

BUG=635599
TEST=./out/msan/content_browsertests --no-sandbox
--gtest_filter="ServiceWorkerVersionBrowserTest.FetchEvent_respondWithRejection"
==========

to

==========
ServiceWorker: Re-enable FetchEvent_respondWithRejection

Memory sanitizer shows an error of uninitialized read in
service_worker_browsertest.cc. This patch adds a check if
WaitForConsoleMessages has been already called, so the variable will be read
only after WaitForConsoleMessages initializes it.

BUG=635599
TEST=./out/msan/content_browsertests --no-sandbox
--gtest_filter="ServiceWorkerVersionBrowserTest.FetchEvent_respondWithRejection"
==========

[shimazu, 2016-08-09 03:16:46]

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

[shimazu, 2016-08-09 03:16:46]

ptal:)

[nhiroki, 2016-08-09 04:08:48]

lgtm

[shimazu, 2016-08-09 04:15:25]

The CQ bit was checked by shimazu@chromium.org

[commit-bot: I haz the power, 2016-08-09 04:15:49]

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, 2016-08-09 05:26:30]

Description was changed from

==========
ServiceWorker: Re-enable FetchEvent_respondWithRejection

Memory sanitizer shows an error of uninitialized read in
service_worker_browsertest.cc. This patch adds a check if
WaitForConsoleMessages has been already called, so the variable will be read
only after WaitForConsoleMessages initializes it.

BUG=635599
TEST=./out/msan/content_browsertests --no-sandbox
--gtest_filter="ServiceWorkerVersionBrowserTest.FetchEvent_respondWithRejection"
==========

to

==========
ServiceWorker: Re-enable FetchEvent_respondWithRejection

Memory sanitizer shows an error of uninitialized read in
service_worker_browsertest.cc. This patch adds a check if
WaitForConsoleMessages has been already called, so the variable will be read
only after WaitForConsoleMessages initializes it.

BUG=635599
TEST=./out/msan/content_browsertests --no-sandbox
--gtest_filter="ServiceWorkerVersionBrowserTest.FetchEvent_respondWithRejection"
==========

[commit-bot: I haz the power, 2016-08-09 05:26:31]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-09 05:28:27]

Description was changed from

==========
ServiceWorker: Re-enable FetchEvent_respondWithRejection

Memory sanitizer shows an error of uninitialized read in
service_worker_browsertest.cc. This patch adds a check if
WaitForConsoleMessages has been already called, so the variable will be read
only after WaitForConsoleMessages initializes it.

BUG=635599
TEST=./out/msan/content_browsertests --no-sandbox
--gtest_filter="ServiceWorkerVersionBrowserTest.FetchEvent_respondWithRejection"
==========

to

==========
ServiceWorker: Re-enable FetchEvent_respondWithRejection

Memory sanitizer shows an error of uninitialized read in
service_worker_browsertest.cc. This patch adds a check if
WaitForConsoleMessages has been already called, so the variable will be read
only after WaitForConsoleMessages initializes it.

BUG=635599
TEST=./out/msan/content_browsertests --no-sandbox
--gtest_filter="ServiceWorkerVersionBrowserTest.FetchEvent_respondWithRejection"

Committed: https://crrev.com/4d4c3711a48ca7006ecf7e0a860994eaa8fd006e
Cr-Commit-Position: refs/heads/master@{#410584}
==========

[commit-bot: I haz the power, 2016-08-09 05:28:28]

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