Don't do any special normalization if a boilerplate contains function literals.

src/parsing/parser.h

Index: src/parsing/parser.h
diff --git a/src/parsing/parser.h b/src/parsing/parser.h
index f5e8a7878ac4d0644f8d9ffc1bd691d4fd6ebc79..6cfa4bb83c02ec56d0c4ec431f5dfd3d27bdb27f 100644
--- a/src/parsing/parser.h
+++ b/src/parsing/parser.h
@@ -405,11 +405,10 @@ class ParserTraits {
   }
 
   static void CheckFunctionLiteralInsideTopLevelObjectLiteral(
-      Scope* scope, ObjectLiteralProperty* property, bool* has_function) {
+      Scope* scope, ObjectLiteralProperty* property) {
     Expression* value = property->value();
     if (scope->DeclarationScope()->is_script_scope() &&
         value->AsFunctionLiteral() != NULL) {
-      *has_function = true;
       value->AsFunctionLiteral()->set_pretenure();

[adamk, 2016/03/08 19:56:49]

Does the pretenuring still make sense? My understanding of this code was that
the pretenuring was for the exact purpose of setting the functions as constant
properties.

[Toon Verwaest, 2016/03/08 20:10:27]

Probably not. MigrateSlowToFast (for objects used as prototypes) doesn't seem to
care whether the functions are tenured.

     }
   }