Wake Lock API implementation (Blink part)

Source/core/dom/DocumentWakeLock.idl

+// Copyright 2015 The Chromium Authors. All rights reserved.

[mlamouri (slow - plz ping), 2015/04/28 08:00:25]

Could you move that to a modules instead? Like Sources/modules/wake-lock/

[alogvinov, 2015/04/28 16:49:10]

We actually tried doing that, but got a strange runtime assert in generated file
"gen/blink/bindings/core/v8/V8Document.cpp":

ASSERT(V8Document::installV8DocumentTemplateFunction !=
V8Document::installV8DocumentTemplate);

Can you perhaps point me at a possible cause?

There is no existing case of extending Document interface in modules, so we have
no working example to learn from.

[mlamouri (slow - plz ping), 2015/04/29 10:07:08]

Hmm, I'm not sure where that's coming from. Did you try to move the
RuntimeEnabled=WakeLock to the attribute?

[alogvinov, 2015/04/29 16:50:59]

Tried that, makes no difference. I also tried to moving dom/Document.idl from
'core_idl_files' to 'core_idl_with_modules_dependency_files' in core.gypi, but
then I got a even stranger code generation problem. In
gen/blink/bindings/modules/v8/V8DocumentPartial.cpp I got function definition
for V8Document::PrivateScript::transformDocumentToTreeViewMethod, the same one
as in gen/blink/bindings/core/v8/V8Document.cpp, so linker complains about
duplicate function definition. "transformDocumentToTreeView" is actually defined
in core/xml/DocumentXMLTreeViewer.idl, and that file is listed in
'core_dependency_idl_files' in core.gypi. DocumentWakeLock.idl is listed in
'modules_dependency_idl_files' in modules.gypi. Could it be a v8 bindings code
generator bug, or am I doing something wrong? I can update this CL with what I
currently have if that helps.

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+[
+    RuntimeEnabled=WakeLock,
+] partial interface Document {
+    attribute boolean keepScreenAwake;
+};