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

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

Issue 343066: Now that the UI layer is accessible cross-platform, coalesce the DesktopNotif... (Closed) Base URL: svn://chrome-svn/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/desktop_notification_service_linux.cc
===================================================================
--- chrome/browser/notifications/desktop_notification_service_linux.cc (revision 30584)
+++ chrome/browser/notifications/desktop_notification_service_linux.cc (working copy)
@@ -1,30 +0,0 @@
-// 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/desktop_notification_service.h"
-
-bool DesktopNotificationService::ShowDesktopNotification(
- const GURL& url,
- const GURL&,
- int process_id,
- int route_id,
- NotificationSource source,
- int notification_id) {
- // TODO(johnnyg): http://crbug.com/23954 Linux support coming soon.
- return false;
-}
-
-bool DesktopNotificationService::ShowDesktopNotificationText(
- const GURL& origin,
- const GURL& url,
- const string16& title,
- const string16& text,
- int process_id,
- int route_id,
- NotificationSource source,
- int notification_id) {
- // TODO(johnnyg): http://crbug.com/23066 Linux support coming soon.
- // Coming soon.
- return false;
-}

Powered by Google App Engine
This is Rietveld 408576698