Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1135)

Unified Diff: ppapi/tests/test_network_proxy.cc

Issue 16819002: PPAPI: Introduce PPB_NetworkProxy_Dev (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: merge Created 7 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « ppapi/tests/test_network_proxy.h ('k') | ppapi/thunk/interfaces_ppb_public_dev.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ppapi/tests/test_network_proxy.cc
diff --git a/ppapi/tests/test_network_proxy.cc b/ppapi/tests/test_network_proxy.cc
new file mode 100644
index 0000000000000000000000000000000000000000..4eb41498f6d33d4df408169f47b32aeb8fe81e62
--- /dev/null
+++ b/ppapi/tests/test_network_proxy.cc
@@ -0,0 +1,77 @@
+// 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 "ppapi/tests/test_network_proxy.h"
+
+#include "ppapi/cpp/dev/network_proxy_dev.h"
+#include "ppapi/cpp/instance.h"
+#include "ppapi/cpp/var.h"
+#include "ppapi/tests/testing_instance.h"
+
+REGISTER_TEST_CASE(NetworkProxy);
+
+TestNetworkProxy::TestNetworkProxy(TestingInstance* instance)
+ : TestCase(instance) {
+}
+
+void TestNetworkProxy::RunTests(const std::string& filter) {
+ RUN_CALLBACK_TEST(TestNetworkProxy, GetProxyForURL, filter);
+}
+
+std::string TestNetworkProxy::TestGetProxyForURL() {
+ TestCompletionCallbackWithOutput<pp::Var> callback(instance_->pp_instance(),
+ callback_type());
+ callback.WaitForResult(
+ pp::NetworkProxy::GetProxyForURL(instance_,
+ pp::Var("http://127.0.0.1/foobar/"),
+ callback.GetCallback()));
+ CHECK_CALLBACK_BEHAVIOR(callback);
+ ASSERT_EQ(PP_OK, callback.result());
+ pp::Var output = callback.output();
+ ASSERT_TRUE(output.is_string());
+ // Assume no one configures a proxy for localhost.
+ ASSERT_EQ("DIRECT", callback.output().AsString());
+
+ callback.WaitForResult(
+ pp::NetworkProxy::GetProxyForURL(instance_,
+ pp::Var("http://www.google.com"),
+ callback.GetCallback()));
+ CHECK_CALLBACK_BEHAVIOR(callback);
+ ASSERT_EQ(PP_OK, callback.result());
+ output = callback.output();
+ // Don't know what the proxy might be, but it should be a valid result.
+ ASSERT_TRUE(output.is_string());
+
+ callback.WaitForResult(
+ pp::NetworkProxy::GetProxyForURL(instance_,
+ pp::Var("file:///tmp"),
+ callback.GetCallback()));
+ CHECK_CALLBACK_BEHAVIOR(callback);
+ ASSERT_EQ(PP_OK, callback.result());
+ output = callback.output();
+ ASSERT_TRUE(output.is_string());
+ // Should get "DIRECT" for file:// URLs.
+ ASSERT_EQ("DIRECT", output.AsString());
+
+ callback.WaitForResult(
+ pp::NetworkProxy::GetProxyForURL(instance_,
+ pp::Var("this isn't a url"),
+ callback.GetCallback()));
+ CHECK_CALLBACK_BEHAVIOR(callback);
+ // TODO(dmichael): Use bad address when it's available.
+ ASSERT_EQ(PP_ERROR_BADARGUMENT, callback.result());
+ // TODO(dmichael): Add this check below when crbug.com/250046 is fixed.
+ // ASSERT_TRUE(callback.output().is_undefined());
+
+ callback.WaitForResult(
+ pp::NetworkProxy::GetProxyForURL(instance_,
+ pp::Var(42), // non-string Var
+ callback.GetCallback()));
+ CHECK_CALLBACK_BEHAVIOR(callback);
+ ASSERT_EQ(PP_ERROR_BADARGUMENT, callback.result());
+ // TODO(dmichael): Add this check below when crbug.com/250046 is fixed.
+ // ASSERT_TRUE(callback.output().is_undefined());
+
+ PASS();
+}
« no previous file with comments | « ppapi/tests/test_network_proxy.h ('k') | ppapi/thunk/interfaces_ppb_public_dev.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698