Removing commented code. No functional changes.

Author: jimevans

cpp/iedriver/CommandHandlers/DeleteAllCookiesCommandHandler.h

@@ -47,8 +47,6 @@ class DeleteAllCookiesCommandHandler : public IECommandHandler {
     browser_wrapper->GetCookies(&cookies);
     std::vector<BrowserCookie>::const_iterator it = cookies.begin();
     for (; it != cookies.end(); ++it) {
-      //std::string cookie_name = it->name();
-      //status_code = browser_wrapper->DeleteCookie(cookie_name);
       status_code = browser_wrapper->DeleteCookie(*it);
       if (status_code != WD_SUCCESS) {
         response->SetErrorResponse(status_code,

cpp/iedriver/CommandHandlers/DeleteCookieCommandHandler.h

@@ -48,7 +48,6 @@ class DeleteCookieCommandHandler : public IECommandHandler {
       response->SetErrorResponse(status_code, "Unable to get browser");
       return;
     }
-    //status_code = browser_wrapper->DeleteCookie(cookie_name);
     BrowserCookie cookie;
     cookie.set_name(cookie_name);
     status_code = browser_wrapper->DeleteCookie(cookie);

cpp/iedriver/CookieManager.cpp

@@ -186,26 +186,6 @@ bool CookieManager::DeleteCookie(const std::string& url,
                                        cookie.is_httponly());
 }
 
-//bool CookieManager::DeleteCookie(const std::string& url,
-//                                 const std::string& cookie_name) {
-//  
-//  std::wstring wide_url = StringUtilities::ToWString(url);
-//  CComPtr<IUri> uri_pointer;
-//  ::CreateUri(wide_url.c_str(), Uri_CREATE_ALLOW_RELATIVE, 0, &uri_pointer);
-//
-//  CComBSTR host_bstr;
-//  uri_pointer->GetHost(&host_bstr);
-//  std::wstring wide_domain = host_bstr;
-//  
-//  CComBSTR path_bstr;
-//  uri_pointer->GetPath(&path_bstr);
-//  std::wstring wide_path = path_bstr;
-//
-//  std::string domain = StringUtilities::ToString(wide_domain);
-//  std::string path = StringUtilities::ToString(wide_path);
-//  return this->RecursivelyDeleteCookie(url, cookie_name, domain, path, false);
-//}
-
 bool CookieManager::RecursivelyDeleteCookie(const std::string& url,
                                             const std::string& name,
                                             const std::string& domain,

cpp/iedriver/CookieManager.h

@@ -37,7 +37,6 @@ class CookieManager {
                  std::vector<BrowserCookie>* all_cookies);
   bool SetCookie(const std::string& url, const std::string& cookie_data);
   bool DeleteCookie(const std::string& url, const BrowserCookie& cookie);
-  bool DeleteCookie(const std::string& url, const std::string& cookie_name);
 
  private:
   std::wstring SendGetCookieMessage(const std::wstring& url,

cpp/iedriver/DocumentHost.cpp

@@ -250,14 +250,6 @@ int DocumentHost::AddCookie(const std::string& cookie,
   return WD_SUCCESS;
 }
 
-//int DocumentHost::DeleteCookie(const std::string& cookie_name) {
-//  LOG(TRACE) << "Entering DocumentHost::DeleteCookie";
-//  // TODO: Optimize and return legitimate error conditions.
-//  bool deletesucceeded = this->cookie_manager_->DeleteCookie(this->GetCurrentUrl(),
-//                                                             cookie_name);
-//  return WD_SUCCESS;
-//}
-
 int DocumentHost::DeleteCookie(const BrowserCookie& cookie) {
   LOG(TRACE) << "Entering DocumentHost::DeleteCookie";
   // TODO: Optimize and return legitimate error conditions.

cpp/iedriver/DocumentHost.h

@@ -68,7 +68,6 @@ class DocumentHost {
 
   void GetCookies(std::vector<BrowserCookie>* cookies);
   int AddCookie(const std::string& cookie, const bool validate_document_type);
-  //int DeleteCookie(const std::string& cookie_name);
   int DeleteCookie(const BrowserCookie& cookie);
 
   int SetFocusedFrameByIndex(const int frame_index);