[Presentation API] PresentationRequest should throw SecurityError for mixed contents

third_party/WebKit/Source/modules/presentation/PresentationRequest.h

 // 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.
 
 #ifndef PresentationRequest_h
 #define PresentationRequest_h
 
 #include "bindings/core/v8/ActiveScriptWrappable.h"
 #include "bindings/core/v8/ScriptPromise.h"
 #include "core/dom/ActiveDOMObject.h"
@@ skipping 36 matching lines @@
 
   DECLARE_VIRTUAL_TRACE();
 
  protected:
   // EventTarget implementation.
   void addedEventListener(const AtomicString& eventType,
                           RegisteredEventListener&) override;
 
  private:
   PresentationRequest(ExecutionContext*, const KURL&);
+  ScriptPromise rejectWithMixedContentException(ScriptState*);
+  ScriptPromise rejectWithSandBoxException(ScriptState*);

[mlamouri (slow - plz ping), 2016/10/21 09:45:43]

I don't think these methods need to be part of the PresentationRequest class.
Can you add them in the anonymous namespace of the cpp file?

[zhaobin, 2016/10/21 18:41:01]

Done.

 
   KURL m_url;
 };
 
 }  // namespace blink
 
 #endif  // PresentationRequest_h