Rewrite LogDog log viewer app.

Rewrite LogDog log viewer app.

Rewrite the LogDog log viewer app in TypeScript, and integrate it into
the Gulp viewer. This adds many features, including:
- Actual TypeScript 2.0 versions of the LogDog libraries.
- The ability to perform log tails and manual (on-demand) log loading.
- Much faster log rendering (by default).
- Prefetch initial log chunk so it feels more responsive.
- Better model/view/controller separation.
- Transient HTTP and RPC retries.
- A tighter authentication loop, hopefully better now.
- Better support for mobile apps.

The current state of things is hybrid JavaScript / Polymer / TypeScript
because too much change in one CL is bad. The minimum bridges were built
between the JavaScript libraries that weren't rewritten and the
TypeScript code that uses them.

The TypeScript code is smaller, cleaner, and (obivously) stronger typed.
This should lead to its significantly-increased enhancement and
maintainability.

Polymer integrates TypeScript by including it during vulcanization. This
requires the Gulp build system to include an intermediate "build"
directory step, where the original sources are copied over and the
transpiled JavaScript code is generated.

BUG=chromium:667990
TEST=local
  - Ran the app locally, seems to work under all expected circumstances.

Review-Url: https://codereview.chromium.org/2543323004
Committed: https://github.com/luci/luci-go/commit/2cc68ad46a2c1c332d5d1d3b3b86cd3efd5c6281

[dnj, 2016-12-03 18:37:14]

Patchset #1 (id:1) has been deleted

[dnj, 2016-12-05 19:32:11]

dnj@chromium.org changed reviewers:
+ estaab@chromium.org, hinoka@chromium.org

[dnj, 2016-12-05 19:32:12]

Just throwing this up there for y'all. This is the TypeScript rewrite of the
LogDog log UI. The description has a lot of nice details about what changed.

[dnj, 2016-12-05 19:33:25]

Description was changed from

==========
Rewrite LogDog log viewer app.

Rewrite the LogDog log viewer app in TypeScript, and integrate it into
the Gulp viewer. This adds many features, including:
- Actual TypeScript 2.0 versions of the LogDog libraries.
- A true model/view/controller separation.
- Transient RPC retries.
- A tighter authentication loop, hopefully better now.
- The ability to perform log tails and manual (on-demand) log loading.
- Better support for mobile apps.

The current state of things is hybrid JavaScript / Polymer / TypeScript
because too much change in one CL is bad. The minimum bridges were built
between the JavaScript libraries that weren't rewritten and the
TypeScript code that uses them.

The TypeScript code is smaller, cleaner, and (obivously) stronger typed.
This should lead to its significantly-increased enhancement and
maintainability.

Polymer integrates TypeScript by including it during vulcanization. This
requires the Gulp build system to include an intermediate "build"
directory step, where the original sources are copied over and the
transpiled JavaScript code is generated.

BUG=chromium:667990
TEST=local
  - Ran the app locally, seems to work under all expected circumstances.
==========

to

==========
Rewrite LogDog log viewer app.

Rewrite the LogDog log viewer app in TypeScript, and integrate it into
the Gulp viewer. This adds many features, including:
- Actual TypeScript 2.0 versions of the LogDog libraries.
- The ability to perform log tails and manual (on-demand) log loading.
- Much faster log rendering (by default).
- Prefetch initial log chunk so it feels more responsive.
- Better model/view/controller separation.
- Transient HTTP and RPC retries.
- A tighter authentication loop, hopefully better now.
- Better support for mobile apps.

The current state of things is hybrid JavaScript / Polymer / TypeScript
because too much change in one CL is bad. The minimum bridges were built
between the JavaScript libraries that weren't rewritten and the
TypeScript code that uses them.

The TypeScript code is smaller, cleaner, and (obivously) stronger typed.
This should lead to its significantly-increased enhancement and
maintainability.

Polymer integrates TypeScript by including it during vulcanization. This
requires the Gulp build system to include an intermediate "build"
directory step, where the original sources are copied over and the
transpiled JavaScript code is generated.

BUG=chromium:667990
TEST=local
  - Ran the app locally, seems to work under all expected circumstances.
==========

[Ryan Tseng, 2016-12-06 03:38:10]

hinoka@google.com changed reviewers:
+ hinoka@google.com

[Ryan Tseng, 2016-12-06 03:38:11]

lgtm, tho it's a really superficial review.  Maybe wait for estaab too.

In general i like the typescript, feels better than javascript.

Also in general, I feel like it's a bit long for something that's supposed to
run simple queries and display lines.

https://codereview.chromium.org/2543323004/diff/60001/web/apps/gulp-common.js
File web/apps/gulp-common.js (right):

https://codereview.chromium.org/2543323004/diff/60001/web/apps/gulp-common.js...
web/apps/gulp-common.js:31: var hyd = require('hydrolysis');
I'm but a simple python engineer, but the number of javascript imports confuse
and frightens me

https://codereview.chromium.org/2543323004/diff/60001/web/apps/gulp-common.js...
web/apps/gulp-common.js:332: port: 8000,
is this configurable?

