[Findit] Fracas crash triage dashboard

[Findit] Fracas crash triage dashboard

Add dashboard handler and template

There are 5 filters for fracas crashes:
[start date, end date) filter

found suspects. (True, False)
has regression range. (True, False)
suspected cls triage status (Untriaged, Triaged-Correct, Triaged-Incorrect, Triaged-Unsure)
Regression range triage status (Untriaged, Triaged-Correct, Triaged-Incorrect, Triaged-Unsure)

The results are sorted by the requested time of the analysis.

Move _RemoveMicrosecondsFromDelta, _FormatTimedelta, _FormatDatetime to time_util.py under the common directory.

https://findit-for-me.googleplex.com/crash/fracas-dashboard

BUG=615267
Committed: https://chromium.googlesource.com/infra/infra/+/0a46c9494b0c8e09edb5aecda004ac78d493f9b3

[Sharu Jiang, 2016-06-07 01:16:43]

Description was changed from

==========
Fix nit.


Add test files.


Fracas dashboard


add triage_status


Add dashboard handler and template


fracas dashboard.

BUG=
==========

to

==========
[Findit] Fracas crash triage dashboard

Add dashboard handler and template

There are 5 filters for fracas crashes:
[start date, end date) filter

found suspects. (True, False)
has regression range. (True, False)
suspected cls triage status (Triaged, Triaged-Correct, Triaged-Incorrect,
Untriaged)
Regression range triage status (Triaged, Triaged-Correct, Triaged-Incorrect,
Untriaged)

The results are sorted by the requested time of the analysis.

Move _RemoveMicrosecondsFromDelta, _FormatTimedelta, _FormatDatetime to
time_util.py under the common directory.

BUG=615267
==========

[Sharu Jiang, 2016-06-07 01:16:43]

katesonia@chromium.org changed reviewers:
+ stgao@chromium.org

[Sharu Jiang, 2016-06-07 01:16:56]

PTAL :)

[stgao, 2016-06-08 17:26:13]

On 2016/06/07 01:16:56, sharu jiang wrote:
> PTAL :)

FYI: I'm going to review this CL now and not to wait for the UI and the bug fix.

[stgao, 2016-06-08 17:53:45]

I will review the html template more closely after I have a UI for reference as
we discussed yesterday.

https://codereview.chromium.org/2043973002/diff/1/appengine/findit/handlers/c...
File appengine/findit/handlers/crash/fracas_dashboard.py (right):

https://codereview.chromium.org/2043973002/diff/1/appengine/findit/handlers/c...
appengine/findit/handlers/crash/fracas_dashboard.py:33: """Shows a list of
Findit try job results and statuses in an HTML page."""
This is not updated. This one is for crash, not for try-job for waterfall
failures :)

