Revert of [Cronet] Write effective experimental options to NetLog

Revert of [Cronet] Write effective experimental options to NetLog (patchset #5 id:80001 of https://codereview.chromium.org/2738813004/ )

Reason for revert:
The bot

Android Cronet Builder Asan

is reporting

Failed steps failed cronet_unittests failed cronet_test_instrumentation_apk

I suspect this patch might be why.


https://uberchromegw.corp.google.com/i/chromium.android/builders/Android%20Cronet%20Builder%20Asan/builds/1791

Original issue's description:
> [Cronet] Write effective experimental options to NetLog
> 
> This CL does the following:
> (1) Have a warning in Experimental Options parsing code if there is an
>     unrecognized option.
> (2) Make cronet_url_request_context_adapter use the new FileNetLogObserver which
>     allows writing polled data.
> (3) Put the effective experimental options in the polled data.
> 
> BUG=584806, 699567
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
> 
> Review-Url: https://codereview.chromium.org/2738813004
> Cr-Commit-Position: refs/heads/master@{#457158}
> Committed: https://chromium.googlesource.com/chromium/src/+/3c275e0a2333511ec831b3c396dc9f38a161a8c6

TBR=kapishnikov@chromium.org,mgersh@chromium.org,xunjieli@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=584806, 699567
Review-Url: https://codereview.chromium.org/2751113004
Cr-Commit-Position: refs/heads/master@{#457408}
Committed: https://chromium.googlesource.com/chromium/src/+/2e20ab7ecb1ffe0fc6ece1e5c57c14b8ce1ec01f

[cblume, 2017-03-16 05:14:14]

The CQ bit was checked by cblume@chromium.org

[cblume, 2017-03-16 05:14:15]

Created Revert of [Cronet] Write effective experimental options to NetLog

[commit-bot: I haz the power, 2017-03-16 05:14:40]

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, 2017-03-16 05:14:40]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-16 05:14:41]

No L-G-T-M from a valid reviewer yet.
CQ run can only be started once the patch has received an L-G-T-M from a full
committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,_not_ a
full super star committer.
Committers are members of the group "project-chromium-committers".
Note that this has nothing to do with OWNERS files.

[aelias_OOO_until_Jul13, 2017-03-16 05:56:03]

aelias@chromium.org changed reviewers:
+ aelias@chromium.org

[aelias_OOO_until_Jul13, 2017-03-16 05:56:03]

lgtm

[clamy, 2017-03-16 12:42:53]

The CQ bit was checked by clamy@chromium.org

[commit-bot: I haz the power, 2017-03-16 12:43:05]

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, 2017-03-16 12:43:56]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1489668173262140, "parent_rev":
"a87dcc8636090e23c4301a26b4645324653d9a52", "commit_rev":
"2e20ab7ecb1ffe0fc6ece1e5c57c14b8ce1ec01f"}

[commit-bot: I haz the power, 2017-03-16 12:44:38]

Description was changed from

==========
Revert of [Cronet] Write effective experimental options to NetLog (patchset #5
id:80001 of https://codereview.chromium.org/2738813004/ )

Reason for revert:
The bot

Android Cronet Builder Asan

is reporting

Failed steps failed cronet_unittests failed cronet_test_instrumentation_apk

I suspect this patch might be why.


https://uberchromegw.corp.google.com/i/chromium.android/builders/Android%20Cr...

Original issue's description:
> [Cronet] Write effective experimental options to NetLog
> 
> This CL does the following:
> (1) Have a warning in Experimental Options parsing code if there is an
>     unrecognized option.
> (2) Make cronet_url_request_context_adapter use the new FileNetLogObserver
which
>     allows writing polled data.
> (3) Put the effective experimental options in the polled data.
> 
> BUG=584806,699567
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
> 
> Review-Url: https://codereview.chromium.org/2738813004
> Cr-Commit-Position: refs/heads/master@{#457158}
> Committed:
https://chromium.googlesource.com/chromium/src/+/3c275e0a2333511ec831b3c396dc...

TBR=kapishnikov@chromium.org,mgersh@chromium.org,xunjieli@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=584806,699567
==========

to

==========
Revert of [Cronet] Write effective experimental options to NetLog (patchset #5
id:80001 of https://codereview.chromium.org/2738813004/ )

Reason for revert:
The bot

Android Cronet Builder Asan

is reporting

Failed steps failed cronet_unittests failed cronet_test_instrumentation_apk

I suspect this patch might be why.


https://uberchromegw.corp.google.com/i/chromium.android/builders/Android%20Cr...

Original issue's description:
> [Cronet] Write effective experimental options to NetLog
> 
> This CL does the following:
> (1) Have a warning in Experimental Options parsing code if there is an
>     unrecognized option.
> (2) Make cronet_url_request_context_adapter use the new FileNetLogObserver
which
>     allows writing polled data.
> (3) Put the effective experimental options in the polled data.
> 
> BUG=584806,699567
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
> 
> Review-Url: https://codereview.chromium.org/2738813004
> Cr-Commit-Position: refs/heads/master@{#457158}
> Committed:
https://chromium.googlesource.com/chromium/src/+/3c275e0a2333511ec831b3c396dc...

TBR=kapishnikov@chromium.org,mgersh@chromium.org,xunjieli@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=584806,699567

Review-Url: https://codereview.chromium.org/2751113004
Cr-Commit-Position: refs/heads/master@{#457408}
Committed:
https://chromium.googlesource.com/chromium/src/+/2e20ab7ecb1ffe0fc6ece1e5c57c...
==========

[commit-bot: I haz the power, 2017-03-16 12:44:39]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/2e20ab7ecb1ffe0fc6ece1e5c57c...