Define default but still private MediaSession constructor

Source/modules/mediasession/MediaSession.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 MediaSession_h
 #define MediaSession_h
 
 #include "bindings/core/v8/ScriptWrappable.h"
 #include "platform/heap/Handle.h"
 
 namespace blink {
 
 class MediaSession : public GarbageCollected<MediaSession>, public ScriptWrappable {
     DEFINE_WRAPPERTYPEINFO();
 public:
     static MediaSession* create();
 
     void activate();
     void deactivate();
 
     DEFINE_INLINE_TRACE() { }
 
 private:
-    MediaSession();
+    MediaSession() { }

[whywhat, 2015/07/02 15:05:16]

 = default

[davve, 2015/07/02 15:19:53]

https://www.chromium.org/developers/coding-style/cpp-dos-and-donts#TOC-Stop-i...

was a good read. Thanks!

So you would recommend moving back the implementation to the .cpp file and use
'= default' there?

[whywhat, 2015/07/02 15:33:50]

As I understand, you either leave it as it was (empty implementation in .cpp
file) or use '= default' in the header which is not the same as inlining so the
forward declaration example would compile.

[mlamouri (slow - plz ping), 2015/07/02 17:28:55]

+1 for using "= default". lgtm with that.

 };
 
 } // namespace blink
 
 #endif // MediaSession_h