Fix linking problem on Windows with ENABLE_LEGACY_NOTIFICATIONS disabled

Fix linking problem on Windows with LEGACY_NOTIFICATIONS support disabled

V8NotificationCenter bindings (generated from NotificationCenter.idl) are
required to link this code. Those are conditional on LEGACY_NOTIFICATIONS
though so same condition must be applied to this code.

BUG=

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

[Peter Beverloo, 2013-09-12 15:35:32]

lgtm, thanks!

I wonder why this error didn't show up on the Android bots, where
ENABLE_LEGACY_NOTIFICATIONS is also set to 0.  Locally the build works with or
without this change for me.  I've launched try-bots for you, and please wait for
a modules/ OWNER to sign off as well.

[Rafał Chłodnicki, 2013-09-12 15:38:31]

On 2013/09/12 15:35:32, Peter Beverloo wrote:
> lgtm, thanks!
> 
> I wonder why this error didn't show up on the Android bots, where
> ENABLE_LEGACY_NOTIFICATIONS is also set to 0.  Locally the build works with or
> without this change for me.  I've launched try-bots for you, and please wait
for
> a modules/ OWNER to sign off as well.

Might be Windows specific. V8NotificationCenter.cpp has some comment near the
top, above the function that triggers the problem. Might be related.

Also not sure if related but this was spotted in a "component" build. Maybe it
is not a problem in a static build.

[commit-bot: I haz the power, 2013-09-12 19:24:48]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rchlodnicki@opera.com/23583043/1003

[commit-bot: I haz the power, 2013-09-12 19:35:53]

Retried try job too often on blink_presubmit for step(s) presubmit
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=blink_pres...

[Peter Beverloo, 2013-09-13 13:12:23]

On 2013/09/12 19:35:53, I haz the power (commit-bot) wrote:
> Retried try job too often on blink_presubmit for step(s) presubmit
>
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=blink_pres...

You still need OWNERS approval for Source/modules/.  Perhaps Adam can have a
look, alternatively you can try any of the other people listed in the most local
OWNERS file:

https://code.google.com/p/chromium/codesearch#chromium/src/third_party/WebKit...

[abarth-chromium, 2013-09-13 17:43:37]

lgtm

[commit-bot: I haz the power, 2013-09-13 17:43:47]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rchlodnicki@opera.com/23583043/1003

[commit-bot: I haz the power, 2013-09-13 19:48:24]

Change committed as 157776