Temporarily disabling JSON generation for Zone statistics to resolve flakiness in tests until a bet…

Temporarily disabling JSON generation for Zone statistics to resolve flakiness in tests until a better implementation is written. Issue #28885 (https://github.com/dart-lang/sdk/issues/28885).

BUG=
R=zra@google.com

Committed: https://github.com/dart-lang/sdk/commit/6e02846b6da018bef753cd1eb8685ee577677682

[bkonyi, 2017-03-02 20:46:33]

bkonyi@google.com changed reviewers:
+ asiva@google.com, zra@google.com

[zra, 2017-03-02 21:02:11]

Should we just delete this instead of commenting it out? We still have the CL
adding it in here and in git.

[bkonyi, 2017-03-02 21:10:40]

On 2017/03/02 21:02:11, zra wrote:
> Should we just delete this instead of commenting it out? We still have the CL
> adding it in here and in git.

I guess we can, I just figured this would make life easier when this
functionality is re-enabled in one way or another. I'll remove it if that's what
you'd prefer.

[bkonyi, 2017-03-02 21:26:30]

On 2017/03/02 21:10:40, bkonyi wrote:
> On 2017/03/02 21:02:11, zra wrote:
> > Should we just delete this instead of commenting it out? We still have the
CL
> > adding it in here and in git.
> 
> I guess we can, I just figured this would make life easier when this
> functionality is re-enabled in one way or another. I'll remove it if that's
what
> you'd prefer.

I went ahead and removed the code for now. This is going to leave some dead code
temporarily until I have a chance to discuss what we want to do with zone
information in the Observatory next week.

[zra, 2017-03-02 22:09:12]

lgtm

Please add a related #<issue number> to the CL description, so we have a record
of this on the issue tracker.

Maybe also make a pass over the code you know is dead after this CL, and add a
TODO referencing that issue.

[bkonyi, 2017-03-03 18:40:52]

Description was changed from

==========
Temporarily disabling JSON generation for Zone statistics to resolve flakiness
in tests until a better implementation is written.

BUG=
==========

to

==========
Temporarily disabling JSON generation for Zone statistics to resolve flakiness
in tests until a better implementation is written. Issue #28885
(https://github.com/dart-lang/sdk/issues/28885).

BUG=
==========

[bkonyi, 2017-03-03 18:43:17]

Description was changed from

==========
Temporarily disabling JSON generation for Zone statistics to resolve flakiness
in tests until a better implementation is written. Issue #28885
(https://github.com/dart-lang/sdk/issues/28885).

BUG=
==========

to

==========
Temporarily disabling JSON generation for Zone statistics to resolve flakiness
in tests until a better implementation is written. Issue #28885
(https://github.com/dart-lang/sdk/issues/28885).

BUG=
R=zra@google.com

Committed:
https://github.com/dart-lang/sdk/commit/6e02846b6da018bef753cd1eb8685ee577677682
==========

[bkonyi, 2017-03-03 18:43:19]

Committed patchset #2 (id:20001) manually as
6e02846b6da018bef753cd1eb8685ee577677682 (presubmit successful).