https://codereview.chromium.org/2043973002/diff/1/appengine/findit/handlers/c...
appengine/findit/handlers/crash/fracas_dashboard.py:99: 'crash_identifiers':
base64.b64encode(
Why we need this? For identifying the crash?

If that is the case, we could use the key of the crash model instead.

https://codereview.chromium.org/2043973002/diff/1/appengine/findit/model/cras...
File appengine/findit/model/crash/crash_analysis.py (right):

https://codereview.chromium.org/2043973002/diff/1/appengine/findit/model/cras...
appengine/findit/model/crash/crash_analysis.py:84:
suspected_components_triage_history = ndb.JsonProperty(indexed=False)
Why we save the triage history separately? Can we merge them into one field?

https://codereview.chromium.org/2043973002/diff/1/appengine/findit/model/tria...
File appengine/findit/model/triage_status.py (right):

https://codereview.chromium.org/2043973002/diff/1/appengine/findit/model/tria...
appengine/findit/model/triage_status.py:1: TRIAGED = 'triaged'
Can we reuse model/result_status.py instead?

https://codereview.chromium.org/2043973002/diff/1/appengine/findit/templates/...
File appengine/findit/templates/crash/fracas_dashboard.html (right):

https://codereview.chromium.org/2043973002/diff/1/appengine/findit/templates/...
appengine/findit/templates/crash/fracas_dashboard.html:32:
window.location.replace(newUrl);
As this will refresh the page, can we just submit the form to the server side
instead of handling the parameters on the browser side?

That would be simpler.

https://codereview.chromium.org/2043973002/diff/1/appengine/findit/templates/...
appengine/findit/templates/crash/fracas_dashboard.html:75: <select
id="suspected_cls_triage_status" class=select_filter>
Where is the style definition for "select_filter"?

[Sharu Jiang, 2016-06-09 06:08:59]

Description was changed from

==========
[Findit] Fracas crash triage dashboard

Add dashboard handler and template

There are 5 filters for fracas crashes:
[start date, end date) filter

found suspects. (True, False)
has regression range. (True, False)
suspected cls triage status (Triaged, Triaged-Correct, Triaged-Incorrect,
Untriaged)
Regression range triage status (Triaged, Triaged-Correct, Triaged-Incorrect,
Untriaged)

The results are sorted by the requested time of the analysis.

Move _RemoveMicrosecondsFromDelta, _FormatTimedelta, _FormatDatetime to
time_util.py under the common directory.

BUG=615267
==========

to

==========
[Findit] Fracas crash triage dashboard

Add dashboard handler and template

There are 5 filters for fracas crashes:
[start date, end date) filter

found suspects. (True, False)
has regression range. (True, False)
suspected cls triage status (Triaged, Triaged-Correct, Triaged-Incorrect,
Untriaged)
Regression range triage status (Triaged, Triaged-Correct, Triaged-Incorrect,
Untriaged)

The results are sorted by the requested time of the analysis.

Move _RemoveMicrosecondsFromDelta, _FormatTimedelta, _FormatDatetime to
time_util.py under the common directory.

https://findit-for-me.googleplex.com/crash/fracas-dashboard

BUG=615267
==========

[Sharu Jiang, 2016-06-09 06:13:28]

updated the url in the description :)

https://codereview.chromium.org/2043973002/diff/1/appengine/findit/handlers/c...
File appengine/findit/handlers/crash/fracas_dashboard.py (right):

https://codereview.chromium.org/2043973002/diff/1/appengine/findit/handlers/c...
appengine/findit/handlers/crash/fracas_dashboard.py:33: """Shows a list of
Findit try job results and statuses in an HTML page."""
On 2016/06/08 17:53:45, stgao wrote:
> This is not updated. This one is for crash, not for try-job for waterfall
> failures :)

Oops, changed it.

https://codereview.chromium.org/2043973002/diff/1/appengine/findit/handlers/c...
appengine/findit/handlers/crash/fracas_dashboard.py:99: 'crash_identifiers':
base64.b64encode(
On 2016/06/08 17:53:45, stgao wrote:
> Why we need this? For identifying the crash?
> 
> If that is the case, we could use the key of the crash model instead.

Done.

https://codereview.chromium.org/2043973002/diff/1/appengine/findit/model/cras...
File appengine/findit/model/crash/crash_analysis.py (right):

https://codereview.chromium.org/2043973002/diff/1/appengine/findit/model/cras...
appengine/findit/model/crash/crash_analysis.py:84:
suspected_components_triage_history = ndb.JsonProperty(indexed=False)
On 2016/06/08 17:53:45, stgao wrote:
> Why we save the triage history separately? Can we merge them into one field?

Done.

https://codereview.chromium.org/2043973002/diff/1/appengine/findit/model/tria...
File appengine/findit/model/triage_status.py (right):

https://codereview.chromium.org/2043973002/diff/1/appengine/findit/model/tria...
appengine/findit/model/triage_status.py:1: TRIAGED = 'triaged'
On 2016/06/08 17:53:45, stgao wrote:
> Can we reuse model/result_status.py instead?

As we discussed, I think this is different, since I decouple the result status.

https://codereview.chromium.org/2043973002/diff/1/appengine/findit/templates/...
File appengine/findit/templates/crash/fracas_dashboard.html (right):

https://codereview.chromium.org/2043973002/diff/1/appengine/findit/templates/...
appengine/findit/templates/crash/fracas_dashboard.html:32:
window.location.replace(newUrl);
On 2016/06/08 17:53:45, stgao wrote:
> As this will refresh the page, can we just submit the form to the server side
> instead of handling the parameters on the browser side?
> 
> That would be simpler.

Will fix this in next patch :)

https://codereview.chromium.org/2043973002/diff/1/appengine/findit/templates/...
appengine/findit/templates/crash/fracas_dashboard.html:75: <select
id="suspected_cls_triage_status" class=select_filter>
On 2016/06/08 17:53:45, stgao wrote:
> Where is the style definition for "select_filter"?

