[chromedriver] Add an error autoreporting feature that automatically raises errors from browser logs

chrome/test/chromedriver/session_commands.cc

Index: chrome/test/chromedriver/session_commands.cc
diff --git a/chrome/test/chromedriver/session_commands.cc b/chrome/test/chromedriver/session_commands.cc
index a80aa46baeb0f6f3be972bdec16fc25b28bb8800..ea51ef8e884173152d58062b3a257e7ec7ac5a2a 100644
--- a/chrome/test/chromedriver/session_commands.cc
+++ b/chrome/test/chromedriver/session_commands.cc
@@ -607,3 +607,22 @@ Status ExecuteUploadFile(
   value->reset(new base::StringValue(upload.value()));
   return Status(kOk);
 }
+
+Status ExecuteIsAutoReportingEnabled(
+    Session* session,
+    const base::DictionaryValue& params,
+    scoped_ptr<base::Value>* value) {
+  value->reset(new base::FundamentalValue(session->auto_reporting_enabled));
+  return Status(kOk);
+}
+
+Status ExecuteSetAutoReportingEnabled(
+    Session* session,
+    const base::DictionaryValue& params,
+    scoped_ptr<base::Value>* value) {
+  bool enable;
+  if (!params.GetBoolean("enable", &enable))
+    return Status(kUnknownError, "missing parameter 'enable'");
+  session->auto_reporting_enabled = enable;
+  return ExecuteIsAutoReportingEnabled(session, params, value);

[chrisgao (Use stgao instead), 2014/01/09 22:11:55]

Hmm, do we want to return the new value? It seems there is no such convention
for webdriver api.

[samuong, 2014/01/16 00:29:28]

I thought it might be useful for clients but you're right that the convention
does not exist, so I won't do this for consistency with the rest of the
WebDriver API.

+}