Switch the order of experimental global initialization and running natives.

Switch the order of experimental global initialization and running natives.

The order should match the one for normal natives.

R=yangguo@chromium.org

Committed: https://crrev.com/c071af5ae34cdb179f12efefdb93064e2e2f17fd
Cr-Commit-Position: refs/heads/master@{#27165}

[Dmitry Lomov (no reviews), 2015-03-12 14:02:40]

PTAL

[Yang, 2015-03-12 14:20:32]

On 2015/03/12 14:02:40, Dmitry Lomov (chromium) wrote:
> PTAL

lgtm.

[Dmitry Lomov (no reviews), 2015-03-12 14:33:54]

The CQ bit was checked by dslomov@chromium.org

[commit-bot: I haz the power, 2015-03-12 14:34:03]

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

[commit-bot: I haz the power, 2015-03-12 14:58:55]

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

[commit-bot: I haz the power, 2015-03-12 14:58:58]

Failed to apply patch for src/regexp.js:
While running git apply --index -3 -p1;
  error: patch failed: src/regexp.js:8
  Falling back to three-way merge...
  Applied patch to 'src/regexp.js' with conflicts.
  U src/regexp.js

Patch:       src/regexp.js
Index: src/regexp.js
diff --git a/src/regexp.js b/src/regexp.js
index
416f5865e217973554c2192c816b915db372c20a..babbae74aab450eee92032d21761f9fc598a6b7e
100644
--- a/src/regexp.js
+++ b/src/regexp.js
@@ -8,6 +8,8 @@
 // var $Array = global.Array;
 
 var $RegExp = global.RegExp;
+var harmony_regexps = false;
+var harmony_unicode_regexps = false;
 
 // -------------------------------------------------------------------

[commit-bot: I haz the power, 2015-03-12 14:59:00]

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

[Dmitry Lomov (no reviews), 2015-03-12 15:25:07]

The CQ bit was checked by dslomov@chromium.org

[Dmitry Lomov (no reviews), 2015-03-12 15:25:07]

The patchset sent to the CQ was uploaded after l-g-t-m from yangguo@chromium.org
Link to the patchset: https://codereview.chromium.org/988513003/#ps20001 (title:
"Rebase")

[commit-bot: I haz the power, 2015-03-12 15:25:20]

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

[commit-bot: I haz the power, 2015-03-12 15:44:40]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-03-12 15:44:51]

Patchset 2 (id:??) landed as
https://crrev.com/c071af5ae34cdb179f12efefdb93064e2e2f17fd
Cr-Commit-Position: refs/heads/master@{#27165}

[arv (Not doing code reviews), 2015-03-12 15:47:30]

FYI

https://codereview.chromium.org/988513003/diff/20001/src/regexp.js
File src/regexp.js (right):

https://codereview.chromium.org/988513003/diff/20001/src/regexp.js#newcode9
src/regexp.js:9: var harmony_regexps = false;
Why the underscores? JS style is camelCase.

[Jakob Kummerow, 2015-03-12 16:05:36]

https://codereview.chromium.org/988513003/diff/20001/src/regexp.js
File src/regexp.js (right):

https://codereview.chromium.org/988513003/diff/20001/src/regexp.js#newcode9
src/regexp.js:9: var harmony_regexps = false;
On 2015/03/12 15:47:30, arv wrote:
> Why the underscores? JS style is camelCase.

DBC: I think this is acceptable, considering prevalent styles across the V8
project, which comprises mostly C++ code.

In fact, my personal preference for having consistent style across the project
is so strong that when I write JS or Python code that's checked into the V8
repo, I like to just follow the C++ style guide (as far as it can be applied to
those languages).

That said, I can see the point that the current mix of styles we have in *.js
files is less than ideal. It doesn't bother me much, though.

I'm sure we could waste a splendid amount of time discussing various aspects of
this issue. (Did I mention that predator pink is the bestest shed color of all
time?)

[Dmitry Lomov (no reviews), 2015-03-12 16:40:15]

On 2015/03/12 15:47:30, arv wrote:
> FYI
> 
> https://codereview.chromium.org/988513003/diff/20001/src/regexp.js
> File src/regexp.js (right):
> 
> https://codereview.chromium.org/988513003/diff/20001/src/regexp.js#newcode9
> src/regexp.js:9: var harmony_regexps = false;
> Why the underscores? JS style is camelCase.

[Note that this CL does not introduce the variable, just changes the way it is
declared]

Consistency with how the flag is called in the rest of V8 code (I want to be
able to do 'git grep harmony_regexps')