IDL: [NoImplHeader] should not add '#include "None"'

IDL: [NoImplHeader] should not add '#include "None"'

[NoImplHeader] extended attribute is added for Blink-in-JS
so that developers can add DOM features without having C++
implementation. However, the current IDL compiler adds
'#include "None"' when an interface has[NoImplHeader], which
causes build errors. Fix this by not adding
interface_info['include_path'] when it is None.

Note that we can't implement DOM features in pure-JS yet;
we need .h/.cpp implementation to make privates script workable.

BUG=341031
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=188179

[bashi, 2015-01-09 08:20:23]

bashi@chromium.org changed reviewers:
+ haraken@chromium.org

[bashi, 2015-01-09 08:20:23]

PTAL?

[Jens Widell, 2015-01-09 09:09:16]

jl@opera.com changed reviewers:
+ jl@opera.com

[Jens Widell, 2015-01-09 09:09:17]

LGTM

https://codereview.chromium.org/839633003/diff/1/Source/bindings/scripts/code...
File Source/bindings/scripts/code_generator_v8.py (right):

https://codereview.chromium.org/839633003/diff/1/Source/bindings/scripts/code...
Source/bindings/scripts/code_generator_v8.py:194: else:
Use "elif"?

[bashi, 2015-01-09 09:13:21]

Thank you for review!

https://codereview.chromium.org/839633003/diff/1/Source/bindings/scripts/code...
File Source/bindings/scripts/code_generator_v8.py (right):

https://codereview.chromium.org/839633003/diff/1/Source/bindings/scripts/code...
Source/bindings/scripts/code_generator_v8.py:194: else:
On 2015/01/09 09:09:17, Jens Widell wrote:
> Use "elif"?

Done.

[haraken, 2015-01-09 09:34:29]

LGTM

[bashi, 2015-01-09 10:02:55]

The CQ bit was checked by bashi@chromium.org

[commit-bot: I haz the power, 2015-01-09 10:03:36]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/839633003/20001

[commit-bot: I haz the power, 2015-01-09 12:00:18]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-01-09 12:00:19]

Try jobs failed on following builders:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/45489)

[bashi, 2015-01-10 00:38:01]

The CQ bit was checked by bashi@chromium.org

[commit-bot: I haz the power, 2015-01-10 00:39:03]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/839633003/20001

[commit-bot: I haz the power, 2015-01-10 02:16:34]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=188179