Added connection history (minus the history data)

Added connection history (minus the history data)
Since this functionality is not yet available, I've removed the link to it on the home page, but I wanted to get it ready for when we have connection history data so that I can trim the CSS and get the strings translated.

BUG=115350
TEST=Manual

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=124556

[Jamie, 2012-03-02 00:36:46]

ptal

[garykac, 2012-03-02 00:53:56]

LGTM

http://codereview.chromium.org/9562044/diff/1/remoting/webapp/_locales/en/mes...
File remoting/webapp/_locales/en/messages.json (right):

http://codereview.chromium.org/9562044/diff/1/remoting/webapp/_locales/en/mes...
remoting/webapp/_locales/en/messages.json:116: "description": "Column header in
the connection history table showing the length of time for which a connection
was active, if available."
Do we have a translatable string that we show when this info is not available?

http://codereview.chromium.org/9562044/diff/1/remoting/webapp/connection_hist...
File remoting/webapp/connection_history.js (right):

http://codereview.chromium.org/9562044/diff/1/remoting/webapp/connection_hist...
remoting/webapp/connection_history.js:119: // space between them.
For the 2-column case, shouldn't the space between them be controlled by the
<td> padding and margin?

http://codereview.chromium.org/9562044/diff/1/remoting/webapp/remoting.js
File remoting/webapp/remoting.js (right):

http://codereview.chromium.org/9562044/diff/1/remoting/webapp/remoting.js#new...
remoting/webapp/remoting.js:45: remoting.connectionHistory = new
remoting.ConnectionHistory();
Comment out? Or do you want this here to make sure it inits correctly.

[Jamie, 2012-03-02 01:11:05]

http://codereview.chromium.org/9562044/diff/1/remoting/webapp/_locales/en/mes...
File remoting/webapp/_locales/en/messages.json (right):

http://codereview.chromium.org/9562044/diff/1/remoting/webapp/_locales/en/mes...
remoting/webapp/_locales/en/messages.json:116: "description": "Column header in
the connection history table showing the length of time for which a connection
was active, if available."
On 2012/03/02 00:53:56, garykac wrote:
> Do we have a translatable string that we show when this info is not available?

I did have one in an earlier revision, but I decided against singling out this
particular piece of information as being optional. If it turns out we can't get
this data in some cases, we can always fall back on display -- or something
similar (or just add a string if we can take afford the translation delay).

http://codereview.chromium.org/9562044/diff/1/remoting/webapp/connection_hist...
File remoting/webapp/connection_history.js (right):

http://codereview.chromium.org/9562044/diff/1/remoting/webapp/connection_hist...
remoting/webapp/connection_history.js:119: // space between them.
On 2012/03/02 00:53:56, garykac wrote:
> For the 2-column case, shouldn't the space between them be controlled by the
> <td> padding and margin?

<td> has no margin IIRC. I tried your suggestion of setting the widths of these
columns to 1px, which works as far as ensuring that the gap between them is
controllable, but you end up with too narrow a gap between the second and third
columns. A solution involving flexboxes in the first column is probably feasible
but doesn't seem worth it at this stage.

http://codereview.chromium.org/9562044/diff/1/remoting/webapp/remoting.js
File remoting/webapp/remoting.js (right):

http://codereview.chromium.org/9562044/diff/1/remoting/webapp/remoting.js#new...
remoting/webapp/remoting.js:45: remoting.connectionHistory = new
remoting.ConnectionHistory();
On 2012/03/02 00:53:56, garykac wrote:
> Comment out? Or do you want this here to make sure it inits correctly.

Done.