Cookie commands for the webdriver protocol

chrome/test/webdriver/commands/command.cc

 // 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.
 
 #include "chrome/test/webdriver/commands/command.h"
 
 namespace webdriver {
 
 // Error message taken from:
 // http://code.google.com/p/selenium/wiki/JsonWireProtocol#Response_Status_Codes
@@ skipping 19 matching lines @@
 bool Command::GetBooleanParameter(const std::string& key,
                                   bool* out) const {
   return parameters_.get() != NULL && parameters_->GetBoolean(key, out);
 }
 
 bool Command::GetIntegerParameter(const std::string& key,
                                   int* out) const {
   return parameters_.get() != NULL && parameters_->GetInteger(key, out);
 }
 
+bool Command::GetDictionaryParameter(const std::string& key,

[kkania, 2011/01/28 23:25:59]

can we just get rid of these functions and let parameters_ be protected? There
doesn't seem to be much point in them.

[Joe, 2011/02/03 10:02:05]

It was like that in the original version but Jason Leyba set this up with the
refractor.  I think this is actually much cleaner since it doesn't litter the
code with a bunch of if statements.

On 2011/01/28 23:25:59, kkania wrote:

+                                     DictionaryValue** out) const {
+  return parameters_.get() != NULL && parameters_->GetDictionary(key, out);
+}
+        
 }  // namespace webdriver