summaryrefslogtreecommitdiffstats
path: root/chrome/service/cloud_print
diff options
context:
space:
mode:
authorhans@chromium.org <hans@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2012-08-10 22:08:08 +0000
committerhans@chromium.org <hans@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2012-08-10 22:08:08 +0000
commit23827ecacc856f01a55cd7102270c3ff86df042b (patch)
treeac16dba390c6563f4085991a21ecdb07d4913086 /chrome/service/cloud_print
parent77c6d060599dccd23ae1bb4ddbc809cb550d2178 (diff)
downloadchromium_src-23827ecacc856f01a55cd7102270c3ff86df042b.zip
chromium_src-23827ecacc856f01a55cd7102270c3ff86df042b.tar.gz
chromium_src-23827ecacc856f01a55cd7102270c3ff86df042b.tar.bz2
Clean-up inline members of nested classes (chrome/)
Due to a bug, the Clang-plugin style checker failed to warn about inline constructors, destructors, non-empty virtual methods, etc. for nested classes. The plugin has been fixed, and this patch is part of a clean-up of all the code that now causes the plugin to issue errors. BUG=139346 Review URL: https://chromiumcodereview.appspot.com/10855075 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151135 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/service/cloud_print')
-rw-r--r--chrome/service/cloud_print/cloud_print_url_fetcher.cc29
-rw-r--r--chrome/service/cloud_print/cloud_print_url_fetcher.h16
2 files changed, 36 insertions, 9 deletions
diff --git a/chrome/service/cloud_print/cloud_print_url_fetcher.cc b/chrome/service/cloud_print/cloud_print_url_fetcher.cc
index 95ed8db..77a37ce 100644
--- a/chrome/service/cloud_print/cloud_print_url_fetcher.cc
+++ b/chrome/service/cloud_print/cloud_print_url_fetcher.cc
@@ -18,6 +18,35 @@
#include "net/url_request/url_fetcher.h"
#include "net/url_request/url_request_status.h"
+
+CloudPrintURLFetcher::ResponseAction
+CloudPrintURLFetcher::Delegate::HandleRawResponse(
+ const net::URLFetcher* source,
+ const GURL& url,
+ const net::URLRequestStatus& status,
+ int response_code,
+ const net::ResponseCookies& cookies,
+ const std::string& data) {
+ return CONTINUE_PROCESSING;
+}
+
+CloudPrintURLFetcher::ResponseAction
+CloudPrintURLFetcher::Delegate::HandleRawData(
+ const net::URLFetcher* source,
+ const GURL& url,
+ const std::string& data) {
+ return CONTINUE_PROCESSING;
+}
+
+CloudPrintURLFetcher::ResponseAction
+CloudPrintURLFetcher::Delegate::HandleJSONData(
+ const net::URLFetcher* source,
+ const GURL& url,
+ base::DictionaryValue* json_data,
+ bool succeeded) {
+ return CONTINUE_PROCESSING;
+}
+
CloudPrintURLFetcher::CloudPrintURLFetcher()
: delegate_(NULL),
num_retries_(0) {
diff --git a/chrome/service/cloud_print/cloud_print_url_fetcher.h b/chrome/service/cloud_print/cloud_print_url_fetcher.h
index 77bf8f68..d976f712 100644
--- a/chrome/service/cloud_print/cloud_print_url_fetcher.h
+++ b/chrome/service/cloud_print/cloud_print_url_fetcher.h
@@ -50,9 +50,8 @@ class CloudPrintURLFetcher
const net::URLRequestStatus& status,
int response_code,
const net::ResponseCookies& cookies,
- const std::string& data) {
- return CONTINUE_PROCESSING;
- }
+ const std::string& data);
+
// This will be invoked only if HandleRawResponse returns
// CONTINUE_PROCESSING AND if there are no network errors and the HTTP
// response code is 200. The delegate implementation returns
@@ -62,21 +61,20 @@ class CloudPrintURLFetcher
// response).
virtual ResponseAction HandleRawData(const net::URLFetcher* source,
const GURL& url,
- const std::string& data) {
- return CONTINUE_PROCESSING;
- }
+ const std::string& data);
+
// This will be invoked only if HandleRawResponse and HandleRawData return
// CONTINUE_PROCESSING AND if the response contains a valid JSON dictionary.
// |succeeded| is the value of the "success" field in the response JSON.
virtual ResponseAction HandleJSONData(const net::URLFetcher* source,
const GURL& url,
base::DictionaryValue* json_data,
- bool succeeded) {
- return CONTINUE_PROCESSING;
- }
+ bool succeeded);
+
// Invoked when the retry limit for this request has been reached (if there
// was a retry limit - a limit of -1 implies no limit).
virtual void OnRequestGiveUp() { }
+
// Invoked when the request returns a 403 error (applicable only when
// HandleRawResponse returns CONTINUE_PROCESSING).
// Returning RETRY_REQUEST will retry current request. (auth information