[Background Sync] Sent sync registration details to worker

content/browser/background_sync/background_sync_registration.h

 // 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.
 
 #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 {
 
 class CONTENT_EXPORT BackgroundSyncRegistration {
  public:
   using RegistrationId = int64_t;
   static const RegistrationId kInitialId;
 
   BackgroundSyncRegistration();
   bool Equals(const BackgroundSyncRegistration& other) const;
@@ skipping 11 matching lines @@
  private:
   static const RegistrationId kInvalidRegistrationId;
 
   BackgroundSyncRegistrationOptions options_;
   RegistrationId id_ = kInvalidRegistrationId;
   SyncState sync_state_ = SYNC_STATE_PENDING;
 };
 
 }  // 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_