[wasm][asm.js] Print a deterministic success result by default.

[wasm][asm.js] Print a deterministic success result by default.

Printing an asm.js success message and timings is useful,
but also non-deterministic. Making the message stable unless a flag is passed.
This will avoid making it a hassle in the future to create LayoutTests
that use asm.js and verify console output.

BUG=v8:4203
R=titzer@chromium.org

Committed: https://crrev.com/5124589642ba12228dcd66a8cb8c84c986a13f35
Cr-Commit-Position: refs/heads/master@{#41700}

[bradnelson, 2016-12-14 17:31:26]

The CQ bit was checked by bradnelson@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-14 17:31:27]

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

[bradn, 2016-12-14 17:31:38]

bradnelson@google.com changed reviewers:
+ bradnelson@google.com

[titzer, 2016-12-14 17:33:09]

lgtm with nit

https://codereview.chromium.org/2574273002/diff/1/src/flag-definitions.h
File src/flag-definitions.h (right):

https://codereview.chromium.org/2574273002/diff/1/src/flag-definitions.h#newc...
src/flag-definitions.h:509: DEFINE_BOOL(trace_asm_timing, false, "log asm.js
timing to the console")
s/timing/time/g to be consistent with previously existing flags.

[bradn, 2016-12-14 17:42:38]

https://codereview.chromium.org/2574273002/diff/1/src/flag-definitions.h
File src/flag-definitions.h (right):

https://codereview.chromium.org/2574273002/diff/1/src/flag-definitions.h#newc...
src/flag-definitions.h:509: DEFINE_BOOL(trace_asm_timing, false, "log asm.js
timing to the console")
On 2016/12/14 17:33:08, titzer wrote:
> s/timing/time/g to be consistent with previously existing flags.

Done.

[bradn, 2016-12-14 17:42:41]

The CQ bit was checked by bradnelson@google.com

[bradn, 2016-12-14 17:42:41]

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

[commit-bot: I haz the power, 2016-12-14 17:42:45]

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-12-14 18:12:05]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1481737361599530,
"parent_rev": "90cd2c1e82994d9ac9aeada3868d73847fc9a2c3", "commit_rev":
"6c3ad266147372fa6f29a49ee99ddfb809f28f04"}

[commit-bot: I haz the power, 2016-12-14 18:12:12]

Description was changed from

==========
[wasm][asm.js] Print a deterministic success result by default.

Printing an asm.js success message and timings is useful,
but also non-deterministic. Making the message stable unless a flag is passed.
This will avoid making it a hassle in the future to create LayoutTests
that use asm.js and verify console output.

BUG=v8:4203
R=titzer@chromium.org
==========

to

==========
[wasm][asm.js] Print a deterministic success result by default.

Printing an asm.js success message and timings is useful,
but also non-deterministic. Making the message stable unless a flag is passed.
This will avoid making it a hassle in the future to create LayoutTests
that use asm.js and verify console output.

BUG=v8:4203
R=titzer@chromium.org

Review-Url: https://codereview.chromium.org/2574273002
==========

[commit-bot: I haz the power, 2016-12-14 18:12:13]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-14 18:13:32]

Description was changed from

==========
[wasm][asm.js] Print a deterministic success result by default.

Printing an asm.js success message and timings is useful,
but also non-deterministic. Making the message stable unless a flag is passed.
This will avoid making it a hassle in the future to create LayoutTests
that use asm.js and verify console output.

BUG=v8:4203
R=titzer@chromium.org

Review-Url: https://codereview.chromium.org/2574273002
==========

to

==========
[wasm][asm.js] Print a deterministic success result by default.

Printing an asm.js success message and timings is useful,
but also non-deterministic. Making the message stable unless a flag is passed.
This will avoid making it a hassle in the future to create LayoutTests
that use asm.js and verify console output.

BUG=v8:4203
R=titzer@chromium.org

Committed: https://crrev.com/5124589642ba12228dcd66a8cb8c84c986a13f35
Cr-Commit-Position: refs/heads/master@{#41700}
==========

[commit-bot: I haz the power, 2016-12-14 18:13:33]

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