Flip the true/false result of ScriptState::contextIsValid

Flip the true/false result of ScriptState::contextIsValid

Currently ScriptState::contextIsValid returns true when the context is not valid.
This is very confusing. This CL flips the true/false.

This CL doesn't fix bug 419624.
This is just a preparation for fixing bug 419624 (actually I have no idea at the moment though).

BUG=419624
TBR=jochen

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

[haraken, 2014-10-03 10:03:17]

haraken@chromium.org changed reviewers:
+ jl@opera.com

[haraken, 2014-10-03 10:03:18]

PTAL

[Jens Widell, 2014-10-03 10:11:39]

What about the three calls in
Source/bindings/core/v8/V8CustomElementLifecycleCallbacks.cpp?

[haraken, 2014-10-03 10:21:38]

On 2014/10/03 10:11:39, Jens Widell wrote:
> What about the three calls in
> Source/bindings/core/v8/V8CustomElementLifecycleCallbacks.cpp?

Nice catch! I doubly confirmed that I replaced all the contextIsValid's.

[Jens Widell, 2014-10-03 10:24:13]

LGTM

The new name is less misleading, certainly. :-)

[haraken, 2014-10-03 11:57:04]

Thanks for the careful review.

[haraken, 2014-10-03 11:57:08]

The CQ bit was checked by haraken@chromium.org

[haraken, 2014-10-03 11:57:31]

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

[haraken, 2014-10-03 11:57:36]

The CQ bit was unchecked by haraken@chromium.org

[haraken, 2014-10-03 11:57:44]

TBRing jochen@ for web/.

[haraken, 2014-10-03 11:57:48]

The CQ bit was checked by haraken@chromium.org

[commit-bot: I haz the power, 2014-10-03 11:59:14]

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

[commit-bot: I haz the power, 2014-10-03 13:09:59]

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

[commit-bot: I haz the power, 2014-10-03 13:10:01]

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

[jochen (gone - plz use gerrit), 2014-10-06 07:25:19]

d'oh

lgtm

[haraken, 2014-10-06 08:59:18]

The CQ bit was checked by haraken@chromium.org

[commit-bot: I haz the power, 2014-10-06 08:59:51]

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

[commit-bot: I haz the power, 2014-10-06 09:24:32]

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

[commit-bot: I haz the power, 2014-10-06 09:24:33]

Try jobs failed on following builders:
  linux_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/2...)
  mac_gpu_retina_triggered_tests on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu_retina_tr...)

[haraken, 2014-10-06 13:04:05]

The CQ bit was checked by haraken@chromium.org

[commit-bot: I haz the power, 2014-10-06 13:04:44]

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

[commit-bot: I haz the power, 2014-10-06 14:16:08]

Committed patchset #5 (id:80001) as 183271