Toggle a multiple select option instead of setting it as selected when

chrome/test/webdriver/commands/webelement_commands.cc

Index: chrome/test/webdriver/commands/webelement_commands.cc
diff --git a/chrome/test/webdriver/commands/webelement_commands.cc b/chrome/test/webdriver/commands/webelement_commands.cc
index 561bd5303764624a9103eca125c58416985e9a32..1b12c1178eadc598d6d9a9f3b03ad9aa2a5e3c0c 100644
--- a/chrome/test/webdriver/commands/webelement_commands.cc
+++ b/chrome/test/webdriver/commands/webelement_commands.cc
@@ -334,24 +334,19 @@ bool ElementSelectedCommand::DoesPost() {
 }
 
 void ElementSelectedCommand::ExecuteGet(Response* const response) {
-  ListValue args;
-  args.Append(element.ToValue());
-
-  std::string script = base::StringPrintf(
-      "return (%s).apply(null, arguments);", atoms::IS_SELECTED);
-
-  Value* result = NULL;
-  Error* error = session_->ExecuteScript(script, &args, &result);
+  bool is_selected;
+  Error* error =session_->IsOptionElementSelected(

[Huyen, 2011/08/08 21:05:21]

add space after "="

+      session_->current_target(), element, &is_selected);
   if (error) {
     response->SetError(error);
     return;
   }
-  response->SetValue(result);
+  response->SetValue(Value::CreateBooleanValue(is_selected));
 }
 
 void ElementSelectedCommand::ExecutePost(Response* const response) {
-  Error* error = session_->SelectOptionElement(
-      session_->current_target(), element);
+  Error* error = session_->SetOptionElementSelected(
+      session_->current_target(), element, true);
   if (error) {
     response->SetError(error);
     return;