Introduce collective --harmony flag.

src/v8.cc

 // Copyright 2011 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 39 matching lines @@
 bool V8::has_been_setup_ = false;
 bool V8::has_been_disposed_ = false;
 bool V8::has_fatal_error_ = false;
 bool V8::use_crankshaft_ = true;
 
 static Mutex* entropy_mutex = OS::CreateMutex();
 static EntropySource entropy_source;
 
 
 bool V8::Initialize(Deserializer* des) {
+  // Setting --harmony implies all other harmony flags.
+  // TODO(rossberg): Is there a better place to put this?

[Steven, 2011/10/12 11:54:38]

This is indeed not such a cool place to put it. You could consider putting it at
the end of  FlagList::SetFlagsFromCommandLine in flags.cc or 
doing something in Shell::SetOptions in d8.cc. At least that would group this
code with other options/flags code.

[rossberg, 2011/10/12 12:06:56]

Well, I think flags.cc would be the wrong place to put it, since that should be
generic and hence oblivious to individual flags. (It would also create a
circular dependency, I suspect.)

D8 is also the wrong place, because then it will only work for D8. ;-)

Ideally, flags.cc would provide a generic way to express implications between
flags. But that is clearly overkill at this point

+  if (FLAG_harmony) {
+    FLAG_harmony_typeof = true;
+    FLAG_harmony_scoping = true;
+    FLAG_harmony_proxies = true;
+    FLAG_harmony_weakmaps = true;
+  }
+
   InitializeOncePerProcess();
 
   // The current thread may not yet had entered an isolate to run.
   // Note the Isolate::Current() may be non-null because for various
   // initialization purposes an initializing thread may be assigned an isolate
   // but not actually enter it.
   if (i::Isolate::CurrentPerIsolateThreadData() == NULL) {
     i::Isolate::EnterDefaultIsolate();
   }
 
@@ skipping 142 matching lines @@
 
   RuntimeProfiler::GlobalSetup();
 
   // Peephole optimization might interfere with deoptimization.
   FLAG_peephole_optimization = !use_crankshaft_;
 
   ElementsAccessor::InitializeOncePerProcess();
 }
 
 } }  // namespace v8::internal