Add PermissionDescriptor to the permissions Mojo interface.

content/renderer/notification_permission_dispatcher.cc

 // Copyright 2014 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 "content/renderer/notification_permission_dispatcher.h"
 
 #include <utility>
 
 #include "base/bind.h"
 #include "content/public/renderer/render_frame.h"
@@ skipping 16 matching lines @@
 
 void NotificationPermissionDispatcher::RequestPermission(
     const blink::WebSecurityOrigin& origin,
     WebNotificationPermissionCallback* callback) {
   if (!permission_service_.get()) {
     render_frame()->GetRemoteInterfaces()->GetInterface(&permission_service_);
   }
 
   std::unique_ptr<WebNotificationPermissionCallback> owned_callback(callback);
 
+  auto descriptor = blink::mojom::PermissionDescriptor::New();
+  descriptor->name = blink::mojom::PermissionName::NOTIFICATIONS;

[ddorwin, 2016/08/24 02:43:55]

This pattern occurs frequently. ISTM, the constructor should take the permission
name. See the comment in the .mojom file.

+
   // base::Unretained is safe here because the Mojo channel, with associated
   // callbacks, will be deleted before the "this" instance is deleted.
   permission_service_->RequestPermission(
-      blink::mojom::PermissionName::NOTIFICATIONS, origin,
+      std::move(descriptor), origin,
       blink::WebUserGestureIndicator::isProcessingUserGesture(),
       base::Bind(&NotificationPermissionDispatcher::OnPermissionRequestComplete,
                  base::Unretained(this),
                  base::Passed(std::move(owned_callback))));
 }
 
 void NotificationPermissionDispatcher::OnPermissionRequestComplete(
     std::unique_ptr<WebNotificationPermissionCallback> callback,
     blink::mojom::PermissionStatus status) {
   DCHECK(callback);
 
   // Blink can't use non-blink bindings so we need to cast to int32.
   int32_t blink_status = static_cast<int32_t>(status);
 
   callback->permissionRequestComplete(blink_status);
 }
 
 void NotificationPermissionDispatcher::OnDestruct() {
   delete this;
 }
 
 }  // namespace content