Cross-script variables handling fixed. It was possible to write to read-only global variable.

Cross-script variables handling fixed. It was possible to write to read-only global variable.

Committed: https://crrev.com/156042f7f71effd101fbfb2ca1b5a2c3aec62079
Cr-Commit-Position: refs/heads/master@{#29860}

[Igor Sheludko, 2015-07-25 16:34:51]

ishell@chromium.org changed reviewers:
+ bmeurer@chromium.org

[Igor Sheludko, 2015-07-25 16:34:51]

PTAL & CQ.

We forgot add a cell validity check in StoreGlobalViaContext. I also wrote a
test which caught a couple of issues in our new system and even one issue in
Crankshaft. So now it is clear that loads and stores require different cells if
we don't want to call runtime for reading from read-only vars. 
Having one cell and two handlers for loads and stores will probably fix this
problem. I'll work on that when I get back.

[Benedikt Meurer, 2015-07-27 05:24:15]

LGTM.

[Benedikt Meurer, 2015-07-27 05:24:22]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2015-07-27 05:24:32]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1259853002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1259853002/1

[commit-bot: I haz the power, 2015-07-27 05:53:04]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-07-27 05:53:15]

Patchset 1 (id:??) landed as
https://crrev.com/156042f7f71effd101fbfb2ca1b5a2c3aec62079
Cr-Commit-Position: refs/heads/master@{#29860}