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

Unified Diff: dbus/end_to_end_async_unittest.cc

Issue 1559873005: dbus: Use randomly generated string as a TestService's service name (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 11 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 | « no previous file | dbus/end_to_end_sync_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: dbus/end_to_end_async_unittest.cc
diff --git a/dbus/end_to_end_async_unittest.cc b/dbus/end_to_end_async_unittest.cc
index 121c6609d8b135d8cdcfb7fdaadef4ce139bd0f4..8f628649d6fc7782f3683d89dd97c761ed564b62 100644
--- a/dbus/end_to_end_async_unittest.cc
+++ b/dbus/end_to_end_async_unittest.cc
@@ -63,7 +63,7 @@ class EndToEndAsyncTest : public testing::Test {
bus_options.dbus_task_runner = dbus_thread_->task_runner();
bus_ = new Bus(bus_options);
object_proxy_ = bus_->GetObjectProxy(
- "org.chromium.TestService",
+ test_service_->service_name(),
ObjectPath("/org/chromium/TestObject"));
ASSERT_TRUE(bus_->HasDBusThread());
@@ -97,7 +97,7 @@ class EndToEndAsyncTest : public testing::Test {
run_loop_->Run();
// Create a second object proxy for the root object.
- root_object_proxy_ = bus_->GetObjectProxy("org.chromium.TestService",
+ root_object_proxy_ = bus_->GetObjectProxy(test_service_->service_name(),
ObjectPath("/"));
ASSERT_TRUE(bus_->HasDBusThread());
@@ -147,7 +147,7 @@ class EndToEndAsyncTest : public testing::Test {
// Create new object proxy.
object_proxy_ = bus_->GetObjectProxy(
- "org.chromium.TestService",
+ test_service_->service_name(),
ObjectPath("/org/chromium/TestObject"));
}
@@ -435,7 +435,7 @@ TEST_F(EndToEndAsyncTest, CancelPendingCalls) {
// Remove the object proxy before receiving the result.
// This results in cancelling the pending method call.
- bus_->RemoveObjectProxy("org.chromium.TestService",
+ bus_->RemoveObjectProxy(test_service_->service_name(),
ObjectPath("/org/chromium/TestObject"),
base::Bind(&base::DoNothing));
@@ -517,7 +517,7 @@ TEST_F(EndToEndAsyncTest, InvalidObjectPath) {
const ObjectPath invalid_object_path("/org/chromium/TestObject/");
// Replace object proxy with new one.
- object_proxy_ = bus_->GetObjectProxy("org.chromium.TestService",
+ object_proxy_ = bus_->GetObjectProxy(test_service_->service_name(),
invalid_object_path);
MethodCall method_call("org.chromium.TestInterface", "Echo");
« no previous file with comments | « no previous file | dbus/end_to_end_sync_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698