Clean-up inline members of nested classes (net/)

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/10854063

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@150917 0039d316-1c4b-4281-b951-d872f2087c98

2 files changed, 6 insertions, 3 deletions

diff --git a/net/url_request/url_request_job_factory.cc b/net/url_request/url_request_job_factory.cc
index f5ff25d..44bc002 100644
--- a/net/url_request/url_request_job_factory.cc
+++ b/net/url_request/url_request_job_factory.cc
@@ -15,6 +15,11 @@ URLRequestJobFactory::ProtocolHandler::~ProtocolHandler() {}
 
 URLRequestJobFactory::Interceptor::~Interceptor() {}
 
+bool URLRequestJobFactory::Interceptor::WillHandleProtocol(
+    const std::string& protocol) const {
+  return false;
+}
+
 URLRequestJobFactory::URLRequestJobFactory() {}
 
 URLRequestJobFactory::~URLRequestJobFactory() {
diff --git a/net/url_request/url_request_job_factory.h b/net/url_request/url_request_job_factory.h
index 2c0c41c..746e948 100644
--- a/net/url_request/url_request_job_factory.h
+++ b/net/url_request/url_request_job_factory.h
@@ -62,9 +62,7 @@ class NET_EXPORT URLRequestJobFactory
     // Returns true if this interceptor handles requests for URLs with the
     // given protocol. Returning false does not imply that this interceptor
     // can't or won't handle requests with the given protocol.
-    virtual bool WillHandleProtocol(const std::string& protocol) const {
-        return false;
-    }
+    virtual bool WillHandleProtocol(const std::string& protocol) const;
   };
 
   URLRequestJobFactory();