Revert of Add network throttling as an enterprise policy

Revert of Add network throttling as an enterprise policy (patchset #17 id:320001 of https://codereview.chromium.org/2364703002/ )

Reason for revert:
Introduces memory leak; see discussion in https://bugs.chromium.org/p/chromium/issues/detail?id=660769.

BUG=660769
Original issue's description:
> Add support for an enterprise policy that lets the admin
> turn on network throttling, and specify the upload and download rates in kbps to throttle to.
>
> BUG=642811
> TEST=(1) Unit-tests
> (2) Added network throttling policy to YAPS server,
> enabled throttling with various values, disabled,
> and checked that changes propagate to client device.
>
> Signed-off-by: Kirtika Ruchandani <kirtika@google.com>;
> Committed: https://crrev.com/7db3c2b899d448dc0235e03a248df83aa5778ebf
> Cr-Commit-Position: refs/heads/master@{#428565}

TBR=bartfab@chromium.org,stevenjb@chromium.org,kirtika@google.com,michaelpg@chromium.org,mnissler@chromium.org,atwilson@chromium.org,gab@chromium.org,brettw@chromium.org,isherman@chromium.org,asvitkine@chromium.org,thestig@chromium.org,kirtika@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=642811
Committed: https://crrev.com/02a8965352416f5c892f511b56bf4c37c778f507
Cr-Commit-Position: refs/heads/master@{#428703}

[phoglund_chromium, 2016-10-31 12:48:47]

The CQ bit was checked by phoglund@chromium.org

[phoglund_chromium, 2016-10-31 12:48:47]

Created Revert of Add network throttling as an enterprise policy

[phoglund_chromium, 2016-10-31 12:48:57]

The CQ bit was unchecked by phoglund@chromium.org

[commit-bot: I haz the power, 2016-10-31 12:48:58]

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

[phoglund_chromium, 2016-10-31 12:49:19]

Description was changed from

==========
Revert of Add network throttling as an enterprise policy (patchset #17 id:320001
of https://codereview.chromium.org/2364703002/ )

Reason for revert:
Introduces memory leak; see discussion in
https://bugs.chromium.org/p/chromium/issues/detail?id=660769.

Original issue's description:
> Add support for an enterprise policy that lets the admin
> turn on network throttling, and specify the upload and download rates in kbps
to throttle to.
> 
> BUG=642811
> TEST=(1) Unit-tests
> (2) Added network throttling policy to YAPS server,
> enabled throttling with various values, disabled,
> and checked that changes propagate to client device.
> 
> Signed-off-by: Kirtika Ruchandani <kirtika@google.com>
> Committed: https://crrev.com/7db3c2b899d448dc0235e03a248df83aa5778ebf
> Cr-Commit-Position: refs/heads/master@{#428565}

TBR=bartfab@chromium.org,stevenjb@chromium.org,kirtika@google.com,michaelpg@c...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=642811
==========

to

==========
Revert of Add network throttling as an enterprise policy (patchset #17 id:320001
of https://codereview.chromium.org/2364703002/ )

Reason for revert:
Introduces memory leak; see discussion in
https://bugs.chromium.org/p/chromium/issues/detail?id=660769.

BUG=660769

Original issue's description:
> Add support for an enterprise policy that lets the admin
> turn on network throttling, and specify the upload and download rates in kbps
to throttle to.
> 
> BUG=642811
> TEST=(1) Unit-tests
> (2) Added network throttling policy to YAPS server,
> enabled throttling with various values, disabled,
> and checked that changes propagate to client device.
> 
> Signed-off-by: Kirtika Ruchandani <kirtika@google.com>
> Committed: https://crrev.com/7db3c2b899d448dc0235e03a248df83aa5778ebf
> Cr-Commit-Position: refs/heads/master@{#428565}

TBR=bartfab@chromium.org,stevenjb@chromium.org,kirtika@google.com,michaelpg@c...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=642811
==========

[phoglund_chromium, 2016-10-31 12:49:29]

The CQ bit was unchecked by phoglund@chromium.org

[phoglund_chromium, 2016-10-31 12:49:37]

The CQ bit was checked by phoglund@chromium.org

[commit-bot: I haz the power, 2016-10-31 12:49:48]

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-10-31 12:54:38]

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

[commit-bot: I haz the power, 2016-10-31 12:54:40]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[phoglund_chromium, 2016-10-31 12:55:48]

The CQ bit was checked by phoglund@chromium.org

[commit-bot: I haz the power, 2016-10-31 12:56:03]

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-10-31 13:02:02]

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

[commit-bot: I haz the power, 2016-10-31 13:02:03]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[phoglund_chromium, 2016-10-31 13:15:51]

On 2016/10/31 13:02:03, commit-bot: I haz the power wrote:
> Try jobs failed on following builders:
>   chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
>
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

Wut, why is the presubmit failing?

# HEAD detached at origin/master
nothing to commit, working tree clean

If there was a patch failure (or non-application, as it seems above) I'd imagine
the other bots would fail as well.

[phoglund_chromium, 2016-10-31 13:20:16]

The CQ bit was checked by phoglund@chromium.org

[commit-bot: I haz the power, 2016-10-31 13:20:32]

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

[phoglund_chromium, 2016-10-31 13:21:49]

On 2016/10/31 13:15:51, phoglund_chrome wrote:
> On 2016/10/31 13:02:03, commit-bot: I haz the power wrote:
> > Try jobs failed on following builders:
> >   chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
> >
>
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)
> 
> Wut, why is the presubmit failing?
> 
> # HEAD detached at origin/master
> nothing to commit, working tree clean
> 
> If there was a patch failure (or non-application, as it seems above) I'd
imagine
> the other bots would fail as well.

When I think of it, I don't understand why patch set 17 shows changes to two
files,
whereas the crrev link shows modifications to 25 files (like in the list above).
Oh well, revert should go in now.

[commit-bot: I haz the power, 2016-10-31 14:20:35]

Description was changed from

==========
Revert of Add network throttling as an enterprise policy (patchset #17 id:320001
of https://codereview.chromium.org/2364703002/ )

Reason for revert:
Introduces memory leak; see discussion in
https://bugs.chromium.org/p/chromium/issues/detail?id=660769.

BUG=660769

Original issue's description:
> Add support for an enterprise policy that lets the admin
> turn on network throttling, and specify the upload and download rates in kbps
to throttle to.
> 
> BUG=642811
> TEST=(1) Unit-tests
> (2) Added network throttling policy to YAPS server,
> enabled throttling with various values, disabled,
> and checked that changes propagate to client device.
> 
> Signed-off-by: Kirtika Ruchandani <kirtika@google.com>
> Committed: https://crrev.com/7db3c2b899d448dc0235e03a248df83aa5778ebf
> Cr-Commit-Position: refs/heads/master@{#428565}

TBR=bartfab@chromium.org,stevenjb@chromium.org,kirtika@google.com,michaelpg@c...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=642811
==========

to

==========
Revert of Add network throttling as an enterprise policy (patchset #17 id:320001
of https://codereview.chromium.org/2364703002/ )

Reason for revert:
Introduces memory leak; see discussion in
https://bugs.chromium.org/p/chromium/issues/detail?id=660769.

BUG=660769

Original issue's description:
> Add support for an enterprise policy that lets the admin
> turn on network throttling, and specify the upload and download rates in kbps
to throttle to.
> 
> BUG=642811
> TEST=(1) Unit-tests
> (2) Added network throttling policy to YAPS server,
> enabled throttling with various values, disabled,
> and checked that changes propagate to client device.
> 
> Signed-off-by: Kirtika Ruchandani <kirtika@google.com>
> Committed: https://crrev.com/7db3c2b899d448dc0235e03a248df83aa5778ebf
> Cr-Commit-Position: refs/heads/master@{#428565}

TBR=bartfab@chromium.org,stevenjb@chromium.org,kirtika@google.com,michaelpg@c...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=642811
==========

[commit-bot: I haz the power, 2016-10-31 14:20:38]

Committed patchset #2 (id:70001)

[commit-bot: I haz the power, 2016-10-31 14:22:15]

Description was changed from

==========
Revert of Add network throttling as an enterprise policy (patchset #17 id:320001
of https://codereview.chromium.org/2364703002/ )

Reason for revert:
Introduces memory leak; see discussion in
https://bugs.chromium.org/p/chromium/issues/detail?id=660769.

BUG=660769

Original issue's description:
> Add support for an enterprise policy that lets the admin
> turn on network throttling, and specify the upload and download rates in kbps
to throttle to.
> 
> BUG=642811
> TEST=(1) Unit-tests
> (2) Added network throttling policy to YAPS server,
> enabled throttling with various values, disabled,
> and checked that changes propagate to client device.
> 
> Signed-off-by: Kirtika Ruchandani <kirtika@google.com>
> Committed: https://crrev.com/7db3c2b899d448dc0235e03a248df83aa5778ebf
> Cr-Commit-Position: refs/heads/master@{#428565}

TBR=bartfab@chromium.org,stevenjb@chromium.org,kirtika@google.com,michaelpg@c...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=642811
==========

to

==========
Revert of Add network throttling as an enterprise policy (patchset #17 id:320001
of https://codereview.chromium.org/2364703002/ )

Reason for revert:
Introduces memory leak; see discussion in
https://bugs.chromium.org/p/chromium/issues/detail?id=660769.

BUG=660769

Original issue's description:
> Add support for an enterprise policy that lets the admin
> turn on network throttling, and specify the upload and download rates in kbps
to throttle to.
>
> BUG=642811
> TEST=(1) Unit-tests
> (2) Added network throttling policy to YAPS server,
> enabled throttling with various values, disabled,
> and checked that changes propagate to client device.
>
> Signed-off-by: Kirtika Ruchandani <kirtika@google.com>
> Committed: https://crrev.com/7db3c2b899d448dc0235e03a248df83aa5778ebf
> Cr-Commit-Position: refs/heads/master@{#428565}

TBR=bartfab@chromium.org,stevenjb@chromium.org,kirtika@google.com,michaelpg@c...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=642811

Committed: https://crrev.com/02a8965352416f5c892f511b56bf4c37c778f507
Cr-Commit-Position: refs/heads/master@{#428703}
==========

[commit-bot: I haz the power, 2016-10-31 14:22:17]

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