[chromedriver] Fix bug in command for GetElementValueOfCSSProperty.

chrome/test/chromedriver/element_commands.cc

 // Copyright (c) 2013 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/chromedriver/element_commands.h"
 
 #include <list>
 #include <vector>
 
 #include "base/callback.h"
@@ skipping 395 matching lines @@
     return Status(kUnknownError, "missing 'name'");
   return GetElementAttribute(session, web_view, element_id, name, value);
 }
 
 Status ExecuteGetElementValueOfCSSProperty(
     Session* session,
     WebView* web_view,
     const std::string& element_id,
     const base::DictionaryValue& params,
     scoped_ptr<base::Value>* value) {
-  base::ListValue args;
-  args.Append(CreateElement(element_id));
-  return web_view->CallFunction(
-      session->GetCurrentFrameId(),
-      webdriver::atoms::asString(webdriver::atoms::GET_EFFECTIVE_STYLE),
-      args,
-      value);
+  std::string property_name;
+  if (!params.GetString("propertyName", &property_name))
+    return Status(kUnknownError, "missing 'name'");

[kkania, 2013/04/18 00:38:52]

propertyName

[chrisgao (Use stgao instead), 2013/04/18 01:39:48]

Done.

+  std::string property_value;
+  Status status = GetElementEffectiveStyle(
+      session->GetCurrentFrameId(), web_view, element_id,
+      property_name, &property_value);
+  if (status.IsError())
+    return status;
+  value->reset(new base::StringValue(property_value));
+  return Status(kOk);
 }
 
 Status ExecuteElementEquals(
     Session* session,
     WebView* web_view,
     const std::string& element_id,
     const base::DictionaryValue& params,
     scoped_ptr<base::Value>* value) {
   std::string other_element_id;
   if (!params.GetString("other", &other_element_id))
     return Status(kUnknownError, "'other' must be a string");
   value->reset(new base::FundamentalValue(element_id == other_element_id));
   return Status(kOk);
 }