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

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

Issue 271052: Browser side support (sans UI) for desktop notifications.... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: 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
===================================================================
--- chrome/browser/notifications/notification_object_proxy.cc (revision 0)
+++ chrome/browser/notifications/notification_object_proxy.cc (revision 0)
@@ -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);
+}
Property changes on: chrome/browser/notifications/notification_object_proxy.cc
___________________________________________________________________
Added: svn:eol-style
+ LF
« no previous file with comments | « chrome/browser/notifications/notification_object_proxy.h ('k') | chrome/browser/notifications/notifications_prefs_cache.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698