Done.

[stgao, 2016-06-13 21:25:51]

https://codereview.chromium.org/2043973002/diff/60001/appengine/findit/handle...
File appengine/findit/handlers/crash/fracas_dashboard.py (right):

https://codereview.chromium.org/2043973002/diff/60001/appengine/findit/handle...
appengine/findit/handlers/crash/fracas_dashboard.py:16: from model import
analysis_status
Not used?

https://codereview.chromium.org/2043973002/diff/60001/appengine/findit/index....
File appengine/findit/index.yaml (right):

https://codereview.chromium.org/2043973002/diff/60001/appengine/findit/index....
appengine/findit/index.yaml:12: properties:
Why this is needed?

https://codereview.chromium.org/2043973002/diff/60001/appengine/findit/model/...
File appengine/findit/model/crash/fracas_crash_analysis.py (right):

https://codereview.chromium.org/2043973002/diff/60001/appengine/findit/model/...
appengine/findit/model/crash/fracas_crash_analysis.py:16: historical_metadata =
ndb.JsonProperty(indexed=False)
Note: this change won't be backward-compatible.

https://codereview.chromium.org/2043973002/diff/60001/appengine/findit/model/...
File appengine/findit/model/triage_status.py (right):

https://codereview.chromium.org/2043973002/diff/60001/appengine/findit/model/...
appengine/findit/model/triage_status.py:1: TRIAGED_UNSURE = 'triage-unsure'
Can we use number instead of strings to reduce the size of the entities?

https://codereview.chromium.org/2043973002/diff/60001/appengine/findit/model/...
appengine/findit/model/triage_status.py:1: TRIAGED_UNSURE = 'triage-unsure'
Also missing file header.

https://codereview.chromium.org/2043973002/diff/60001/appengine/findit/templa...
File appengine/findit/templates/crash/fracas_dashboard.html (right):

https://codereview.chromium.org/2043973002/diff/60001/appengine/findit/templa...
appengine/findit/templates/crash/fracas_dashboard.html:32:
window.location.replace(newUrl);
As in previous comment, can we let the form submit by itself?
This still cause a page reload instead an in-place refresh.

https://codereview.chromium.org/2043973002/diff/60001/appengine/findit/templa...
appengine/findit/templates/crash/fracas_dashboard.html:77: <option
value="triaged">Triaged</option>
As discussed, this should be triaged-unsure.

https://codereview.chromium.org/2043973002/diff/60001/appengine/findit/templa...
appengine/findit/templates/crash/fracas_dashboard.html:80: <option
value="untriaged">Untriaged</option>
Order this list by the expected frequency of usage.
Same for the other dropdown list.

https://codereview.chromium.org/2043973002/diff/60001/appengine/findit/templa...
appengine/findit/templates/crash/fracas_dashboard.html:86: <option
value="triaged">Triaged</option>
Same here.

https://codereview.chromium.org/2043973002/diff/60001/appengine/findit/templa...
appengine/findit/templates/crash/fracas_dashboard.html:92: <input type="submit"
id="submit"></input>
This shows "Submit" in button text, can we make it more clear?

https://codereview.chromium.org/2043973002/diff/60001/appengine/findit/templa...
appengine/findit/templates/crash/fracas_dashboard.html:111: <a
href="fracas-result-feedback?key={{fracas_crash.key}}"
target="_blank">{{fracas_crash.signature}}</a>
Use relative path "./relative-path?param=value" ?

https://codereview.chromium.org/2043973002/diff/60001/appengine/findit/templa...
File appengine/findit/templates/crash/fracas_result_feedback.html (right):

https://codereview.chromium.org/2043973002/diff/60001/appengine/findit/templa...
appengine/findit/templates/crash/fracas_result_feedback.html:1: <!DOCTYPE html>
Is this file supposed to be included in this CL?

[stgao, 2016-06-13 21:26:19]

BTW, the CL description needs some updating.

[Sharu Jiang, 2016-06-21 00:23:12]

Patchset #5 (id:80001) has been deleted

[Sharu Jiang, 2016-06-21 00:24:19]

Patchset #5 (id:100001) has been deleted

[Sharu Jiang, 2016-06-21 00:27:29]

Patchset #5 (id:120001) has been deleted

[Sharu Jiang, 2016-06-21 00:28:14]

