It suffices to just forward-declare DnsCertProvenanceChecker

It suffices to just forward-declare DnsCertProvenanceChecker
in net/url_request/url_request_context.h.  Remove an unnecessary
forward declaration of SSLConfigService.

R=agl@chromium.org
BUG=none
TEST=no compilation errors

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=86452

[wtc, 2011-05-24 14:53:37]

Please review url_request_context.h first.  The changes to
other files are a result of that change.

http://codereview.chromium.org/6990057/diff/1/net/url_request/url_request_con...
File net/url_request/url_request_context.h (left):

http://codereview.chromium.org/6990057/diff/1/net/url_request/url_request_con...
net/url_request/url_request_context.h:21: #include
"net/socket/dns_cert_provenance_checker.h"
We only need the forward declaration on line 26.

http://codereview.chromium.org/6990057/diff/1/net/url_request/url_request_con...
net/url_request/url_request_context.h:34: class SSLConfigService;
We include "net/base/ssl_config_service.h" on line 18, so
this forward declaration is not necessary.

[agl, 2011-05-24 14:56:26]

LGTM

[commit-bot: I haz the power, 2011-05-24 16:56:36]

Presubmit check for 6990057-1 failed and returned exit status 1.

Running presubmit commit checks ...

** Presubmit ERRORS **
Missing LGTM from an OWNER for: chrome/browser/net/chrome_url_request_context.h

[willchan no longer on Chromium, 2011-05-24 16:57:45]

LGTM stamp.

On Tue, May 24, 2011 at 4:56 PM, <commit-bot@chromium.org> wrote:

> Presubmit check for 6990057-1 failed and returned exit status 1.
>
> Running presubmit commit checks ...
>
> ** Presubmit ERRORS **
> Missing LGTM from an OWNER for:
> chrome/browser/net/chrome_url_request_context.h
>
>
>
>
> http://codereview.chromium.org/6990057/
>

[commit-bot: I haz the power, 2011-05-24 18:03:11]

Change committed as 86452