Even more test cleanup. Some fixes to non-test code.

chrome/test/webdriver/commands/session_with_id.h

 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_TEST_WEBDRIVER_COMMANDS_SESSION_WITH_ID_H_
 #define CHROME_TEST_WEBDRIVER_COMMANDS_SESSION_WITH_ID_H_
 
 #include <string>
 #include <vector>
 
 #include "chrome/test/webdriver/commands/command.h"
 #include "chrome/test/webdriver/commands/webdriver_command.h"
 
 namespace webdriver {
 
 // Retrieve the capabilities of the specified session. If the HTTP Delete
 // method is used then all chrome instances linked to the session ID are
 // closed.  The session's capabilities will be returned in a JSON object
 // with the following properties:
 // http://code.google.com/p/selenium/wiki/JsonWireProtocol#/session/:sessionId
 // If the HTTP Delete request is sent the session is closed and all temp
 // files and directories created are deleted.
 class SessionWithID : public WebDriverCommand {
  public:
-  inline SessionWithID(const std::vector<std::string>& path_segments,
-                       const DictionaryValue* const parameters)
-      : WebDriverCommand(path_segments, parameters) {}
-  virtual ~SessionWithID() {}
+  SessionWithID(const std::vector<std::string>& path_segments,
+                const DictionaryValue* const parameters);
+  virtual ~SessionWithID();
 
-  virtual bool DoesGet() { return true; }
-  virtual bool DoesDelete() { return true; }
+  virtual bool DoesGet();
+  virtual bool DoesDelete();
 
   virtual void ExecuteGet(Response* const response);
   virtual void ExecuteDelete(Response* const response);
 
  private:
-  virtual bool RequiresValidTab() { return false; }
+  virtual bool RequiresValidTab();
 
   DISALLOW_COPY_AND_ASSIGN(SessionWithID);
 };
 
 }  // namespace webdriver
 
 #endif  // CHROME_TEST_WEBDRIVER_COMMANDS_SESSION_WITH_ID_H_