[Background Fetch] Post-submit review comments for 1155118

Change-Id: I9a2287102428a6c386fde09d9d237c6a9419e824
Reviewed-on: https://chromium-review.googlesource.com/1156589Reviewed-by: Mugdha Lakhani <nator@chromium.org>
Commit-Queue: Peter Beverloo <peter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579865}

content/browser/background_fetch/background_fetch_context.cc

@@ -220,8 +220,8 @@ void BackgroundFetchContext::AbandonFetches(
             service_worker_registration_id) {
       DCHECK(saved_iter->second);
 
-      // TODO(peter): Temporary work-around for a crash where fetches for a
-      // given Service Worker registration are abandoned twice.
+      // TODO(crbug.com/850512): Temporary work-around for a crash where fetches
+      // for a given Service Worker registration are abandoned twice.
       if (saved_iter->second->aborted())
         continue;
 

third_party/blink/renderer/modules/background_fetch/background_fetch_manager.cc

@@ -209,8 +209,9 @@ ScriptPromise BackgroundFetchManager::fetch(
                                  "that URL is invalid");
     }
 
-    // 6.3.1.7.2: If |internalRequest|’s mode is "no-cors", then return a
-    //            promise rejected with a TypeError.
+    // https://wicg.github.io/background-fetch/#dom-backgroundfetchmanager-fetch
+    // ""If |internalRequest|’s mode is "no-cors", then return a promise
+    //   rejected with a TypeError.""
     if (web_request.Mode() == network::mojom::FetchRequestMode::kNoCORS) {
       return RejectWithTypeError(script_state, request_url,
                                  "the request mode must not be no-cors");