diff options
author | jam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-10-25 17:56:12 +0000 |
---|---|---|
committer | jam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-10-25 17:56:12 +0000 |
commit | 7cc6e5639982a9f912c46572dafa4a42682fbd04 (patch) | |
tree | aeda0f815ef8b72e5c2c154d66ab21c43dc9f9dc /content/test/test_url_fetcher_factory.h | |
parent | ea725b3aa4fe6ffdb15610d3a6a14d620088d878 (diff) | |
download | chromium_src-7cc6e5639982a9f912c46572dafa4a42682fbd04.zip chromium_src-7cc6e5639982a9f912c46572dafa4a42682fbd04.tar.gz chromium_src-7cc6e5639982a9f912c46572dafa4a42682fbd04.tar.bz2 |
Create a content::UrlFetcher interface that lives in content/public/common and convert users to it. I have added a static create function, but will switch instantiations to use it in another change since this has grown a lot. Basically this change converts function names away from unix_hacker style, which they shouldn't have been using anyways since some are virtual, and made all of the other functions virtual.
BUG=98716
Review URL: http://codereview.chromium.org/8375039
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@107151 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'content/test/test_url_fetcher_factory.h')
-rw-r--r-- | content/test/test_url_fetcher_factory.h | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/content/test/test_url_fetcher_factory.h b/content/test/test_url_fetcher_factory.h index 2e7593e..27e03ef 100644 --- a/content/test/test_url_fetcher_factory.h +++ b/content/test/test_url_fetcher_factory.h @@ -71,10 +71,10 @@ class TestURLFetcher : public URLFetcher { // Unique ID in our factory. int id() const { return id_; } - // URL we were created with. Because of how we're using URLFetcher url() - // always returns an empty URL. Chances are you'll want to use original_url() - // in your tests. - virtual const GURL& original_url() const OVERRIDE; + // URL we were created with. Because of how we're using URLFetcher GetUrl() + // always returns an empty URL. Chances are you'll want to use + // GetOriginalUrl() in your tests. + virtual const GURL& GetOriginalUrl() const OVERRIDE; // Returns the data uploaded on this URLFetcher. const std::string& upload_data() const { return URLFetcher::upload_data(); } @@ -88,18 +88,18 @@ class TestURLFetcher : public URLFetcher { } void set_url(const GURL& url) { fake_url_ = url; } - virtual const GURL& url() const OVERRIDE; + virtual const GURL& GetUrl() const OVERRIDE; void set_status(const net::URLRequestStatus& status); - virtual const net::URLRequestStatus& status() const OVERRIDE; + virtual const net::URLRequestStatus& GetStatus() const OVERRIDE; void set_response_code(int response_code) { fake_response_code_ = response_code; } - virtual int response_code() const OVERRIDE; + virtual int GetResponseCode() const OVERRIDE; void set_cookies(const net::ResponseCookies& c) { fake_cookies_ = c; } - virtual const net::ResponseCookies& cookies() const OVERRIDE; + virtual const net::ResponseCookies& GetCookies() const OVERRIDE; void set_was_fetched_via_proxy(bool flag); |