Enable OneShot optimizations by default

BUG=v8:8072

Change-Id: I2068d66644628fe1de7a6be30b8d43ecfae04e93
Reviewed-on: https://chromium-review.googlesource.com/1220047
Commit-Queue: Chandan Reddy <chandanreddy@google.com>
Reviewed-by: Camillo Bruni <cbruni@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55875}

src/flag-definitions.h

@@ -324,7 +324,7 @@ DEFINE_BOOL(optimize_for_size, false,
             "speed")
 
 // Flag for one shot optimiztions.
-DEFINE_BOOL(enable_one_shot_optimization, false,
+DEFINE_BOOL(enable_one_shot_optimization, true,
             "Enable size optimizations for the code that will "
             "only be executed once")
 

test/cctest/test-api-accessors.cc

@@ -337,6 +337,7 @@ TEST(SetAccessorSetSideEffectReceiverCheck2) {
   LocalContext env;
   v8::Isolate* isolate = env->GetIsolate();
   v8::HandleScope scope(isolate);
+  i::FLAG_enable_one_shot_optimization = false;
 
   v8::Local<v8::FunctionTemplate> templ = v8::FunctionTemplate::New(
       isolate, ConstructCallback, v8::Local<v8::Value>(),

test/debugger/debug/side-effect/debug-evaluate-no-side-effect-runtime-check.js

@@ -2,6 +2,8 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
+// Flags: --no-enable-one-shot-optimization
+
 Debug = debug.Debug;
 
 // StaCurrentContextSlot

test/debugger/debug/side-effect/debug-evaluate-no-side-effect.js

@@ -2,6 +2,7 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
+// Flags: --no-enable-one-shot-optimization
 Debug = debug.Debug
 
 var exception = null;

test/mjsunit/regress/regress-crbug-380671.js

@@ -4,11 +4,15 @@
 
 // Flags: --mock-arraybuffer-allocator --expose-gc
 
-var buffer = new ArrayBuffer(0xc0000000);
-assertEquals(0xc0000000, buffer.byteLength);
-// We call the GC here to free up the large array buffer. Otherwise, the
-// mock allocator would allow us to allocate more than the physical memory
-// available on 32bit platforms, leaving the internal counters in an invalid
-// state.
-buffer = null;
-gc();
+function test() {
+  var buffer = new ArrayBuffer(0xc0000000);
+  assertEquals(0xc0000000, buffer.byteLength);
+  // We call the GC here to free up the large array buffer. Otherwise, the
+  // mock allocator would allow us to allocate more than the physical memory
+  // available on 32bit platforms, leaving the internal counters in an invalid
+  // state.
+  buffer = null;
+  gc();
+}
+
+test();