ServiceWorker: Introduce SWMessageSender in order to mock IPC messaging

content/child/service_worker/service_worker_message_sender.h

Index: content/child/service_worker/service_worker_message_sender.h
diff --git a/content/child/service_worker/service_worker_message_sender.h b/content/child/service_worker/service_worker_message_sender.h
new file mode 100644
index 0000000000000000000000000000000000000000..df6726a607be12c17713505c12073a2871a2c746
--- /dev/null
+++ b/content/child/service_worker/service_worker_message_sender.h
@@ -0,0 +1,39 @@
+// 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_CHILD_SERVICE_WORKER_SERVICE_WORKER_MESSAGE_SENDER_H_
+#define CONTENT_CHILD_SERVICE_WORKER_SERVICE_WORKER_MESSAGE_SENDER_H_
+
+#include "base/memory/ref_counted.h"
+#include "content/common/content_export.h"
+
+namespace IPC {
+class Message;
+}
+
+namespace content {
+
+class ThreadSafeSender;
+
+class CONTENT_EXPORT ServiceWorkerMessageSender

[Kunihiko Sakamoto, 2015/03/17 03:40:38]

Can you add a short class comment that says this provides a mockable interface?

[nhiroki, 2015/03/17 07:36:53]

Added.

In addition, I made this class ThreadSafe because ThreadSafeSender was obviously
a thread-safe object and some owners could release this on a different thread
(eg. SWProviderContext::DestructOnMainThread could delete it on the worker
thread).

+    : public base::RefCounted<ServiceWorkerMessageSender> {
+ public:
+  explicit ServiceWorkerMessageSender(ThreadSafeSender* sender);
+
+  virtual bool Send(IPC::Message* message);
+
+ protected:
+  virtual ~ServiceWorkerMessageSender();
+
+ private:
+  friend class base::RefCounted<ServiceWorkerMessageSender>;
+
+  scoped_refptr<ThreadSafeSender> thread_safe_sender_;
+
+  DISALLOW_COPY_AND_ASSIGN(ServiceWorkerMessageSender);
+};
+
+}  // namespace content
+
+#endif  // CONTENT_CHILD_SERVICE_WORKER_SERVICE_WORKER_MESSAGE_SENDER_H_