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

Side by Side Diff: content/renderer/notification_permission_dispatcher.cc

Issue 1898153002: Migrate notifications to use the PermissionStatus enum (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix type traits conflicts Created 4 years, 8 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 "content/renderer/notification_permission_dispatcher.h" 5 #include "content/renderer/notification_permission_dispatcher.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "content/public/common/service_registry.h" 10 #include "content/public/common/service_registry.h"
11 #include "content/public/renderer/render_frame.h" 11 #include "content/public/renderer/render_frame.h"
12 #include "third_party/WebKit/public/platform/WebSecurityOrigin.h" 12 #include "third_party/WebKit/public/platform/WebSecurityOrigin.h"
13 #include "third_party/WebKit/public/platform/WebString.h" 13 #include "third_party/WebKit/public/platform/WebString.h"
14 #include "third_party/WebKit/public/platform/modules/permissions/permission_stat us.mojom.h"
14 #include "third_party/WebKit/public/web/modules/notifications/WebNotificationPer missionCallback.h" 15 #include "third_party/WebKit/public/web/modules/notifications/WebNotificationPer missionCallback.h"
15 16
16 using blink::WebNotificationPermissionCallback; 17 using blink::WebNotificationPermissionCallback;
17 18
18 namespace content { 19 namespace content {
19 20
20 NotificationPermissionDispatcher::NotificationPermissionDispatcher( 21 NotificationPermissionDispatcher::NotificationPermissionDispatcher(
21 RenderFrame* render_frame) 22 RenderFrame* render_frame)
22 : RenderFrameObserver(render_frame) {} 23 : RenderFrameObserver(render_frame) {}
23 24
(...skipping 16 matching lines...) Expand all
40 base::Bind(&NotificationPermissionDispatcher::OnPermissionRequestComplete, 41 base::Bind(&NotificationPermissionDispatcher::OnPermissionRequestComplete,
41 base::Unretained(this), 42 base::Unretained(this),
42 base::Passed(std::move(owned_callback)))); 43 base::Passed(std::move(owned_callback))));
43 } 44 }
44 45
45 void NotificationPermissionDispatcher::OnPermissionRequestComplete( 46 void NotificationPermissionDispatcher::OnPermissionRequestComplete(
46 std::unique_ptr<WebNotificationPermissionCallback> callback, 47 std::unique_ptr<WebNotificationPermissionCallback> callback,
47 blink::mojom::PermissionStatus status) { 48 blink::mojom::PermissionStatus status) {
48 DCHECK(callback); 49 DCHECK(callback);
49 50
50 blink::WebNotificationPermission permission = 51 callback->permissionRequestComplete(status);
51 blink::WebNotificationPermissionDefault;
52 switch (status) {
53 case blink::mojom::PermissionStatus::GRANTED:
54 permission = blink::WebNotificationPermissionAllowed;
55 break;
56 case blink::mojom::PermissionStatus::DENIED:
57 permission = blink::WebNotificationPermissionDenied;
58 break;
59 case blink::mojom::PermissionStatus::ASK:
60 permission = blink::WebNotificationPermissionDefault;
61 break;
62 }
63
64 callback->permissionRequestComplete(permission);
65 } 52 }
66 53
67 } // namespace content 54 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698