Web Locks: Reject promise for non-fully-active documents

The script-facing request() and query() methods in our Web Locks
implementation had an early exit for the non-fully-active a.k.a.
detached document case, but simply returned a dummy promise. Since
this was implemented, the web platform generally and the Web Locks API
specifically have better defined the behavior for this case, where a
rejected promise should be returned.

There was already a WPT for this which we were failing. Make it pass!

BUG: 1234679

Change-Id: I6ad937292c2d768fba783b890f3bb4332cb228cf
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4112406
Reviewed-by: Ayu Ishii <ayui@chromium.org>
Commit-Queue: Joshua Bell <jsbell@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1084539}

third_party/blink/renderer/modules/locks/lock_manager.cc

@@ -43,6 +43,7 @@ namespace {
 
 constexpr char kRequestAbortedMessage[] = "The request was aborted.";
 constexpr char kSecurityErrorMessage[] = "The request was denied.";
+constexpr char kInvalidStateErrorMessage[] = "The document is not active.";
 
 LockInfo* ToLockInfo(const mojom::blink::LockInfoPtr& record) {
   LockInfo* info = LockInfo::Create();
@@ -270,8 +271,11 @@ ScriptPromise LockManager::request(ScriptState* script_state,
                                    V8LockGrantedCallback* callback,
                                    ExceptionState& exception_state) {
   // Observed context may be gone if frame is detached.
-  if (!GetExecutionContext())
+  if (!GetExecutionContext()) {
+    exception_state.ThrowDOMException(DOMExceptionCode::kInvalidStateError,
+                                      kInvalidStateErrorMessage);
     return ScriptPromise();
+  }
 
   ExecutionContext* context = ExecutionContext::From(script_state);
   DCHECK(context->IsContextThread());
@@ -423,6 +427,8 @@ ScriptPromise LockManager::query(ScriptState* script_state,
                                  ExceptionState& exception_state) {
   // Observed context may be gone if frame is detached.
   if (!GetExecutionContext()) {
+    exception_state.ThrowDOMException(DOMExceptionCode::kInvalidStateError,
+                                      kInvalidStateErrorMessage);
     return ScriptPromise();
   }
   ExecutionContext* context = ExecutionContext::From(script_state);