Cookie commands for the webdriver protocol

chrome/test/webdriver/automation.cc

Index: chrome/test/webdriver/automation.cc
diff --git a/chrome/test/webdriver/automation.cc b/chrome/test/webdriver/automation.cc
index e0afa4d83386f72c2ce93cc23e3c0fe7f8bd8c2b..99ac048250aee86cf5bcc39cf6a12789f0f29638 100644
--- a/chrome/test/webdriver/automation.cc
+++ b/chrome/test/webdriver/automation.cc
@@ -90,6 +90,11 @@ void Automation::GetURL(std::string* url,
     *url = gurl.possibly_invalid_spec();
 }
 
+void Automation::GetGURL(GURL* gurl,
+                         bool* success) {
+  *success = tab_->GetCurrentURL(gurl);
+}
+
 void Automation::GetTabTitle(std::string* tab_title,
                              bool* success) {
   std::wstring wide_title;
@@ -98,4 +103,29 @@ void Automation::GetTabTitle(std::string* tab_title,
     *tab_title = WideToUTF8(wide_title);
 }
 
+void Automation::GetCookies(const GURL& gurl,
+                            std::string* cookies,
+                            bool* success) {
+  *success = tab_->GetCookies(gurl, cookies);
+}
+
+void Automation::GetCookieByName(const GURL& gurl,
+                                 const std::string& cookie_name,
+                                 std::string* cookie,
+                                 bool* success) {
+  *success = tab_->GetCookieByName(gurl, cookie_name, cookie);
+}
+
+void Automation::DeleteCookie(const GURL& gurl,
+                              const std::string& cookie_name,
+                              bool* success) {
+  *success = tab_->DeleteCookie(gurl, cookie_name);
+}
+
+void Automation::SetCookie(const GURL& gurl,
+                           const std::string& cookie,
+                           bool* success) {
+  *success = tab_->SetCookie(gurl, cookie);

[John Grabowski, 2011/02/14 21:59:06]

You probably also need a

Automation::SetCookie(const GURL& gurl, bool scout, const std::string& cookies)
{ }

[Joe, 2011/02/15 02:30:48]

I am not sure why?  I don't think chromedriver needs that case since the current
functions implement what we need.

On 2011/02/14 21:59:06, John Grabowski wrote:

+}
+
 }  // namespace webdriver