Refactor chromedriver's script execution to reduce amount of custom Value parsing.

chrome/test/webdriver/web_element_id.h

Index: chrome/test/webdriver/web_element_id.h
diff --git a/chrome/test/webdriver/web_element_id.h b/chrome/test/webdriver/web_element_id.h
index 037f6335fc3fd673ba042e07118939cc475c3df3..05716f5eef3080aff9aaa4f8a978b26e5a230b27 100644
--- a/chrome/test/webdriver/web_element_id.h
+++ b/chrome/test/webdriver/web_element_id.h
@@ -8,6 +8,8 @@
 
 #include <string>
 
+#include "chrome/test/automation/value_conversion_traits.h"
+
 namespace base {
 class Value;
 }
@@ -27,7 +29,7 @@ class WebElementId {
 
   // Creates a |WebElementId| from an element dictionary returned by a WebDriver
   // atom. It will be valid iff the dictionary is correctly constructed.
-  explicit WebElementId(base::Value* value);
+  explicit WebElementId(const base::Value* value);
 
   ~WebElementId();
 
@@ -60,4 +62,12 @@ struct LocatorType {
 
 }  // namespace webdriver
 
+template <>
+struct ValueConversionTraits<webdriver::WebElementId> {

[Paweł Hajdan Jr., 2011/08/02 20:04:47]

Why here? I think you may risk different behavior depending on whether this
header is included or not.

[kkania, 2011/08/03 17:07:56]

I was not quite sure where to put it myself. I thought it would be best to keep
it next to the thing it is converting though. The ValueConversionTraits are
defined in test_support_common, while this is only used in chromedriver. I
didn't want to pull chromedriver stuff into there.

If this header is not included and you try to convert a WebElementId, the code
won't compile. Plus, if you are going to convert to a WebElementId, you should
include the header in the file by the include what you use rule.

[Paweł Hajdan Jr., 2011/08/04 17:51:36]

After thinking more about it I still think it'd be safer to have all those
template instantiations in a single header file. If you have an example of the
code, docs, or an opinion of C++ expert that would suggest otherwise, I'm open
to change that opinion.

+  static base::Value* CreateValueFrom(const webdriver::WebElementId& t);
+  static bool SetFromValue(
+      const base::Value* value, webdriver::WebElementId* t);
+  static bool CanConvert(const base::Value* value);
+};
+
 #endif  // CHROME_TEST_WEBDRIVER_WEB_ELEMENT_ID_H_