Handle tries of results data in upload (fix _is_directory).

Handle tries of results data in upload (fix _is_directory) for TestResultsServer

The file handler was getting confused when we would upload a full_results.json
that contained a "results" key that pointed to a trie of tests.

Also fix some file permissions issues that the presubmit check complained about.

R=ojan@chromium.org
BUG=403663
NOTRY=true

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=180310

[ojan, 2014-08-14 23:15:55]

lgtm

no test?

[Dirk Pranke, 2014-08-14 23:18:42]

On 2014/08/14 23:15:55, ojan-only-code-yellow-reviews wrote:
> lgtm
> 
> no test?

I will see if I can add one.

[Dirk Pranke, 2014-08-14 23:43:31]

Okay, I've added something close to a test, but I don't know if I have this
format close enough to reality to be realistic.

Please take another look?

[ojan, 2014-08-15 00:26:19]

The CQ bit was checked by ojan@chromium.org

[ojan, 2014-08-15 00:26:20]

lgtm

https://codereview.chromium.org/475503005/diff/40001/Tools/TestResultServer/h...
File Tools/TestResultServer/handlers/buildershandler_unittest.py (right):

https://codereview.chromium.org/475503005/diff/40001/Tools/TestResultServer/h...
Tools/TestResultServer/handlers/buildershandler_unittest.py:102: 'groups':
['@ToT Chromium', '@ToT Blink'],
Whoops. This was my fault. Thx for fixing. I really need to add a script to run
all the TestResultServer python tests.

[commit-bot: I haz the power, 2014-08-15 00:27:37]

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

[commit-bot: I haz the power, 2014-08-15 00:28:42]

Committed patchset #3 (40001) as 180310

[Dirk Pranke, 2014-08-15 00:30:49]

On 2014/08/15 00:26:20, ojan-only-code-yellow-reviews wrote:
>I really need to add a script to run all the TestResultServer python tests.

test-webkitpy will run the tests if you have the GAE SDK installed. If we have
the SDK checked in somewhere (we do, I think), we could just add that to
sys.path instead and always run it.

[ojan, 2014-08-15 00:49:02]

On 2014/08/15 at 00:30:49, dpranke wrote:
> On 2014/08/15 00:26:20, ojan-only-code-yellow-reviews wrote:
> >I really need to add a script to run all the TestResultServer python tests.
> 
> test-webkitpy will run the tests if you have the GAE SDK installed. If we have
the SDK checked in somewhere (we do, I think), we could just add that to
sys.path instead and always run it.

That'd be awesome.