https://codereview.chromium.org/2043973002/diff/60001/appengine/findit/handle...
File appengine/findit/handlers/crash/fracas_dashboard.py (right):

https://codereview.chromium.org/2043973002/diff/60001/appengine/findit/handle...
appengine/findit/handlers/crash/fracas_dashboard.py:16: from model import
analysis_status
On 2016/06/13 21:25:50, stgao wrote:
> Not used?

Done.

https://codereview.chromium.org/2043973002/diff/60001/appengine/findit/index....
File appengine/findit/index.yaml (right):

https://codereview.chromium.org/2043973002/diff/60001/appengine/findit/index....
appengine/findit/index.yaml:12: properties:
On 2016/06/13 21:25:51, stgao wrote:
> Why this is needed?

sorry, deleted it.

https://codereview.chromium.org/2043973002/diff/60001/appengine/findit/model/...
File appengine/findit/model/crash/fracas_crash_analysis.py (right):

https://codereview.chromium.org/2043973002/diff/60001/appengine/findit/model/...
appengine/findit/model/crash/fracas_crash_analysis.py:16: historical_metadata =
ndb.JsonProperty(indexed=False)
On 2016/06/13 21:25:51, stgao wrote:
> Note: this change won't be backward-compatible.

Right, however I deleted some old data, so it should work ;)

https://codereview.chromium.org/2043973002/diff/60001/appengine/findit/model/...
File appengine/findit/model/triage_status.py (right):

https://codereview.chromium.org/2043973002/diff/60001/appengine/findit/model/...
appengine/findit/model/triage_status.py:1: TRIAGED_UNSURE = 'triage-unsure'
On 2016/06/13 21:25:51, stgao wrote:
> Can we use number instead of strings to reduce the size of the entities?

Done.

https://codereview.chromium.org/2043973002/diff/60001/appengine/findit/model/...
appengine/findit/model/triage_status.py:1: TRIAGED_UNSURE = 'triage-unsure'
On 2016/06/13 21:25:51, stgao wrote:
> Also missing file header.

Done.

https://codereview.chromium.org/2043973002/diff/60001/appengine/findit/templa...
File appengine/findit/templates/crash/fracas_dashboard.html (right):

https://codereview.chromium.org/2043973002/diff/60001/appengine/findit/templa...
appengine/findit/templates/crash/fracas_dashboard.html:32:
window.location.replace(newUrl);
On 2016/06/13 21:25:51, stgao wrote:
> As in previous comment, can we let the form submit by itself?
> This still cause a page reload instead an in-place refresh.

Done, I still have to replace the url so if someone click refresh, he/she won't
lose the filter settings, use the pushState to change without reloading.

https://codereview.chromium.org/2043973002/diff/60001/appengine/findit/templa...
appengine/findit/templates/crash/fracas_dashboard.html:77: <option
value="triaged">Triaged</option>
On 2016/06/13 21:25:51, stgao wrote:
> As discussed, this should be triaged-unsure.

Done.

https://codereview.chromium.org/2043973002/diff/60001/appengine/findit/templa...
appengine/findit/templates/crash/fracas_dashboard.html:80: <option
value="untriaged">Untriaged</option>
On 2016/06/13 21:25:51, stgao wrote:
> Order this list by the expected frequency of usage.
> Same for the other dropdown list.

Done.

https://codereview.chromium.org/2043973002/diff/60001/appengine/findit/templa...
appengine/findit/templates/crash/fracas_dashboard.html:86: <option
value="triaged">Triaged</option>
On 2016/06/13 21:25:51, stgao wrote:
> Same here.

Done.

https://codereview.chromium.org/2043973002/diff/60001/appengine/findit/templa...
appengine/findit/templates/crash/fracas_dashboard.html:92: <input type="submit"
id="submit"></input>
On 2016/06/13 21:25:51, stgao wrote:
> This shows "Submit" in button text, can we make it more clear?

How about 'Submit filters'?

https://codereview.chromium.org/2043973002/diff/60001/appengine/findit/templa...
appengine/findit/templates/crash/fracas_dashboard.html:111: <a
href="fracas-result-feedback?key={{fracas_crash.key}}"
target="_blank">{{fracas_crash.signature}}</a>
On 2016/06/13 21:25:51, stgao wrote:
> Use relative path "./relative-path?param=value" ?

