[Background Sync] Sent sync registration details to worker

content/browser/background_sync/background_sync_registration.h

Index: content/browser/background_sync/background_sync_registration.h
diff --git a/content/browser/background_sync/background_sync_registration.h b/content/browser/background_sync/background_sync_registration.h
index 9d40e354fd99fdd244f610ca477e1a30c36bf487..94d764811eb6fc2136fa96e1e71f78894af5067d 100644
--- a/content/browser/background_sync/background_sync_registration.h
+++ b/content/browser/background_sync/background_sync_registration.h
@@ -5,9 +5,12 @@
 #ifndef CONTENT_BROWSER_BACKGROUND_SYNC_BACKGROUND_SYNC_REGISTRATION_H_
 #define CONTENT_BROWSER_BACKGROUND_SYNC_BACKGROUND_SYNC_REGISTRATION_H_
 
+#include "base/memory/scoped_ptr.h"
 #include "content/browser/background_sync/background_sync.pb.h"
 #include "content/browser/background_sync/background_sync_registration_options.h"
+#include "content/common/background_sync_service.mojom.h"
 #include "content/common/content_export.h"
+#include "third_party/mojo/src/mojo/public/cpp/bindings/type_converter.h"
 
 namespace content {
 
@@ -39,4 +42,23 @@ class CONTENT_EXPORT BackgroundSyncRegistration {
 
 }  // namespace content
 
+namespace mojo {

[jkarlin, 2015/07/17 18:07:22]

Not sure if it's kosher to have multiple namespaces like this. Are there other
examples?

[iclelland, 2015/07/17 18:34:25]

The style guide is mostly silent on the issue, and it'll take more than a regex
to find an example, if there is one... or I could try to find the kosher police
around here.

[iclelland, 2015/07/17 18:56:35]

So I found these examples of this exact thing:
 
https://code.google.com/p/chromium/codesearch#chromium/src/extensions/browser...
 
https://code.google.com/p/chromium/codesearch#chromium/src/extensions/browser...
 
https://code.google.com/p/chromium/codesearch#chromium/src/components/html_vi...
 
https://code.google.com/p/chromium/codesearch#chromium/src/extensions/browser...

I thought that chrome/browser/ui/webui/omnibox/omnibox_ui_handler.* did the same
thing, but the OmniboxUIHandler doesn't seem to live in *any* namespace, so
that's something else entirely :)

[jkarlin, 2015/07/17 20:29:18]

Sounds good, thanks for looking it up!

+
+template <>
+struct CONTENT_EXPORT
+    TypeConverter<scoped_ptr<content::BackgroundSyncRegistration>,
+                  content::SyncRegistrationPtr> {
+  static scoped_ptr<content::BackgroundSyncRegistration> Convert(
+      const content::SyncRegistrationPtr& input);
+};
+
+template <>
+struct CONTENT_EXPORT TypeConverter<content::SyncRegistrationPtr,
+                                    content::BackgroundSyncRegistration> {
+  static content::SyncRegistrationPtr Convert(
+      const content::BackgroundSyncRegistration& input);
+};
+
+}  // namespace
+
 #endif  // CONTENT_BROWSER_BACKGROUND_SYNC_BACKGROUND_SYNC_REGISTRATION_H_