Skip to content

Commit

Permalink
Changed CloudPrintURLFetcher to be more testable
Browse files Browse the repository at this point in the history
Made it use CreateURLFetcher with an id rather than without one,
because CreateURLFetcher with an id can be rewired for testing to
use a URLFetcherFactory.

BUG=


Review URL: https://chromiumcodereview.appspot.com/12223056

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181577 0039d316-1c4b-4281-b951-d872f2087c98
  • Loading branch information
noamsml@google.com committed Feb 9, 2013
1 parent d00e126 commit 14a6db1
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion chrome/service/cloud_print/cloud_print_url_fetcher.cc
Original file line number Diff line number Diff line change
Expand Up @@ -174,7 +174,7 @@ void CloudPrintURLFetcher::StartRequestHelper(
DCHECK(delegate);
// Persist the additional headers in case we need to retry the request.
additional_headers_ = additional_headers;
request_.reset(net::URLFetcher::Create(url, request_type, this));
request_.reset(net::URLFetcher::Create(0, url, request_type, this));
request_->SetRequestContext(GetRequestContextGetter());
// Since we implement our own retry logic, disable the retry in URLFetcher.
request_->SetAutomaticallyRetryOn5xx(false);
Expand Down

0 comments on commit 14a6db1

Please sign in to comment.