Done.

https://codereview.chromium.org/2043973002/diff/60001/appengine/findit/templa...
File appengine/findit/templates/crash/fracas_result_feedback.html (right):

https://codereview.chromium.org/2043973002/diff/60001/appengine/findit/templa...
appengine/findit/templates/crash/fracas_result_feedback.html:1: <!DOCTYPE html>
On 2016/06/13 21:25:51, stgao wrote:
> Is this file supposed to be included in this CL?

No, accidentally got in...

[Sharu Jiang, 2016-06-23 22:21:19]

Patchset #6 (id:160001) has been deleted

[Sharu Jiang, 2016-06-23 22:22:08]

katesonia@chromium.org changed reviewers:
+ mbarbella@chromium.org

[Sharu Jiang, 2016-06-23 22:22:27]

PTAL :)

[stgao, 2016-06-24 00:49:07]

lgtm once comments are addressed.

https://codereview.chromium.org/2043973002/diff/180001/appengine/findit/templ...
File appengine/findit/templates/crash/fracas_dashboard.html (right):

https://codereview.chromium.org/2043973002/diff/180001/appengine/findit/templ...
appengine/findit/templates/crash/fracas_dashboard.html:39: history.pushState({},
null, new_url);
If the purpose of this is to preserve the filter settings in the url, a "GET"
request in the form could do the same. Will that be simpler?

https://codereview.chromium.org/2043973002/diff/180001/appengine/findit/templ...
appengine/findit/templates/crash/fracas_dashboard.html:66: <option
value="1">True</option>
Yes/No instead? Same below.

https://codereview.chromium.org/2043973002/diff/180001/appengine/findit/templ...
appengine/findit/templates/crash/fracas_dashboard.html:95: <input type="submit"
id="submit" value="Submit Filters"></input>
Just "Filter" or "Refresh"?

[Sharu Jiang, 2016-06-28 00:11:46]

https://codereview.chromium.org/2043973002/diff/180001/appengine/findit/templ...
File appengine/findit/templates/crash/fracas_dashboard.html (right):

https://codereview.chromium.org/2043973002/diff/180001/appengine/findit/templ...
appengine/findit/templates/crash/fracas_dashboard.html:39: history.pushState({},
null, new_url);
On 2016/06/24 00:49:07, stgao wrote:
> If the purpose of this is to preserve the filter settings in the url, a "GET"
> request in the form could do the same. Will that be simpler?

The 'GET' will refresh the page, right? I think this may be more efficient.

https://codereview.chromium.org/2043973002/diff/180001/appengine/findit/templ...
appengine/findit/templates/crash/fracas_dashboard.html:66: <option
value="1">True</option>
On 2016/06/24 00:49:07, stgao wrote:
> Yes/No instead? Same below.

Done.

https://codereview.chromium.org/2043973002/diff/180001/appengine/findit/templ...
appengine/findit/templates/crash/fracas_dashboard.html:95: <input type="submit"
id="submit" value="Submit Filters"></input>
On 2016/06/24 00:49:07, stgao wrote:
> Just "Filter" or "Refresh"?

Done.

[stgao, 2016-06-28 16:40:21]

Again, the CL description is not up-to-date regarding the status of triaging.

https://codereview.chromium.org/2043973002/diff/180001/appengine/findit/templ...
File appengine/findit/templates/crash/fracas_dashboard.html (right):

https://codereview.chromium.org/2043973002/diff/180001/appengine/findit/templ...
appengine/findit/templates/crash/fracas_dashboard.html:39: history.pushState({},
null, new_url);
On 2016/06/28 00:11:46, sharu jiang wrote:
> On 2016/06/24 00:49:07, stgao wrote:
> > If the purpose of this is to preserve the filter settings in the url, a
"GET"
> > request in the form could do the same. Will that be simpler?
> 
> The 'GET' will refresh the page, right? I think this may be more efficient.

Line #38 above does a http get too. I don't think there will be a performance
difference here.

This is not a big issue. We could go with your approach, but let's add some
comment on how the code works here and the purpose.

After comments added, you could go ahead and commit this CL.

[Sharu Jiang, 2016-06-28 18:37:47]

Description was changed from

==========
[Findit] Fracas crash triage dashboard

Add dashboard handler and template

There are 5 filters for fracas crashes:
[start date, end date) filter

