[chromedriver] Implement SwitchToFrame command.

chrome/test/chromedriver/commands.cc

 // Copyright (c) 2012 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/commands.h"
 
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/file_util.h"
 #include "base/format_macros.h"
@@ skipping 85 matching lines @@
     const base::DictionaryValue& params,
     scoped_ptr<base::Value>* value) {
   std::string script;
   if (!params.GetString("script", &script))
     return Status(kUnknownError, "'script' must be a string");
   const base::ListValue* args;
   if (!params.GetList("args", &args))
     return Status(kUnknownError, "'args' must be a list");
 
   return session->chrome->CallFunction(
-      "function(){" + script + "}", *args, value);
+      session->frame, "function(){" + script + "}", *args, value);
 }
+
+Status ExecuteSwitchToFrame(
+    Session* session,
+    const base::DictionaryValue& params,
+    scoped_ptr<base::Value>* value) {
+  const base::Value* id;
+  if (!params.Get("id", &id))
+    return Status(kUnknownError, "missing 'id'");
+
+  if (id->IsType(base::Value::TYPE_NULL)) {
+    session->frame = "";
+    return Status(kOk);
+  }
+
+  std::string script;
+  if (id->IsType(base::Value::TYPE_STRING)) {
+    script =
+        "function(arg) {"
+        "  var xpath = '(/html/body//iframe|/html/frameset/frame)';"
+        "  var sub = function(s) { return s.replace(/\\$/g, arg); };"
+        "  xpath += sub('[@name=\"$\" or @id=\"$\"]');"
+        "  return document.evaluate(xpath, document, null, "

[craigdh, 2012/12/20 20:03:38]

Constructing these xpaths in C++ would reduce the injected Javascript down to a
single method invocation. In general I think it's worth keeping as much logic as
possible in C++, debugging generated code sent to an OOP interpreter is less
than ideal. Thoughts?

[kkania, 2013/01/08 18:30:11]

I've gone and ahead and switched to C++ since it makes it a bit simpler.
Sometimes, if it is easier to do more in JS and it can be tested better, I think
its fine doing it in JS.

+        "      XPathResult.FIRST_ORDERED_NODE_TYPE, null).singleNodeValue;"
+        "}";
+  } else if (id->IsType(base::Value::TYPE_INTEGER)) {
+    script =
+        "function(index) {"
+        "  var xpathIndex = '[' + (index + 1) + ']';"
+        "  var xpath = '(/html/body//iframe|/html/frameset/frame)' + "
+        "              xpathIndex;"
+        "  return document.evaluate(xpath, document, null, "
+        "      XPathResult.FIRST_ORDERED_NODE_TYPE, null).singleNodeValue;"
+        "}";
+  } else if (id->IsType(base::Value::TYPE_DICTIONARY)) {
+    // TODO(kkania): Implement.
+    return Status(kUnknownError, "frame switching by element not implemented");
+  } else {
+    return Status(kUnknownError, "invalid 'id'");
+  }
+  base::ListValue args;
+  args.Append(id->DeepCopy());
+  std::string frame;
+  Status status = session->chrome->GetFrameByFunction(
+      session->frame, script, args, &frame);
+  if (status.IsError())
+    return status;
+  session->frame = frame;
+  return Status(kOk);
+}