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

Side by Side 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, 5 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 unified diff | Download patch | Annotate | Revision Log
« 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 »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "ppapi/tests/test_network_proxy.h"
6
7 #include "ppapi/cpp/dev/network_proxy_dev.h"
8 #include "ppapi/cpp/instance.h"
9 #include "ppapi/cpp/var.h"
10 #include "ppapi/tests/testing_instance.h"
11
12 REGISTER_TEST_CASE(NetworkProxy);
13
14 TestNetworkProxy::TestNetworkProxy(TestingInstance* instance)
15 : TestCase(instance) {
16 }
17
18 void TestNetworkProxy::RunTests(const std::string& filter) {
19 RUN_CALLBACK_TEST(TestNetworkProxy, GetProxyForURL, filter);
20 }
21
22 std::string TestNetworkProxy::TestGetProxyForURL() {
23 TestCompletionCallbackWithOutput<pp::Var> callback(instance_->pp_instance(),
24 callback_type());
25 callback.WaitForResult(
26 pp::NetworkProxy::GetProxyForURL(instance_,
27 pp::Var("http://127.0.0.1/foobar/"),
28 callback.GetCallback()));
29 CHECK_CALLBACK_BEHAVIOR(callback);
30 ASSERT_EQ(PP_OK, callback.result());
31 pp::Var output = callback.output();
32 ASSERT_TRUE(output.is_string());
33 // Assume no one configures a proxy for localhost.
34 ASSERT_EQ("DIRECT", callback.output().AsString());
35
36 callback.WaitForResult(
37 pp::NetworkProxy::GetProxyForURL(instance_,
38 pp::Var("http://www.google.com"),
39 callback.GetCallback()));
40 CHECK_CALLBACK_BEHAVIOR(callback);
41 ASSERT_EQ(PP_OK, callback.result());
42 output = callback.output();
43 // Don't know what the proxy might be, but it should be a valid result.
44 ASSERT_TRUE(output.is_string());
45
46 callback.WaitForResult(
47 pp::NetworkProxy::GetProxyForURL(instance_,
48 pp::Var("file:///tmp"),
49 callback.GetCallback()));
50 CHECK_CALLBACK_BEHAVIOR(callback);
51 ASSERT_EQ(PP_OK, callback.result());
52 output = callback.output();
53 ASSERT_TRUE(output.is_string());
54 // Should get "DIRECT" for file:// URLs.
55 ASSERT_EQ("DIRECT", output.AsString());
56
57 callback.WaitForResult(
58 pp::NetworkProxy::GetProxyForURL(instance_,
59 pp::Var("this isn't a url"),
60 callback.GetCallback()));
61 CHECK_CALLBACK_BEHAVIOR(callback);
62 // TODO(dmichael): Use bad address when it's available.
63 ASSERT_EQ(PP_ERROR_BADARGUMENT, callback.result());
64 // TODO(dmichael): Add this check below when crbug.com/250046 is fixed.
65 // ASSERT_TRUE(callback.output().is_undefined());
66
67 callback.WaitForResult(
68 pp::NetworkProxy::GetProxyForURL(instance_,
69 pp::Var(42), // non-string Var
70 callback.GetCallback()));
71 CHECK_CALLBACK_BEHAVIOR(callback);
72 ASSERT_EQ(PP_ERROR_BADARGUMENT, callback.result());
73 // TODO(dmichael): Add this check below when crbug.com/250046 is fixed.
74 // ASSERT_TRUE(callback.output().is_undefined());
75
76 PASS();
77 }
OLDNEW
« 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