[Presentation API] Added the DefaultPresentationStart event and parsing of the default presentation…

Source/modules/presentation/DefaultPresentationStartEvent.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/presentation/DefaultPresentationStartEvent.h"
+
+namespace blink {
+
+DefaultPresentationStartEvent::~DefaultPresentationStartEvent()
+{
+}
+
+DefaultPresentationStartEvent::DefaultPresentationStartEvent()
+{
+}
+
+DefaultPresentationStartEvent::DefaultPresentationStartEvent(const AtomicString& eventType, PresentationSession* session)
+    : Event(eventType,
+        false, // canBubble

[Peter Beverloo, 2015/03/09 19:05:28]

nit: it's more common in Blink to have something like the following:

Event(eventType, false /* canBubble */, false /* cancelable */)

If you decide to keep this, please fix indentation.

[whywhat, 2015/03/10 13:00:44]

Yes, but  I believe this is harder to parse and stronger violates Chromium's two
spaces between code and comments law.
By fixing do you mean indenting 4 more spaces?

[Peter Beverloo, 2015/03/10 13:06:53]

https://code.google.com/p/chromium/codesearch#search/&q=%22*/,%22%20-f:usr&sq...

It's rather common actually, and I would prefer you to use that. Blink uses this
as well as Chromium does.

Indentation is off by any style guide. Either use the inline style mentioned
above, align with "(" or indent with four spaces as a continuation.

[whywhat, 2015/03/10 15:18:51]

Done.

+        false) // cancelable
+    , m_session(session)
+{
+}
+
+DefaultPresentationStartEvent::DefaultPresentationStartEvent(const AtomicString& eventType, const DefaultPresentationStartEventInit& initializer)
+    : Event(eventType, initializer)
+    , m_session(initializer.session())
+{
+}
+
+const AtomicString& DefaultPresentationStartEvent::interfaceName() const
+{
+    return EventNames::DefaultPresentationStartEvent;
+}
+
+} // namespace blink