[chromedriver] Add a sanity check during session initiation

chrome/test/chromedriver/session_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/session_commands.h"
 
 #include <list>
 
 #include "base/bind.h"
 #include "base/callback.h"
@@ skipping 87 matching lines @@
   if (status.IsOk()) {
     chrome_caps->SetString(
         "userDataDir",
         desktop->command().GetSwitchValueNative("user-data-dir"));
   }
 
   caps->Set("chrome", chrome_caps.release());
   return caps.Pass();
 }
 
+Status CheckSessionCreated(Session* session) {
+  WebView* web_view = NULL;
+  Status status = session->GetTargetWindow(&web_view);
+  if (status.IsError())
+    return Status(kSessionNotCreatedException, status);
+
+  status = web_view->ConnectIfNecessary();
+  if (status.IsError())
+    return Status(kSessionNotCreatedException, status);
+
+  base::ListValue args;
+  scoped_ptr<base::Value> result(new base::FundamentalValue(0));
+  status = web_view->CallFunction(session->GetCurrentFrameId(),
+                                  "function(s) { return 1; }", args, &result);
+  if (status.IsError())
+    return Status(kSessionNotCreatedException, status);
+
+  int response;
+  if (!result->GetAsInteger(&response) || response != 1) {
+    return Status(kSessionNotCreatedException,
+                  "unexpected response from browser");
+  }
+
+  return Status(kOk);
+}
+
 Status InitSessionHelper(
     const InitSessionParams& bound_params,
     Session* session,
     const base::DictionaryValue& params,
     scoped_ptr<base::Value>* value) {
   session->driver_log.reset(
       new WebDriverLog(WebDriverLog::kDriverType, Log::kAll));
   const base::DictionaryValue* desired_caps;
   if (!params.GetDictionary("desiredCapabilities", &desired_caps))
     return Status(kUnknownError, "cannot find dict 'desiredCapabilities'");
@@ skipping 40 matching lines @@
   if (status.IsError() || web_view_ids.empty()) {
     return status.IsError() ? status :
         Status(kUnknownError, "unable to discover open window in chrome");
   }
 
   session->window = web_view_ids.front();
   session->detach = capabilities.detach;
   session->force_devtools_screenshot = capabilities.force_devtools_screenshot;
   session->capabilities = CreateCapabilities(session->chrome.get());
   value->reset(session->capabilities->DeepCopy());
-  return Status(kOk);
+  return CheckSessionCreated(session);
 }
 
 }  // namespace
 
 Status ExecuteInitSession(
     const InitSessionParams& bound_params,
     Session* session,
     const base::DictionaryValue& params,
     scoped_ptr<base::Value>* value) {
   Status status = InitSessionHelper(bound_params, session, params, value);
@@ skipping 469 matching lines @@
 Status ExecuteSetAutoReporting(
     Session* session,
     const base::DictionaryValue& params,
     scoped_ptr<base::Value>* value) {
   bool enabled;
   if (!params.GetBoolean("enabled", &enabled))
     return Status(kUnknownError, "missing parameter 'enabled'");
   session->auto_reporting_enabled = enabled;
   return Status(kOk);
 }