https://codereview.chromium.org/2543323004/diff/60001/web/apps/logdog-view/el...
File web/apps/logdog-view/elements/logdog-view/logdog-view.html (right):

https://codereview.chromium.org/2543323004/diff/60001/web/apps/logdog-view/el...
web/apps/logdog-view/elements/logdog-view/logdog-view.html:134: "only screen and
(max-width: 760px)").maches;
matches?

https://codereview.chromium.org/2543323004/diff/60001/web/apps/logdog-view/el...
web/apps/logdog-view/elements/logdog-view/logdog-view.html:144:
this._setMetadata(!!logdog.getQueryValue(queryString, "m", false));
!!?

https://codereview.chromium.org/2543323004/diff/60001/web/inc/logdog-stream-v...
File web/inc/logdog-stream-view/logdog-stream-view.html (right):

https://codereview.chromium.org/2543323004/diff/60001/web/inc/logdog-stream-v...
web/inc/logdog-stream-view/logdog-stream-view.html:847: return ( (playing) ?
is this backwards?

https://codereview.chromium.org/2543323004/diff/60001/web/inc/logdog-stream-v...
File web/inc/logdog-stream-view/query.ts (right):

https://codereview.chromium.org/2543323004/diff/60001/web/inc/logdog-stream-v...
web/inc/logdog-stream-view/query.ts:66: switch (params.streamType) {
default?

[dnj, 2016-12-06 03:41:42]

Feel free to suggest a place to cut code :)

[dnj, 2016-12-10 00:18:52]

The CQ bit was checked by dnj@chromium.org

[commit-bot: I haz the power, 2016-12-10 00:18:57]

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-12-10 00:24:56]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1481329132310660,
"parent_rev": "ad2e521dd75ba10725fe15a4943aa9deada2106b", "commit_rev":
"2cc68ad46a2c1c332d5d1d3b3b86cd3efd5c6281"}

[commit-bot: I haz the power, 2016-12-10 00:24:59]

Description was changed from

==========
Rewrite LogDog log viewer app.

Rewrite the LogDog log viewer app in TypeScript, and integrate it into
the Gulp viewer. This adds many features, including:
- Actual TypeScript 2.0 versions of the LogDog libraries.
- The ability to perform log tails and manual (on-demand) log loading.
- Much faster log rendering (by default).
- Prefetch initial log chunk so it feels more responsive.
- Better model/view/controller separation.
- Transient HTTP and RPC retries.
- A tighter authentication loop, hopefully better now.
- Better support for mobile apps.

The current state of things is hybrid JavaScript / Polymer / TypeScript
because too much change in one CL is bad. The minimum bridges were built
between the JavaScript libraries that weren't rewritten and the
TypeScript code that uses them.

The TypeScript code is smaller, cleaner, and (obivously) stronger typed.
This should lead to its significantly-increased enhancement and
maintainability.

Polymer integrates TypeScript by including it during vulcanization. This
requires the Gulp build system to include an intermediate "build"
directory step, where the original sources are copied over and the
transpiled JavaScript code is generated.

BUG=chromium:667990
TEST=local
  - Ran the app locally, seems to work under all expected circumstances.
==========

to

==========
Rewrite LogDog log viewer app.

Rewrite the LogDog log viewer app in TypeScript, and integrate it into
the Gulp viewer. This adds many features, including:
- Actual TypeScript 2.0 versions of the LogDog libraries.
- The ability to perform log tails and manual (on-demand) log loading.
- Much faster log rendering (by default).
- Prefetch initial log chunk so it feels more responsive.
- Better model/view/controller separation.
- Transient HTTP and RPC retries.
- A tighter authentication loop, hopefully better now.
- Better support for mobile apps.

The current state of things is hybrid JavaScript / Polymer / TypeScript
because too much change in one CL is bad. The minimum bridges were built
between the JavaScript libraries that weren't rewritten and the
TypeScript code that uses them.

The TypeScript code is smaller, cleaner, and (obivously) stronger typed.
This should lead to its significantly-increased enhancement and
maintainability.

Polymer integrates TypeScript by including it during vulcanization. This
requires the Gulp build system to include an intermediate "build"
directory step, where the original sources are copied over and the
transpiled JavaScript code is generated.

BUG=chromium:667990
TEST=local
  - Ran the app locally, seems to work under all expected circumstances.

Review-Url: https://codereview.chromium.org/2543323004
Committed:
https://github.com/luci/luci-go/commit/2cc68ad46a2c1c332d5d1d3b3b86cd3efd5c6281
==========

[commit-bot: I haz the power, 2016-12-10 00:25:01]

Committed patchset #3 (id:60001) as
https://github.com/luci/luci-go/commit/2cc68ad46a2c1c332d5d1d3b3b86cd3efd5c6281

[dnj, 2016-12-10 00:37:28]

I'm committing this because I'm using it in production and I want it stamped in
our repository.

We're upping its usage b/c of a P0 right now. I don't want to run code
review-only code while simultaneously depending on it for reals.

[dnj, 2016-12-13 23:35:02]

A revert of this CL (patchset #3 id:60001) has been created in
https://codereview.chromium.org/2570963003/ by dnj@chromium.org.

The reason for reverting is: TypeScript is not ready yet..