Create HostList class to facilitate future work.

Create HostList class to facilitate future work.

BUG=None
TEST=Manual

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=110400

[Jamie, 2011-11-16 01:18:34]

PTAL.

[Wez, 2011-11-16 21:41:22]

LGTM, although it seems strange that the HostList isn't really the host list -
it isn't responsible for fetching and populating the host list - but rather it's
the _view_ of the host list, with the host list itself still being managed by
client_screen.js.

http://codereview.chromium.org/8568038/diff/1/remoting/webapp/me2mom/client_s...
File remoting/webapp/me2mom/client_screen.js (right):

http://codereview.chromium.org/8568038/diff/1/remoting/webapp/me2mom/client_s...
remoting/webapp/me2mom/client_screen.js:396: /** @type {remoting.Host} */
Does this annotation apply to the var definition?

http://codereview.chromium.org/8568038/diff/1/remoting/webapp/me2mom/home_scr...
File remoting/webapp/me2mom/home_screen.js (right):

http://codereview.chromium.org/8568038/diff/1/remoting/webapp/me2mom/home_scr...
remoting/webapp/me2mom/home_screen.js:56: function parseHostListResponse_(xhr) {
Would it make sense to move the XHR dispatch and parsing of the response into
remoting.hostList, too?

[Jamie, 2011-11-16 22:09:09]

http://codereview.chromium.org/8568038/diff/1/remoting/webapp/me2mom/client_s...
File remoting/webapp/me2mom/client_screen.js (right):

http://codereview.chromium.org/8568038/diff/1/remoting/webapp/me2mom/client_s...
remoting/webapp/me2mom/client_screen.js:396: /** @type {remoting.Host} */
On 2011/11/16 21:41:23, Wez wrote:
> Does this annotation apply to the var definition?

Yes. For some reason, JSCompiler can't infer the type of an array element from
the type of the array, possibly because it has to account for out-of-bounds
conditions.

http://codereview.chromium.org/8568038/diff/1/remoting/webapp/me2mom/home_scr...
File remoting/webapp/me2mom/home_screen.js (right):

http://codereview.chromium.org/8568038/diff/1/remoting/webapp/me2mom/home_scr...
remoting/webapp/me2mom/home_screen.js:56: function parseHostListResponse_(xhr) {
On 2011/11/16 21:41:23, Wez wrote:
> Would it make sense to move the XHR dispatch and parsing of the response into
> remoting.hostList, too?

You could make a case either way. I think I'd prefer to leave it here for now to
keep the HostList class clean while I implement new stuff. Since it is going to
have to do some xhr stuff to implement rename and delete, it probably makes
sense to move this stuff into it as well at a later date.