New issue
Advanced search Search tips

Issue 701062 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug

Blocking:
issue 634061



Sign in to add a comment

Add a test to ensure that net stack accepts brotli encoded content from DRP proxy

Project Member Reported by tbansal@chromium.org, Mar 13 2017

Issue description

Add a test to ensure that net stack accepts brotli encoded content from DRP proxy.

Network stack wants to add checks which will reject responses with encoding type that do not match the advertised encoding type (https://groups.google.com/a/chromium.org/d/topic/chromium-dev/GxIgxok9f2k/discussion). The plan is to add these checks below the proxy delegate. So, DRP usage of Brotli should remain unaffected.

To ensure that DRP usage remains unaffected, DRP should add additional tests that ensure that Brotli encoded content from DRP is successfully handled.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Mar 13 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/48a95392c57e24be127436d4e318a573467591df

commit 48a95392c57e24be127436d4e318a573467591df
Author: tbansal <tbansal@chromium.org>
Date: Mon Mar 13 22:38:56 2017

DRP: Add a test to ensure that brotli encoded content is decoded correctly

This CL adds a test that verifies that if the response from data
reduction proxy is Brotli encoded, then it is successfully decoded.

BUG= 701062 

Review-Url: https://codereview.chromium.org/2744393002
Cr-Commit-Position: refs/heads/master@{#456509}

[modify] https://crrev.com/48a95392c57e24be127436d4e318a573467591df/components/data_reduction_proxy/core/browser/data_reduction_proxy_network_delegate_unittest.cc

Blocking: 634061
Status: (was: Started)
Labels: M-59
Status: Fixed

Sign in to add a comment