Added tools/bots/editor.py to build the editor (using gyp/build.py) and run editor tests

Added tools/bots/editor.py to build the editor (using gyp/build.py) and run editor tests

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

[ricow1, 2013-03-21 15:13:05]

LGTM
I think we should move the testing into the testing scripts - but let us talk
about that in person

https://codereview.chromium.org/12450026/diff/1/tools/bots/editor.py
File tools/bots/editor.py (right):

https://codereview.chromium.org/12450026/diff/1/tools/bots/editor.py#newcode2
tools/bots/editor.py:2: 
remove blank line

https://codereview.chromium.org/12450026/diff/1/tools/bots/editor.py#newcode14
tools/bots/editor.py:14: linux_path = os.path.join('out', configuration_dir,
'editor')
I am fine with us experimenting with this in this manner, but I think this
actually should be encapsulated in out tools/test.dart script somehow - that we
we only have these paths in one place (two if you count the gyp files)

https://codereview.chromium.org/12450026/diff/1/tools/bots/editor.py#newcode26
tools/bots/editor.py:26: return os.path.join(mac_path, executable)
we do the same thing in  both the if and the else clause

[kustermann, 2013-03-21 15:24:48]

https://codereview.chromium.org/12450026/diff/1/tools/bots/editor.py
File tools/bots/editor.py (right):

https://codereview.chromium.org/12450026/diff/1/tools/bots/editor.py#newcode2
tools/bots/editor.py:2: 
On 2013/03/21 15:13:05, ricow1 wrote:
> remove blank line

Done.

https://codereview.chromium.org/12450026/diff/1/tools/bots/editor.py#newcode14
tools/bots/editor.py:14: linux_path = os.path.join('out', configuration_dir,
'editor')
On 2013/03/21 15:13:05, ricow1 wrote:
> I am fine with us experimenting with this in this manner, but I think this
> actually should be encapsulated in out tools/test.dart script somehow - that
we
> we only have these paths in one place (two if you count the gyp files)

I know, it should be
 * ./tools/build.py -mrelease editor
 * ./tools/test.py -mrelease editor

https://codereview.chromium.org/12450026/diff/1/tools/bots/editor.py#newcode26
tools/bots/editor.py:26: return os.path.join(mac_path, executable)
On 2013/03/21 15:13:05, ricow1 wrote:
> we do the same thing in  both the if and the else clause
Thanks for catching this.

[kustermann, 2013-03-21 15:25:52]

Committed patchset #2 manually as r20319 (presubmit successful).