PaymentHandler: Should use NotAllowedError instead of InvalidAccessError

When openWindow() is not triggered by user activation, currently throws
InvalidAccessError. But it's already deprecated in spec level. Instead
we should throw NotAllowedError.

Related spec change:
  https://github.com/w3c/payment-handler/pull/291

Bug: 833532
Change-Id: Icc21301abcafa0ecd00d6232d73a273ca6e4dd9b
Reviewed-on: https://chromium-review.googlesource.com/1013659
Commit-Queue: Jinho Bang <jinho.bang@samsung.com>
Reviewed-by: Ganggui Tang <gogerald@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553685}

third_party/blink/renderer/modules/payments/payment_request_event.cc

@@ -96,8 +96,9 @@ ScriptPromise PaymentRequestEvent::openWindow(ScriptState* script_state,
   }
 
   if (!context->IsWindowInteractionAllowed()) {
-    resolver->Reject(DOMException::Create(kInvalidAccessError,
-                                          "Not allowed to open a window."));
+    resolver->Reject(DOMException::Create(
+        kNotAllowedError,
+        "Not allowed to open a window without user activation"));
     return promise;
   }
   context->ConsumeWindowInteraction();