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

Unified Diff: content/browser/intents/internal_web_intents_dispatcher.h

Issue 9692017: An internal intents dispatcher useful for initiating an intent from the browser process. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Add typedef Created 8 years, 9 months 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: content/browser/intents/internal_web_intents_dispatcher.h
diff --git a/content/browser/intents/internal_web_intents_dispatcher.h b/content/browser/intents/internal_web_intents_dispatcher.h
new file mode 100644
index 0000000000000000000000000000000000000000..e4a2b49139a6bb2586464da9c7d6b4eb739b93be
--- /dev/null
+++ b/content/browser/intents/internal_web_intents_dispatcher.h
@@ -0,0 +1,66 @@
+// Copyright (c) 2012 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_INTENTS_INTERNAL_WEB_INTENTS_DISPATCHER_H_
+#define CONTENT_BROWSER_INTENTS_INTERNAL_WEB_INTENTS_DISPATCHER_H_
+
+#include <vector>
+
+#include "base/callback.h"
+#include "base/compiler_specific.h"
+#include "content/public/browser/web_intents_dispatcher.h"
+#include "webkit/glue/web_intent_data.h"
+
+class IntentInjector;
+
+// This class implements a web intents dispatcher which originates
+// within the browser process rather than with a particular renderer.
+// The implementation handles replies to the web intents invocation by
+// notifying a registered callback rather than returning
+// those messages to any renderer.
+class InternalWebIntentsDispatcher : public content::WebIntentsDispatcher {
+ public:
+ typedef base::Callback<void(webkit_glue::WebIntentReplyType,
+ const string16&)> ReplyCallback;
James Hawkins 2012/03/13 20:15:44 Document the callback and parameters.
Greg Billock 2012/03/13 20:34:42 Done.
+
+ // |intent| is the intent payload to be dispatched.
+ explicit InternalWebIntentsDispatcher(
+ const webkit_glue::WebIntentData& intent);
+
+ // |intent| is the intent payload to be dispatched.
+ // |reply_callback| is the callback to notify when the intent is replied to.
+ InternalWebIntentsDispatcher(
+ const webkit_glue::WebIntentData& intent,
+ ReplyCallback reply_callback);
James Hawkins 2012/03/13 20:15:44 const&
Greg Billock 2012/03/13 20:34:42 Done.
+
+ virtual ~InternalWebIntentsDispatcher();
+
+ // WebIntentsDispatcher implementation.
+ virtual const webkit_glue::WebIntentData& GetIntent() OVERRIDE;
+ virtual void DispatchIntent(content::WebContents* destination_tab) OVERRIDE;
+ virtual void SendReplyMessage(webkit_glue::WebIntentReplyType reply_type,
+ const string16& data) OVERRIDE;
+ virtual void RegisterReplyNotification(
+ const content::WebIntentsDispatcher::ReplyNotification& closure) OVERRIDE;
+
+ private:
+ // The intent data to be delivered.
+ webkit_glue::WebIntentData intent_;
+
+ // Weak pointer to the internal object which provides the intent to the
James Hawkins 2012/03/13 20:15:44 When I read "provides" I thought it meant "provide
Greg Billock 2012/03/13 20:34:42 Done.
+ // newly-created service tab contents. This object is self-deleting
+ // (connected to the service TabContents).
+ IntentInjector* intent_injector_;
+
+ // Callbacks to be notified when SendReplyMessage is called.
+ std::vector<content::WebIntentsDispatcher::ReplyNotification>
+ reply_notifiers_;
+
+ // Callback to be invoked when the intent is replied to.
+ ReplyCallback reply_callback_;
+
+ DISALLOW_COPY_AND_ASSIGN(InternalWebIntentsDispatcher);
+};
+
+#endif // CONTENT_BROWSER_INTENTS_INTERNAL_WEB_INTENTS_DISPATCHER_H_

Powered by Google App Engine
This is Rietveld 408576698