Enable support for Rice encoding

Enable support for Rice encoding

BUG=543161, 624567
Committed: https://crrev.com/f55c5b229c13a6266c16b2234af973c569303de1
Cr-Commit-Position: refs/heads/master@{#413066}

[vakh (use Gerrit instead), 2016-08-17 22:31:38]

Enable Chrome to request Rice encoding

[vakh (use Gerrit instead), 2016-08-17 22:32:56]

vakh@chromium.org changed reviewers:
+ nparker@chromium.org

[Nathan Parker, 2016-08-19 00:18:23]

lgtm

https://codereview.chromium.org/2241863004/diff/20001/components/safe_browsin...
File components/safe_browsing_db/v4_update_protocol_manager.cc (right):

https://codereview.chromium.org/2241863004/diff/20001/components/safe_browsin...
components/safe_browsing_db/v4_update_protocol_manager.cc:215: // TODO(vakh):
Accept other compression formats also.
remove todo

https://codereview.chromium.org/2241863004/diff/20001/components/safe_browsin...
components/safe_browsing_db/v4_update_protocol_manager.cc:218:
list_update_request->mutable_constraints()->add_supported_compressions(
Are there any tests that is affects or should affect?  We might want to verify
we're requesting the expected list of threat types, among others.

[vakh (use Gerrit instead), 2016-08-19 00:45:05]

Remove TODO. Update test to expect RICE as supported compression.

[vakh (use Gerrit instead), 2016-08-19 00:45:07]

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

[commit-bot: I haz the power, 2016-08-19 00:45:57]

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

[vakh (use Gerrit instead), 2016-08-19 00:46:35]

https://codereview.chromium.org/2241863004/diff/20001/components/safe_browsin...
File components/safe_browsing_db/v4_update_protocol_manager.cc (right):

https://codereview.chromium.org/2241863004/diff/20001/components/safe_browsin...
components/safe_browsing_db/v4_update_protocol_manager.cc:215: // TODO(vakh):
Accept other compression formats also.
On 2016/08/19 at 00:18:22, Nathan Parker wrote:
> remove todo

Done.

https://codereview.chromium.org/2241863004/diff/20001/components/safe_browsin...
components/safe_browsing_db/v4_update_protocol_manager.cc:218:
list_update_request->mutable_constraints()->add_supported_compressions(
On 2016/08/19 at 00:18:22, Nathan Parker wrote:
> Are there any tests that is affects or should affect?  We might want to verify
we're requesting the expected list of threat types, among others.

Good catch. Done.

[commit-bot: I haz the power, 2016-08-19 02:24:26]

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

[commit-bot: I haz the power, 2016-08-19 02:24:27]

Dry run: This issue passed the CQ dry run.

[vakh (use Gerrit instead), 2016-08-19 06:11:50]

The CQ bit was checked by vakh@chromium.org

[vakh (use Gerrit instead), 2016-08-19 06:11:50]

The patchset sent to the CQ was uploaded after l-g-t-m from nparker@chromium.org
Link to the patchset: https://codereview.chromium.org/2241863004/#ps40001
(title: "Remove TODO. Update test to expect RICE as supported compression.")

[commit-bot: I haz the power, 2016-08-19 06:12:13]

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-19 06:15:50]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-08-19 06:19:29]

Description was changed from

==========
Enable support for Rice encoding

BUG=543161, 624567
==========

to

==========
Enable support for Rice encoding

BUG=543161, 624567

Committed: https://crrev.com/f55c5b229c13a6266c16b2234af973c569303de1
Cr-Commit-Position: refs/heads/master@{#413066}
==========

[commit-bot: I haz the power, 2016-08-19 06:19:30]

Patchset 3 (id:??) landed as
https://crrev.com/f55c5b229c13a6266c16b2234af973c569303de1
Cr-Commit-Position: refs/heads/master@{#413066}