Hook SetKeyOnIO in Webview to DataReductionProxyIOData

Hook SetKeyOnIO in Webview to DataReductionProxyIOData

AwURLRequestContextGetter::SetKeyOnIO now calls into the
DataReductionProxyAuthRequestHandler that is managed by
DataReductionProxyIOData.

BUG=453519
Committed: https://crrev.com/76b3a496ce4e6da8779cd408202fe7e0f6edda46
Cr-Commit-Position: refs/heads/master@{#315119}

[bengr, 2015-02-05 18:22:59]

Patchset #1 (id:1) has been deleted

[bengr, 2015-02-05 18:24:07]

bengr@chromium.org changed reviewers:
+ sgurun@chromium.org

[bengr, 2015-02-05 18:24:08]

sgurun: aw

[bengr, 2015-02-05 18:24:27]

bengr@chromium.org changed reviewers:
+ sclittle@chromium.org

[bengr, 2015-02-05 18:24:39]

And sclittle: components/*

[sgurun-gerrit only, 2015-02-05 18:42:53]

On 2015/02/05 18:24:39, bengr wrote:
> And sclittle: components/*

can we write a test for this somehow?

[sclittle, 2015-02-05 19:27:18]

On 2015/02/05 18:42:53, sgurun wrote:
> On 2015/02/05 18:24:39, bengr wrote:
> > And sclittle: components/*
> 
> can we write a test for this somehow?

+1 for testing this if possible.

[bengr, 2015-02-06 00:31:45]

On 2015/02/05 19:27:18, sclittle wrote:
> On 2015/02/05 18:42:53, sgurun wrote:
> > On 2015/02/05 18:24:39, bengr wrote:
> > > And sclittle: components/*
> > 
> > can we write a test for this somehow?
> 
> +1 for testing this if possible.

@sgurun: Where are tests for AwURLRequestContextGetter?

[sgurun-gerrit only, 2015-02-06 18:15:37]

On 2015/02/06 00:31:45, bengr wrote:
> On 2015/02/05 19:27:18, sclittle wrote:
> > On 2015/02/05 18:42:53, sgurun wrote:
> > > On 2015/02/05 18:24:39, bengr wrote:
> > > > And sclittle: components/*
> > > 
> > > can we write a test for this somehow?
> > 
> > +1 for testing this if possible.
> 
> @sgurun: Where are tests for AwURLRequestContextGetter?

ok seems like hush has filed a bug on that.  crbug/455926

lgtm

[sclittle, 2015-02-06 18:23:31]

LGTM

[bengr, 2015-02-06 21:55:02]

The CQ bit was checked by bengr@chromium.org

[commit-bot: I haz the power, 2015-02-06 21:56:00]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/881893004/2

[commit-bot: I haz the power, 2015-02-06 22:00:03]

Committed patchset #1 (id:2)

[commit-bot: I haz the power, 2015-02-06 22:00:21]

Patchset 1 (id:??) landed as
https://crrev.com/76b3a496ce4e6da8779cd408202fe7e0f6edda46
Cr-Commit-Position: refs/heads/master@{#315119}