Implement 3 PIN browser tests

chrome/test/remoting/pin_browsertest.cc

+// Copyright 2014 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 "base/file_util.h"
+#include "base/files/file_path.h"
+#include "chrome/test/remoting/remote_desktop_browsertest.h"
+#include "chrome/test/remoting/waiter.h"
+
+namespace remoting {
+
+IN_PROC_BROWSER_TEST_F(RemoteDesktopBrowserTest, MANUAL_Cancel_PIN) {
+  SetUpTestForMe2Me();
+
+  content::WebContents* content = app_web_content();
+  LoadScript(content, FILE_PATH_LITERAL("browser_test.js"));

[Jamie, 2014/05/13 18:59:54]

Wasn't this previously done in the helper function? Is there any reason not to
do so?

[kelvinp, 2014/05/13 23:11:01]

Yes, because I thought including the script file is specific to me2me browser
test that is written in JavaScript.  But I guess it doesn't hurt to include
browser_test.js in the helper function as well.

+  LoadScript(content, FILE_PATH_LITERAL("cancel_pin_browser_test.js"));
+
+  RunJavaScriptTest(content, "Cancel_PIN", "{"
+    "pin: '" + me2me_pin() + "'"
+  "}");
+
+  Cleanup();
+}
+
+IN_PROC_BROWSER_TEST_F(RemoteDesktopBrowserTest, MANUAL_Invalid_PIN) {
+  SetUpTestForMe2Me();
+
+  content::WebContents* content = app_web_content();
+  LoadScript(content, FILE_PATH_LITERAL("browser_test.js"));
+  LoadScript(content, FILE_PATH_LITERAL("invalid_pin_browser_test.js"));
+
+  RunJavaScriptTest(content, "Invalid_PIN", "{"
+    "pin: '" + me2me_pin() + "'"
+  "}");
+
+  Cleanup();
+}
+
+IN_PROC_BROWSER_TEST_F(RemoteDesktopBrowserTest, MANUAL_Update_PIN) {
+  SetUpTestForMe2Me();
+
+  content::WebContents* content = app_web_content();
+  LoadScript(content, FILE_PATH_LITERAL("browser_test.js"));
+  LoadScript(content, FILE_PATH_LITERAL("update_pin_browser_test.js"));
+
+  RunJavaScriptTest(content, "Update_PIN", "{"
+    "old_pin: '" + me2me_pin() + "',"
+    "new_pin: '314159'"
+  "}");
+
+  Cleanup();
+}
+
+}  // namespace remoting