found suspects. (True, False)
has regression range. (True, False)
suspected cls triage status (Triaged, Triaged-Correct, Triaged-Incorrect,
Untriaged)
Regression range triage status (Triaged, Triaged-Correct, Triaged-Incorrect,
Untriaged)

The results are sorted by the requested time of the analysis.

Move _RemoveMicrosecondsFromDelta, _FormatTimedelta, _FormatDatetime to
time_util.py under the common directory.

https://findit-for-me.googleplex.com/crash/fracas-dashboard

BUG=615267
==========

to

==========
[Findit] Fracas crash triage dashboard

Add dashboard handler and template

There are 5 filters for fracas crashes:
[start date, end date) filter

found suspects. (True, False)
has regression range. (True, False)
suspected cls triage status (Untriaged, Triaged-Correct, Triaged-Incorrect,
Triaged-Unsure)
Regression range triage status (Untriaged, Triaged-Correct, Triaged-Incorrect,
Triaged-Unsure)

The results are sorted by the requested time of the analysis.

Move _RemoveMicrosecondsFromDelta, _FormatTimedelta, _FormatDatetime to
time_util.py under the common directory.

https://findit-for-me.googleplex.com/crash/fracas-dashboard

BUG=615267
==========

[Sharu Jiang, 2016-06-28 18:55:36]

The CQ bit was checked by katesonia@chromium.org

[Sharu Jiang, 2016-06-28 18:55:36]

The patchset sent to the CQ was uploaded after l-g-t-m from stgao@chromium.org
Link to the patchset: https://codereview.chromium.org/2043973002/#ps220001
(title: "Add comments.")

[commit-bot: I haz the power, 2016-06-28 18:55:45]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-06-28 19:00:15]

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

[commit-bot: I haz the power, 2016-06-28 19:00:16]

Try jobs failed on following builders:
  Infra Presubmit on master.tryserver.infra (JOB_FAILED,
https://build.chromium.org/p/tryserver.infra/builders/Infra%20Presubmit/build...)

[Sharu Jiang, 2016-06-29 18:24:31]

The CQ bit was checked by katesonia@chromium.org

[Sharu Jiang, 2016-06-29 18:24:33]

The patchset sent to the CQ was uploaded after l-g-t-m from stgao@chromium.org
Link to the patchset: https://codereview.chromium.org/2043973002/#ps260001
(title: "Surpass pylint errors.")

[commit-bot: I haz the power, 2016-06-29 18:24:39]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-06-29 18:40:58]

Description was changed from

==========
[Findit] Fracas crash triage dashboard

Add dashboard handler and template

There are 5 filters for fracas crashes:
[start date, end date) filter

found suspects. (True, False)
has regression range. (True, False)
suspected cls triage status (Untriaged, Triaged-Correct, Triaged-Incorrect,
Triaged-Unsure)
Regression range triage status (Untriaged, Triaged-Correct, Triaged-Incorrect,
Triaged-Unsure)

The results are sorted by the requested time of the analysis.

Move _RemoveMicrosecondsFromDelta, _FormatTimedelta, _FormatDatetime to
time_util.py under the common directory.

https://findit-for-me.googleplex.com/crash/fracas-dashboard

BUG=615267
==========

to

==========
[Findit] Fracas crash triage dashboard

Add dashboard handler and template

There are 5 filters for fracas crashes:
[start date, end date) filter

found suspects. (True, False)
has regression range. (True, False)
suspected cls triage status (Untriaged, Triaged-Correct, Triaged-Incorrect,
Triaged-Unsure)
Regression range triage status (Untriaged, Triaged-Correct, Triaged-Incorrect,
Triaged-Unsure)

The results are sorted by the requested time of the analysis.

Move _RemoveMicrosecondsFromDelta, _FormatTimedelta, _FormatDatetime to
time_util.py under the common directory.

https://findit-for-me.googleplex.com/crash/fracas-dashboard

BUG=615267

Committed:
https://chromium.googlesource.com/infra/infra/+/0a46c9494b0c8e09edb5aecda004a...
==========

[commit-bot: I haz the power, 2016-06-29 18:41:00]

Committed patchset #10 (id:260001) as
https://chromium.googlesource.com/infra/infra/+/0a46c9494b0c8e09edb5aecda004a...

[Sharu Jiang, 2016-10-29 07:21:15]

Patchset #3 (id:40001) has been deleted