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

Unified Diff: chrome/browser/chromeos/arc/policy/arc_policy_bridge_unittest.cc

Issue 2495913002: arc: Convert more Mojo types to STL (Closed)
Patch Set: Rebased to ToT Created 4 years, 1 month 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
Index: chrome/browser/chromeos/arc/policy/arc_policy_bridge_unittest.cc
diff --git a/chrome/browser/chromeos/arc/policy/arc_policy_bridge_unittest.cc b/chrome/browser/chromeos/arc/policy/arc_policy_bridge_unittest.cc
index d280c3548bbea0f5bc884c4c8a6eea9605cc76eb..3906042581f8299c95853f77d8991689aa776cfa 100644
--- a/chrome/browser/chromeos/arc/policy/arc_policy_bridge_unittest.cc
+++ b/chrome/browser/chromeos/arc/policy/arc_policy_bridge_unittest.cc
@@ -23,7 +23,7 @@
namespace {
-char kFakeONC[] =
+constexpr char kFakeONC[] =
"{\"NetworkConfigurations\":["
"{\"GUID\":\"{485d6076-dd44-6b6d-69787465725f5040}\","
"\"Type\":\"WiFi\","
@@ -68,17 +68,16 @@ class CheckedBoolean {
};
void ExpectPolicyString(std::unique_ptr<CheckedBoolean> was_run,
- mojo::String expected,
- mojo::String policies) {
+ const std::string& expected,
+ const std::string& policies) {
EXPECT_EQ(expected, policies);
was_run->set_value(true);
}
arc::ArcPolicyBridge::GetPoliciesCallback PolicyStringCallback(
- mojo::String expected) {
- std::unique_ptr<CheckedBoolean> was_run(new CheckedBoolean);
- return base::Bind(&ExpectPolicyString, base::Passed(&was_run),
- base::Passed(&expected));
+ const std::string& expected) {
+ std::unique_ptr<CheckedBoolean> was_run(new CheckedBoolean());
+ return base::Bind(&ExpectPolicyString, base::Passed(&was_run), expected);
}
} // namespace
« no previous file with comments | « chrome/browser/chromeos/arc/policy/arc_policy_bridge.cc ('k') | chrome/browser/chromeos/arc/process/arc_process_service.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698