Add Mojo types and service definitions for Background Sync API

content/child/background_sync/background_sync_type_converters_unittest.cc

Index: content/child/background_sync/background_sync_type_converters_unittest.cc
diff --git a/content/child/background_sync/background_sync_type_converters_unittest.cc b/content/child/background_sync/background_sync_type_converters_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..d1fd2b193f64dce1e386e17bf674d2a500bd8ee4
--- /dev/null
+++ b/content/child/background_sync/background_sync_type_converters_unittest.cc
@@ -0,0 +1,172 @@
+// Copyright 2015 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 "content/child/background_sync/background_sync_type_converters.h"

[jkarlin, 2015/04/27 18:25:11]

blank line after this include

[iclelland, 2015/04/28 12:41:30]

Done.

+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace mojo {
+namespace test {

[jkarlin, 2015/04/27 18:25:11]

is this namespace used for anything?

[iclelland, 2015/04/28 12:41:30]

Not by me, but it was a pattern I'd seen in mojo tests
(/src/mojo/shell/url_resolver_unittest.cc,
src/mojo/common/common_type_converters_unittest.cc), so I cargo-culted it in
there.
Removed, as unit tests still run fine without.
Done.

+namespace {
+
+TEST(BackgroundSyncTypeConverterTest, TestBlinkToMojoPeriodicityConversions) {
+  ASSERT_EQ(blink::WebSyncRegistration::PeriodicityPeriodic,
+      ConvertTo<blink::WebSyncRegistration::Periodicity>(
+          content::SYNC_PERIODICITY_Periodic));
+  ASSERT_EQ(blink::WebSyncRegistration::PeriodicityOneShot,
+      ConvertTo<blink::WebSyncRegistration::Periodicity>(
+          content::SYNC_PERIODICITY_OneShot));
+}
+
+TEST(BackgroundSyncTypeConverterTest, TestMojoToBlinkPeriodicityConversions) {
+  ASSERT_EQ(content::SYNC_PERIODICITY_Periodic,
+      ConvertTo<content::SyncPeriodicity>(
+          blink::WebSyncRegistration::PeriodicityPeriodic));
+  ASSERT_EQ(content::SYNC_PERIODICITY_OneShot,
+      ConvertTo<content::SyncPeriodicity>(
+          blink::WebSyncRegistration::PeriodicityOneShot));
+}
+
+TEST(BackgroundSyncTypeConverterTest, TestBlinkToMojoNetworkStateConversions) {
+  ASSERT_EQ(blink::WebSyncRegistration::NetworkStateAny,
+      ConvertTo<blink::WebSyncRegistration::NetworkState>(
+          content::SYNC_NETWORK_STATE_Any));
+  ASSERT_EQ(blink::WebSyncRegistration::NetworkStateAvoidCellular,
+      ConvertTo<blink::WebSyncRegistration::NetworkState>(
+          content::SYNC_NETWORK_STATE_AvoidCellular));
+  ASSERT_EQ(blink::WebSyncRegistration::NetworkStateOnline,
+      ConvertTo<blink::WebSyncRegistration::NetworkState>(
+          content::SYNC_NETWORK_STATE_Online));
+}
+
+TEST(BackgroundSyncTypeConverterTest, TestMojoToBlinkNetworkStateConversions) {
+  ASSERT_EQ(content::SYNC_NETWORK_STATE_Any,
+      ConvertTo<content::SyncNetworkState>(
+          blink::WebSyncRegistration::NetworkStateAny));
+  ASSERT_EQ(content::SYNC_NETWORK_STATE_AvoidCellular,
+      ConvertTo<content::SyncNetworkState>(
+          blink::WebSyncRegistration::NetworkStateAvoidCellular));
+  ASSERT_EQ(content::SYNC_NETWORK_STATE_Online,
+      ConvertTo<content::SyncNetworkState>(
+          blink::WebSyncRegistration::NetworkStateOnline));
+}
+
+TEST(BackgroundSyncTypeConverterTest, TestBlinkToMojoPowerStateConversions) {
+  ASSERT_EQ(blink::WebSyncRegistration::PowerStateAuto,
+      ConvertTo<blink::WebSyncRegistration::PowerState>(
+          content::SYNC_POWER_STATE_Auto));
+  ASSERT_EQ(blink::WebSyncRegistration::PowerStateAvoidDraining,
+      ConvertTo<blink::WebSyncRegistration::PowerState>(
+          content::SYNC_POWER_STATE_AvoidDraining));
+}
+
+TEST(BackgroundSyncTypeConverterTest, TestMojoToBlinkPowerStateConversions) {
+  ASSERT_EQ(content::SYNC_POWER_STATE_Auto,
+      ConvertTo<content::SyncPowerState>(
+          blink::WebSyncRegistration::PowerStateAuto));
+  ASSERT_EQ(content::SYNC_POWER_STATE_AvoidDraining,
+      ConvertTo<content::SyncPowerState>(
+          blink::WebSyncRegistration::PowerStateAvoidDraining));
+}
+
+TEST(BackgroundSyncTypeConverterTest, TestDefaultBlinkToMojoConversion) {
+  blink::WebSyncRegistration in;
+  content::SyncRegistrationPtr out =
+      ConvertTo<content::SyncRegistrationPtr>(in);
+
+  ASSERT_EQ(blink::WebSyncRegistration::UNREGISTERED_SYNC_ID, out->id);
+  ASSERT_EQ(content::SYNC_PERIODICITY_OneShot, out->periodicity);
+  ASSERT_EQ("", out->tag);
+  ASSERT_EQ(0UL, out->minPeriodMs);
+  ASSERT_EQ(content::SYNC_NETWORK_STATE_Online, out->networkState);
+  ASSERT_EQ(content::SYNC_POWER_STATE_Auto, out->powerState);
+}
+
+TEST(BackgroundSyncTypeConverterTest, TestFullPeriodicBlinkToMojoConversion) {
+  blink::WebSyncRegistration in(
+      7, blink::WebSyncRegistration::PeriodicityPeriodic, "BlinkToMojo",
+      12340000, blink::WebSyncRegistration::NetworkStateAvoidCellular,
+      blink::WebSyncRegistration::PowerStateAvoidDraining);
+  content::SyncRegistrationPtr out =
+      ConvertTo<content::SyncRegistrationPtr>(in);
+
+  ASSERT_EQ(7, out->id);
+  ASSERT_EQ(content::SYNC_PERIODICITY_Periodic, out->periodicity);
+  ASSERT_EQ("BlinkToMojo", out->tag);
+  ASSERT_EQ(12340000UL, out->minPeriodMs);
+  ASSERT_EQ(content::SYNC_NETWORK_STATE_AvoidCellular, out->networkState);
+  ASSERT_EQ(content::SYNC_POWER_STATE_AvoidDraining, out->powerState);
+}
+
+TEST(BackgroundSyncTypeConverterTest, TestFullOneShotBlinkToMojoConversion) {
+  blink::WebSyncRegistration in(
+      7, blink::WebSyncRegistration::PeriodicityOneShot, "BlinkToMojo",
+      12340000, blink::WebSyncRegistration::NetworkStateAvoidCellular,
+      blink::WebSyncRegistration::PowerStateAvoidDraining);
+  content::SyncRegistrationPtr out =
+      ConvertTo<content::SyncRegistrationPtr>(in);
+
+  ASSERT_EQ(7, out->id);
+  ASSERT_EQ(content::SYNC_PERIODICITY_OneShot, out->periodicity);
+  ASSERT_EQ("BlinkToMojo", out->tag);
+  ASSERT_EQ(12340000UL, out->minPeriodMs);
+  ASSERT_EQ(content::SYNC_NETWORK_STATE_AvoidCellular, out->networkState);
+  ASSERT_EQ(content::SYNC_POWER_STATE_AvoidDraining, out->powerState);
+}
+
+TEST(BackgroundSyncTypeConverterTest, TestDefaultMojoToBlinkConversion) {
+  content::SyncRegistrationPtr in(content::SyncRegistration::New());
+  blink::WebSyncRegistration* out = ConvertTo<blink::WebSyncRegistration*>(in);
+
+  ASSERT_EQ(blink::WebSyncRegistration::UNREGISTERED_SYNC_ID, out->id);
+  ASSERT_EQ(blink::WebSyncRegistration::PeriodicityOneShot, out->periodicity);
+  ASSERT_EQ("",out->tag);
+  ASSERT_EQ(0UL, out->minPeriodMs);
+  ASSERT_EQ(blink::WebSyncRegistration::NetworkStateOnline, out->networkState);
+  ASSERT_EQ(blink::WebSyncRegistration::PowerStateAuto, out->powerState);
+}
+
+TEST(BackgroundSyncTypeConverterTest, TestFullPeriodicMojoToBlinkConversion) {
+  content::SyncRegistrationPtr in(content::SyncRegistration::New());
+  in->id = 41;
+  in->periodicity = content::SYNC_PERIODICITY_Periodic;
+  in->tag = mojo::String("MojoToBlink");
+  in->minPeriodMs = 43210000;
+  in->networkState = content::SYNC_NETWORK_STATE_AvoidCellular;
+  in->powerState = content::SYNC_POWER_STATE_AvoidDraining;
+  blink::WebSyncRegistration* out = ConvertTo<blink::WebSyncRegistration*>(in);

[jkarlin, 2015/04/27 18:25:11]

out is leaked

[iclelland, 2015/04/28 12:41:29]

Fixed with scoped_ptr.

+
+  ASSERT_EQ(41, out->id);
+  ASSERT_EQ(blink::WebSyncRegistration::PeriodicityPeriodic, out->periodicity);
+  ASSERT_EQ("MojoToBlink", out->tag.utf8());
+  ASSERT_EQ(43210000UL, out->minPeriodMs);
+  ASSERT_EQ(blink::WebSyncRegistration::NetworkStateAvoidCellular,
+            out->networkState);
+  ASSERT_EQ(blink::WebSyncRegistration::PowerStateAvoidDraining,
+            out->powerState);
+}
+
+TEST(BackgroundSyncTypeConverterTest, TestFullOneShotMojoToBlinkConversion) {
+  content::SyncRegistrationPtr in(content::SyncRegistration::New());
+  in->id = 41;
+  in->periodicity = content::SYNC_PERIODICITY_OneShot;
+  in->tag = mojo::String("MojoToBlink");
+  in->minPeriodMs = 43210000;
+  in->networkState = content::SYNC_NETWORK_STATE_AvoidCellular;
+  in->powerState = content::SYNC_POWER_STATE_AvoidDraining;
+  blink::WebSyncRegistration* out = ConvertTo<blink::WebSyncRegistration*>(in);

[jkarlin, 2015/04/27 18:25:11]

out is leaked

[iclelland, 2015/04/28 12:41:30]

Fixed with scoped_ptr.

+
+  ASSERT_EQ(41, out->id);
+  ASSERT_EQ(blink::WebSyncRegistration::PeriodicityOneShot, out->periodicity);
+  ASSERT_EQ("MojoToBlink", out->tag.utf8());
+  ASSERT_EQ(43210000UL, out->minPeriodMs);
+  ASSERT_EQ(blink::WebSyncRegistration::NetworkStateAvoidCellular,
+            out->networkState);
+  ASSERT_EQ(blink::WebSyncRegistration::PowerStateAvoidDraining,
+            out->powerState);
+}
+
+}

[jkarlin, 2015/04/27 18:25:11]

add namespace comments to these closing braces

[iclelland, 2015/04/28 12:41:29]

Done.

+}
+}
+