Port the json_results_generator code over from Blink.

Port the json_results_generator code over from Blink.

The android tests use the python json_results_generator code from Blink
in order to upload results to the flakiness dashboard.

Previously they would import the code from Blink directly, creating a
cross-repo dependency on files that weren't really meant to be public.
In addition, eventually Blink stopped needing the JSONResultsGenerator
classes altogether, so the android scripts were the *only* users.

This patch moves the file over and strips the remaining webkitpy dependencies.

This patch also changes the license block to match the Chromium license.
I traced the commit history of these files; there were only ever two
commits by non-Googlers: one was a trivial variable rename by
rniwa after he had left Google, and the other was a change by zandobersek
which was removed as part of me stripping the webkitpy dependencies, so
I think this is safe.

R=ojan@chromium.org
BUG=242206
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=279515

[Dirk Pranke, 2014-06-20 01:13:42]

Take a look. This is the meat of the change.

Assuming this looks good, I'll reformat the python code to match chromium style
and land it.

[bulach, 2014-06-20 01:27:29]

cc jbudorick, he was investigating json-ifying and structuring the test results,
there maybe some overlap. or not. :)

[ojan, 2014-06-20 01:36:54]

LGTM. This is a step in the right direction.

This should really be changed to using the normal gtest code ASAP though:
https://code.google.com/p/chromium/codesearch#chromium/tools/build/scripts/sl....
We're going to be changing the JSON upload format in the coming couple weeks and
removing support for the old format, at which point this code will break.

If this can use that code instead, then it will naturally keep working as we
make changes to it.

[Dirk Pranke, 2014-06-20 01:37:30]

On 2014/06/20 01:27:29, bulach wrote:
> cc jbudorick, he was investigating json-ifying and structuring the test
results,
> there maybe some overlap. or not. :)

ojan is working on consolidating all of the test results for all of the test
types as part of the chromium tooling code yellow. This is a prelude to some of
that, so jbudorick should follow up w/ ojan.

[Dirk Pranke, 2014-06-20 01:38:10]

On 2014/06/20 01:36:54, OOO-only-code-yellow-reviews wrote:
> LGTM. This is a step in the right direction.
> 
> This should really be changed to using the normal gtest code ASAP though:
>
https://code.google.com/p/chromium/codesearch#chromium/tools/build/scripts/sl....
> We're going to be changing the JSON upload format in the coming couple weeks
and
> removing support for the old format, at which point this code will break.
> 
> If this can use that code instead, then it will naturally keep working as we
> make changes to it.

I'll let you take that one in a follow-up patch :).

[Dirk Pranke, 2014-06-20 01:38:46]

On 2014/06/20 01:38:10, Dirk Pranke wrote:

Marcus, can you give me OWNERS approval?

[commit-bot: I haz the power, 2014-06-20 01:40:15]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/dpranke@chromium.org/341143002/60001

[bulach, 2014-06-20 01:42:37]

lgtm for owners, thanks for breaking this dependency!

https://codereview.chromium.org/341143002/diff/60001/build/android/pylib/util...
File build/android/pylib/utils/json_results_generator.py (right):

https://codereview.chromium.org/341143002/diff/60001/build/android/pylib/util...
build/android/pylib/utils/json_results_generator.py:25: # Specify separators in
order to get compact encoding.
not sure how much should we care about chromi-fy this (i.e., 80cols, 2-space
indent, etc..).
anyways, a step in the right direction and breaks dependencies, we can address
the style later..

[ojan, 2014-06-20 01:43:38]

On 2014/06/20 at 01:38:10, dpranke wrote:
> On 2014/06/20 01:36:54, OOO-only-code-yellow-reviews wrote:
> > LGTM. This is a step in the right direction.
> > 
> > This should really be changed to using the normal gtest code ASAP though:
> >
https://code.google.com/p/chromium/codesearch#chromium/tools/build/scripts/sl....
> > We're going to be changing the JSON upload format in the coming couple weeks
and
> > removing support for the old format, at which point this code will break.
> > 
> > If this can use that code instead, then it will naturally keep working as we
> > make changes to it.
> 
> I'll let you take that one in a follow-up patch :).

My plan is to break android uploading when we remove support for the old format.
I suppose I should say that more clearly on the bug itself.

[Dirk Pranke, 2014-06-20 01:44:40]

The CQ bit was unchecked by dpranke@chromium.org

[Dirk Pranke, 2014-06-20 01:45:17]

On 2014/06/20 01:42:37, bulach wrote:
> not sure how much should we care about chromi-fy this (i.e., 80cols, 2-space
> indent, etc..).

I was planning to re-format before landing, so unchecking the CQ box :).

[Dirk Pranke, 2014-06-20 20:21:32]

The CQ bit was checked by dpranke@chromium.org

[commit-bot: I haz the power, 2014-06-20 20:23:40]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/dpranke@chromium.org/341143002/80001

[commit-bot: I haz the power, 2014-06-21 03:04:03]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_dbg/builds/19...)
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)
  win_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_rel/buil...)

[commit-bot: I haz the power, 2014-06-21 03:07:41]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-06-21 03:07:42]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)

[Dirk Pranke, 2014-06-24 19:02:37]

The CQ bit was checked by dpranke@chromium.org

[commit-bot: I haz the power, 2014-06-24 19:04:24]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/dpranke@chromium.org/341143002/100001

[commit-bot: I haz the power, 2014-06-24 22:15:14]

Change committed as 279515