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

Side by Side Diff: chrome/browser/notifications/persistent_notification_delegate.cc

Issue 1422673002: Extract Notifications kClickWasNotAnAction constant Base URL: https://chromium.googlesource.com/chromium/src.git@actions_spec_sync
Patch Set: Created 5 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/notifications/persistent_notification_delegate.h" 5 #include "chrome/browser/notifications/persistent_notification_delegate.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/guid.h" 8 #include "base/guid.h"
9 #include "chrome/browser/notifications/platform_notification_service_impl.h" 9 #include "chrome/browser/notifications/platform_notification_service_impl.h"
10 #include "content/public/common/persistent_notification_status.h" 10 #include "content/public/common/persistent_notification_status.h"
11 #include "content/public/common/platform_notification_data.h"
11 12
12 PersistentNotificationDelegate::PersistentNotificationDelegate( 13 PersistentNotificationDelegate::PersistentNotificationDelegate(
13 content::BrowserContext* browser_context, 14 content::BrowserContext* browser_context,
14 int64_t persistent_notification_id, 15 int64_t persistent_notification_id,
15 const GURL& origin, 16 const GURL& origin,
16 int notification_settings_index) 17 int notification_settings_index)
17 : browser_context_(browser_context), 18 : browser_context_(browser_context),
18 persistent_notification_id_(persistent_notification_id), 19 persistent_notification_id_(persistent_notification_id),
19 origin_(origin), 20 origin_(origin),
20 id_(base::GenerateGUID()), 21 id_(base::GenerateGUID()),
21 notification_settings_index_(notification_settings_index) {} 22 notification_settings_index_(notification_settings_index) {}
22 23
23 PersistentNotificationDelegate::~PersistentNotificationDelegate() {} 24 PersistentNotificationDelegate::~PersistentNotificationDelegate() {}
24 25
25 void PersistentNotificationDelegate::Display() {} 26 void PersistentNotificationDelegate::Display() {}
26 27
27 void PersistentNotificationDelegate::Close(bool by_user) { 28 void PersistentNotificationDelegate::Close(bool by_user) {
28 PlatformNotificationServiceImpl::GetInstance()->OnPersistentNotificationClose( 29 PlatformNotificationServiceImpl::GetInstance()->OnPersistentNotificationClose(
29 browser_context_, 30 browser_context_,
30 persistent_notification_id_, 31 persistent_notification_id_,
31 origin_); 32 origin_);
32 } 33 }
33 34
34 void PersistentNotificationDelegate::Click() { 35 void PersistentNotificationDelegate::Click() {
35 PlatformNotificationServiceImpl::GetInstance()->OnPersistentNotificationClick( 36 PlatformNotificationServiceImpl::GetInstance()->OnPersistentNotificationClick(
36 browser_context_, 37 browser_context_,
37 persistent_notification_id_, 38 persistent_notification_id_,
38 origin_, 39 origin_,
39 -1 /* action_index */); 40 content::kNotificationClickWasNotAnAction);
40 } 41 }
41 42
42 void PersistentNotificationDelegate::ButtonClick(int button_index) { 43 void PersistentNotificationDelegate::ButtonClick(int button_index) {
43 DCHECK_GE(button_index, 0); 44 DCHECK_GE(button_index, 0);
44 if (button_index == notification_settings_index_) { 45 if (button_index == notification_settings_index_) {
45 PlatformNotificationServiceImpl::GetInstance()->OpenNotificationSettings( 46 PlatformNotificationServiceImpl::GetInstance()->OpenNotificationSettings(
46 browser_context_); 47 browser_context_);
47 return; 48 return;
48 } 49 }
49 50
50 PlatformNotificationServiceImpl::GetInstance()->OnPersistentNotificationClick( 51 PlatformNotificationServiceImpl::GetInstance()->OnPersistentNotificationClick(
51 browser_context_, 52 browser_context_,
52 persistent_notification_id_, 53 persistent_notification_id_,
53 origin_, 54 origin_,
54 button_index); 55 button_index);
55 } 56 }
56 57
57 std::string PersistentNotificationDelegate::id() const { 58 std::string PersistentNotificationDelegate::id() const {
58 return id_; 59 return id_;
59 } 60 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698