permissions: remove duplicates from request multiple at the Blink level

third_party/WebKit/Source/modules/permissions/PermissionsCallback.cpp

 // Copyright 2015 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 "config.h"
 #include "modules/permissions/PermissionsCallback.h"
 
 #include "bindings/core/v8/ScriptPromiseResolver.h"
 #include "modules/permissions/PermissionStatus.h"
 
 namespace blink {
 
-PermissionsCallback::PermissionsCallback(ScriptPromiseResolver* resolver, PassOwnPtr<WebVector<WebPermissionType>> permission_types)
+PermissionsCallback::PermissionsCallback(ScriptPromiseResolver* resolver, PassOwnPtr<Vector<WebPermissionType>> internalPermissions, PassOwnPtr<Vector<int>> callerIndexToInternalIndex)
     : m_resolver(resolver),
-    m_permissionTypes(permission_types)
+    m_internalPermissions(internalPermissions),
+    m_callerIndexToInternalIndex(callerIndexToInternalIndex)
 {
     ASSERT(m_resolver);
 }
 
 void PermissionsCallback::onSuccess(WebPassOwnPtr<WebVector<WebPermissionStatus>> permissionStatus)
 {
     if (!m_resolver->executionContext() || m_resolver->executionContext()->activeDOMObjectsAreStopped())
         return;
 
     OwnPtr<WebVector<WebPermissionStatus>> statusPtr = permissionStatus.release();
-
-    Vector<PermissionStatus*> status;
-    status.reserveCapacity(statusPtr->size());
-    for (size_t i = 0; i < statusPtr->size(); ++i)
-        status.append(PermissionStatus::createAndListen(m_resolver->executionContext(), (*statusPtr)[i], (*m_permissionTypes)[i]));
-
-    m_resolver->resolve(status);
+    Vector<PermissionStatus*> result(m_callerIndexToInternalIndex->size());

[mlamouri (slow - plz ping), 2015/09/29 18:00:07]

nit: need comments

[Lalit Maganti, 2015/10/02 13:57:25]

Done.

+    for (size_t i = 0; i < m_callerIndexToInternalIndex->size(); ++i) {
+        int internalIndex = m_callerIndexToInternalIndex->operator[](i);
+        result[i] = PermissionStatus::createAndListen(m_resolver->executionContext(), statusPtr->operator[](internalIndex), m_internalPermissions->operator[](internalIndex));
+    }
+    m_resolver->resolve(result);
 }
 
 void PermissionsCallback::onError()
 {
     if (!m_resolver->executionContext() || m_resolver->executionContext()->activeDOMObjectsAreStopped())
         return;
     m_resolver->reject();
 }
 
 } // namespace blink