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

chrome/test/webdriver/commands/session_with_id.h

Index: chrome/test/webdriver/commands/session_with_id.h
diff --git a/chrome/test/webdriver/commands/session_with_id.h b/chrome/test/webdriver/commands/session_with_id.h
index a85bc5f615ed624c62be95af3cee0a9563574069..80847166451585e41f5e696c648c31a310bd7f57 100644
--- a/chrome/test/webdriver/commands/session_with_id.h
+++ b/chrome/test/webdriver/commands/session_with_id.h
@@ -22,19 +22,18 @@ namespace webdriver {
 // 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);
 };