Use WebIDBKeyPath type in WebKit API, implement IndexedDBKeyPath type.

content/common/indexed_db/indexed_db_key_path.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 "content/common/indexed_db/indexed_db_key_path.h"
+
+#include "base/logging.h"
+#include "third_party/WebKit/Source/WebKit/chromium/public/platform/WebString.h"
+#include "third_party/WebKit/Source/WebKit/chromium/public/platform/WebVector.h"
+
+using WebKit::WebIDBKeyPath;
+using WebKit::WebString;
+using WebKit::WebVector;
+
+IndexedDBKeyPath::IndexedDBKeyPath()
+    : type_(WebIDBKeyPath::NullType) {
+}
+
+IndexedDBKeyPath::IndexedDBKeyPath(const WebIDBKeyPath& key) {
+  Set(key);
+}
+
+IndexedDBKeyPath::~IndexedDBKeyPath() {
+}
+
+void IndexedDBKeyPath::SetNull() {
+  type_ = WebIDBKeyPath::NullType;
+}
+
+void IndexedDBKeyPath::SetArray(const std::vector<string16>& array) {
+  type_ = WebIDBKeyPath::ArrayType;
+  array_ = array;
+}
+
+void IndexedDBKeyPath::SetString(const string16& string) {
+  type_ = WebIDBKeyPath::StringType;
+  string_ = string;
+}
+
+void IndexedDBKeyPath::Set(const WebIDBKeyPath& keyPath) {
+  type_ = keyPath.type();
+  array_.clear();
+  if (keyPath.type() == WebIDBKeyPath::ArrayType) {

[michaeln, 2012/04/24 21:08:53]

nit: might be more readable as a switch statement or
arr.clear(); str.clear(); if (arrType) {} else if (strType) {}

[jsbell, 2012/04/25 21:49:46]

Done.

+    WebVector<WebString> array = keyPath.array();
+    for (size_t i = 0; i < array.size(); ++i) {
+      array_.push_back(static_cast<string16>(array[i]));
+    }
+  }
+  string_ = keyPath.type() == WebIDBKeyPath::StringType ?
+      static_cast<string16>(keyPath.string()) : string16();
+}
+
+bool IndexedDBKeyPath::IsValid() const {
+  WebIDBKeyPath key_path = *this;
+  return key_path.isValid();
+}
+
+IndexedDBKeyPath::operator WebIDBKeyPath() const {
+  switch (type_) {
+    case WebIDBKeyPath::ArrayType:
+      return WebIDBKeyPath::create(array_);
+    case WebIDBKeyPath::StringType:
+      return WebIDBKeyPath::create(static_cast<WebString>(string_));

[michaeln, 2012/04/24 21:08:53]

Is this static_cast correct/needed?

Looks like you need to construct a real WebString here, and that this ctor will
do that automagically for you, WebString(const string16& s). Seems like you just
need WebIDBKeyPath::create(string_) and if that doesn't work
::create(WebString(string_)) could be a better choice.

[jsbell, 2012/04/25 21:49:46]

Done.

+    case WebIDBKeyPath::NullType:
+      return WebIDBKeyPath::createNull();
+  }
+  NOTREACHED();
+  return WebIDBKeyPath::createNull();
+}