Introduce runtime/platform directory for code shared between vm

Introduce runtime/platform directory for code shared between vm
and bin.

Cleaned up globals.h. Common parts now in platform/globals.h.
vm specific parts are in vm/globals.h and bin/globals.h is gone. 

R=sgjesse@google.com,iposva@google.com
BUG=
TEST=

Committed: https://code.google.com/p/dart/source/detail?r=3030

[Søren Gjesse, 2012-01-05 13:15:31]

lgtm

http://codereview.chromium.org/9114008/diff/2001/runtime/platform/globals.h
File runtime/platform/globals.h (right):

http://codereview.chromium.org/9114008/diff/2001/runtime/platform/globals.h#n...
runtime/platform/globals.h:44: #include "vm/c99_support_win.h"
Move these .h files to platform/ as well.

[Mads Ager (google), 2012-01-05 14:42:27]

http://codereview.chromium.org/9114008/diff/2001/runtime/platform/globals.h
File runtime/platform/globals.h (right):

http://codereview.chromium.org/9114008/diff/2001/runtime/platform/globals.h#n...
runtime/platform/globals.h:44: #include "vm/c99_support_win.h"
On 2012/01/05 13:15:32, Søren Gjesse wrote:
> Move these .h files to platform/ as well.

Yes! Thank you!

[Ivan Posva, 2012-01-07 00:26:46]

Some particular comments and it is the time of the year again where I start to
complain about out of date file headers: 2012?

-Ivan

http://codereview.chromium.org/9114008/diff/5001/runtime/bin/builtin.h
File runtime/bin/builtin.h (right):

http://codereview.chromium.org/9114008/diff/5001/runtime/bin/builtin.h#newcode12
runtime/bin/builtin.h:12: 
Why the extra line?

http://codereview.chromium.org/9114008/diff/5001/runtime/bin/dartutils.h
File runtime/bin/dartutils.h (right):

http://codereview.chromium.org/9114008/diff/5001/runtime/bin/dartutils.h#newc...
runtime/bin/dartutils.h:11: 
ditto, and other places

http://codereview.chromium.org/9114008/diff/5001/runtime/platform/c99_support...
File runtime/platform/c99_support_win.h (right):

http://codereview.chromium.org/9114008/diff/5001/runtime/platform/c99_support...
runtime/platform/c99_support_win.h:1: // Copyright (c) 2011, the Dart project
authors.  Please see the AUTHORS file
Please update the gypi files with the new location for this file.

http://codereview.chromium.org/9114008/diff/5001/runtime/platform/inttypes_su...
File runtime/platform/inttypes_support_win.h (right):

http://codereview.chromium.org/9114008/diff/5001/runtime/platform/inttypes_su...
runtime/platform/inttypes_support_win.h:1: // Copyright (c) 2011, the Dart
project authors.  Please see the AUTHORS file
ditto, update the gypi.

[Mads Ager (google), 2012-01-07 08:24:32]

Thanks for the comments Ivan. I have addressed them in a separate CL:
http://codereview.chromium.org/9124030

Sorry for the gyp mess left by this change. I have introduced a
platform_source.gypi file and included that in the other projects. At this point
it is included wherever builtin_sources.gypi and vm_sources.gypi are included.
When we start having implementation files and not just header files in this
directory we should split into two gypi files so we only get the implementation
in the vm library but the header files in all of them.

Since I was not consistent with updating the copyright notice for all files I
edited I have reverted those changes. I do think that we should update the
copyright notices in the same way that the chromium and v8 projects do. Please
see http://www.chromium.org/developers/coding-style

"All source files in the Chromium project must start with the following header.
If you modify an old file, you must update the copyright to the current year
(not a range, i.e. don't use "2009-2011", even if the original code did).  There
is no need to change the copyright on files that are not otherwise being
modified."

http://codereview.chromium.org/9114008/diff/5001/runtime/bin/builtin.h
File runtime/bin/builtin.h (right):

http://codereview.chromium.org/9114008/diff/5001/runtime/bin/builtin.h#newcode12
runtime/bin/builtin.h:12: 
On 2012/01/07 00:26:46, Ivan Posva wrote:
> Why the extra line?

I guess I found that easier to read. I'll remove the ones I introduced. I'm
pretty sure that there are other places where the includes from the "include"
directory has its own section.

http://codereview.chromium.org/9114008/diff/5001/runtime/bin/dartutils.h
File runtime/bin/dartutils.h (right):

http://codereview.chromium.org/9114008/diff/5001/runtime/bin/dartutils.h#newc...
runtime/bin/dartutils.h:11: 
On 2012/01/07 00:26:46, Ivan Posva wrote:
> ditto, and other places

Done.

http://codereview.chromium.org/9114008/diff/5001/runtime/platform/c99_support...
File runtime/platform/c99_support_win.h (right):

http://codereview.chromium.org/9114008/diff/5001/runtime/platform/c99_support...
runtime/platform/c99_support_win.h:1: // Copyright (c) 2011, the Dart project
authors.  Please see the AUTHORS file
On 2012/01/07 00:26:46, Ivan Posva wrote:
> Please update the gypi files with the new location for this file.

Done.

http://codereview.chromium.org/9114008/diff/5001/runtime/platform/inttypes_su...
File runtime/platform/inttypes_support_win.h (right):

http://codereview.chromium.org/9114008/diff/5001/runtime/platform/inttypes_su...
runtime/platform/inttypes_support_win.h:1: // Copyright (c) 2011, the Dart
project authors.  Please see the AUTHORS file
On 2012/01/07 00:26:46, Ivan Posva wrote:
> ditto, update the gypi.

Done.