Start implementing new navigator.connect API

Source/modules/navigatorconnect/ServicePortCollection.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 "config.h"
+#include "modules/navigatorconnect/ServicePortCollection.h"
+
+#include "core/dom/DOMException.h"
+#include "core/dom/ExceptionCode.h"
+#include "modules/EventTargetModules.h"
+
+namespace blink {
+
+PassRefPtrWillBeRawPtr<ServicePortCollection> ServicePortCollection::create(ExecutionContext* context)
+{
+    return adoptRefWillBeNoop(new ServicePortCollection(context));
+}
+
+ServicePortCollection::ServicePortCollection(ExecutionContext* context)

[scheib, 2015/06/23 03:57:21]

Method definitions in the corresponding .cc file should be the same as the
declaration order, as much as possible.
http://google-styleguide.googlecode.com/svn/trunk/cppguide.html#Declaration_O...

[Marijn Kruisselbrink, 2015/06/23 04:11:05]

Done

+    : ContextLifecycleObserver(context)
+{
+}
+
+ServicePortCollection::~ServicePortCollection()
+{
+}
+
+ScriptPromise ServicePortCollection::connect(ScriptState* scriptState, const String& url, const ServicePortConnectOptions& options)
+{
+    return ScriptPromise::rejectWithDOMException(scriptState, DOMException::create(NotSupportedError));
+}
+
+ScriptPromise ServicePortCollection::match(ScriptState* scriptState, const ServicePortMatchOptions& options)
+{
+    return ScriptPromise::rejectWithDOMException(scriptState, DOMException::create(NotSupportedError));
+}
+
+ScriptPromise ServicePortCollection::matchAll(ScriptState* scriptState, const ServicePortMatchOptions& options)
+{
+    return ScriptPromise::rejectWithDOMException(scriptState, DOMException::create(NotSupportedError));
+}
+
+const AtomicString& ServicePortCollection::interfaceName() const
+{
+    return EventTargetNames::ServicePortCollection;

[scheib, 2015/06/23 03:57:21]

Doesn't seem to be defined.

[Marijn Kruisselbrink, 2015/06/23 04:11:05]

It compiles (see trybots), so obviously it is defined (it's the change to
Source/modules/EventTargetModulesFactory.in
 that defines this).

+}
+
+ExecutionContext* ServicePortCollection::executionContext() const
+{
+    return ContextLifecycleObserver::executionContext();
+}
+
+DEFINE_TRACE(ServicePortCollection)
+{
+    EventTargetWithInlineData::trace(visitor);
+    ContextLifecycleObserver::trace(visitor);
+}
+
+} // namespace blink