Adds HTTP check to PresentationRequest creation.

third_party/WebKit/Source/modules/presentation/PresentationRequest.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 "modules/presentation/PresentationRequest.h"
 
 #include "bindings/core/v8/CallbackPromiseAdapter.h"
 #include "bindings/core/v8/ExceptionState.h"
 #include "bindings/core/v8/ScriptPromise.h"
 #include "bindings/core/v8/ScriptPromiseResolver.h"
@@ skipping 52 matching lines @@
 }
 
 }  // anonymous namespace
 
 // static
 PresentationRequest* PresentationRequest::create(
     ExecutionContext* executionContext,
     const String& url,
     ExceptionState& exceptionState) {
   KURL parsedUrl = KURL(executionContext->url(), url);
-  if (!parsedUrl.isValid() || parsedUrl.protocolIsAbout()) {
+  if (!parsedUrl.isValid() || !parsedUrl.protocolIsInHTTPFamily() ||

[zhaobin, 2016/12/09 18:53:02]

Remove parsedUrl.protocolIsAbout() check if we already checked
protocolIsInHTTPFamily()?

[CJ, 2016/12/13 02:12:38]

I'm not seeing how they overlap with each other. Please explain?

[zhaobin, 2016/12/13 19:06:02]

!protocolIsInHTTPFamily() means not http or https. about is not http or https,
so no need to check protocolIsAbout() again? 

[CJ, 2016/12/13 21:34:23]

Done.

+      parsedUrl.protocolIsAbout()) {
     exceptionState.throwTypeError("'" + url +
                                   "' can't be resolved to a valid URL.");
     return nullptr;
   }
 
   PresentationRequest* request =
       new PresentationRequest(executionContext, parsedUrl);
   request->suspendIfNeeded();
   return request;
 }
@@ skipping 124 matching lines @@
   ActiveDOMObject::trace(visitor);
 }
 
 PresentationRequest::PresentationRequest(ExecutionContext* executionContext,
                                          const KURL& url)
     : ActiveScriptWrappable(this),
       ActiveDOMObject(executionContext),
       m_url(url) {}
 
 }  // namespace blink