IDL: Avoid generating empty ConstantConfiguration arrays

IDL: Avoid generating empty ConstantConfiguration arrays

If all constants in an interface are runtime enabled, an empty
(zero-length) C++ array would be generated, which isn't a good idea.

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

[Jens Widell, 2014-08-25 16:42:58]

PTAL

The issue caused compilation errors in another CL that attempted to make this
change: https://codereview.chromium.org/474013004/patch/20001/30031

[sof, 2014-08-25 20:05:45]

Test coverage would be preferable.

[Jens Widell, 2014-08-26 05:13:38]

On 2014/08/25 20:05:45, sof wrote:
> Test coverage would be preferable.

Fixed by adding a single runtime enabled constant to TestInterface2.idl, which
previously had no constants.

[haraken, 2014-08-26 05:22:19]

LGTM

[Jens Widell, 2014-08-26 05:24:10]

The CQ bit was checked by jl@opera.com

[commit-bot: I haz the power, 2014-08-26 05:25:17]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/jl@opera.com/499293003/20001

[commit-bot: I haz the power, 2014-08-26 05:30:34]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_blink_compile_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/android_blink_compile_db...)
  android_blink_compile_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/android_blink_compile_re...)
  android_chromium_gn_compile_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/android_chromium_gn_comp...)
  blink_presubmit on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/blink_presubmit/builds/1...)
  linux_blink_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_dbg/builds/2...)
  linux_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/2...)
  linux_chromium_gn_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_chromium_gn_rel/bu...)
  mac_blink_compile_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/mac_blink_compile_dbg/bu...)
  mac_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/mac_blink_rel/builds/21089)
  win_blink_compile_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_compile_dbg/bu...)
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/24294)
  linux_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/linux_gpu/builds/...)
  mac_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu/builds/46212)
  win_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/win_gpu/builds/51493)

[commit-bot: I haz the power, 2014-08-26 05:31:44]

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

[commit-bot: I haz the power, 2014-08-26 05:31:45]

Try jobs failed on following builders:
  mac_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu/builds/46216)

[Jens Widell, 2014-08-26 05:39:26]

The CQ bit was checked by jl@opera.com

[commit-bot: I haz the power, 2014-08-26 05:40:20]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/jl@opera.com/499293003/40001

[commit-bot: I haz the power, 2014-08-26 06:42:22]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  mac_blink_compile_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/mac_blink_compile_dbg/bu...)

[commit-bot: I haz the power, 2014-08-26 08:07:35]

Committed patchset #3 (40001) as 180890