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

Unified Diff: chrome/browser/notifications/notification_object_proxy.cc

Issue 194108: adds DesktopNotificationService to Profile (Closed)
Patch Set: more feedback Created 11 years, 2 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: chrome/browser/notifications/notification_object_proxy.cc
diff --git a/chrome/browser/notifications/notification_object_proxy.cc b/chrome/browser/notifications/notification_object_proxy.cc
new file mode 100644
index 0000000000000000000000000000000000000000..f230d19322d8dad94d8b9755ce69d7e7f749ce8d
--- /dev/null
+++ b/chrome/browser/notifications/notification_object_proxy.cc
@@ -0,0 +1,68 @@
+// Copyright (c) 2009 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 "chrome/browser/notifications/notification_object_proxy.h"
+
+#include "base/message_loop.h"
+#include "base/string16.h"
+#include "chrome/browser/browser_process.h"
+#include "chrome/browser/chrome_thread.h"
+#include "chrome/browser/renderer_host/render_view_host.h"
+#include "chrome/common/render_messages.h"
+
+NotificationObjectProxy::NotificationObjectProxy(int process_id, int route_id,
+ int notification_id, bool worker)
+ : process_id_(process_id),
+ route_id_(route_id),
+ notification_id_(notification_id),
+ worker_(worker) {
+}
+
+void NotificationObjectProxy::Display() {
+ if (worker_) {
+ // TODO(johnnyg): http://crbug.com/23065 Worker support coming soon.
+ NOTREACHED();
+ } else {
+ DeliverMessage(new ViewMsg_PostDisplayToNotificationObject(
+ route_id_, notification_id_));
+ }
+}
+
+void NotificationObjectProxy::Error() {
+ if (worker_) {
+ // TODO(johnnyg): http://crbug.com/23065 Worker support coming soon.
+ NOTREACHED();
+ } else {
+ DeliverMessage(new ViewMsg_PostErrorToNotificationObject(
+ route_id_, notification_id_, string16()));
+ }
+}
+
+void NotificationObjectProxy::Close(bool by_user) {
+ if (worker_) {
+ // TODO(johnnyg): http://crbug.com/23065 Worker support coming soon.
+ NOTREACHED();
+ } else {
+ DeliverMessage(new ViewMsg_PostCloseToNotificationObject(
+ route_id_, notification_id_, by_user));
+ }
+}
+
+void NotificationObjectProxy::DeliverMessage(IPC::Message* message) {
+ DCHECK(ChromeThread::CurrentlyOn(ChromeThread::UI));
+ MessageLoop* io_loop = ChromeThread::GetMessageLoop(ChromeThread::IO);
+ if (io_loop) {
+ io_loop->PostTask(FROM_HERE,
+ NewRunnableMethod(this, &NotificationObjectProxy::Send, message));
+ }
+}
+
+// Deferred method which runs on the IO thread and sends a message to the
+// proxied notification, routing it through the correct host in the browser.
+void NotificationObjectProxy::Send(IPC::Message* message) {
+ DCHECK(ChromeThread::CurrentlyOn(ChromeThread::IO));
+ RenderViewHost* host = RenderViewHost::FromID(process_id_, route_id_);
+ if (host)
+ host->Send(message);
+}

Powered by Google App Engine
This is Rietveld 408576698