[Chromoting] Log extension name and version number on startup.

[Chromoting] Log extension name and version number on startup.

BUG=none
TEST=none


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

[Jamie, 2012-03-01 22:35:41]

lgtm with a couple of suggestions.

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

http://codereview.chromium.org/9564029/diff/1/remoting/webapp/remoting.js#new...
remoting/webapp/remoting.js:77: remoting.logExtensionInfoAsync_();
I think this would be better at the start of the function, not least because
that way it will come out in the Me2Me session page as well.

http://codereview.chromium.org/9564029/diff/1/remoting/webapp/remoting.js#new...
remoting/webapp/remoting.js:93: ' (' + manifest.default_locale + ')');
I don't think I'd bother with the default_locale. It's unlikely ever to change.

[garykac, 2012-03-02 00:40:52]

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

http://codereview.chromium.org/9564029/diff/1/remoting/webapp/remoting.js#new...
remoting/webapp/remoting.js:77: remoting.logExtensionInfoAsync_();
On 2012/03/01 22:35:41, Jamie wrote:
> I think this would be better at the start of the function, not least because
> that way it will come out in the Me2Me session page as well.

Done.

http://codereview.chromium.org/9564029/diff/1/remoting/webapp/remoting.js#new...
remoting/webapp/remoting.js:93: ' (' + manifest.default_locale + ')');
On 2012/03/01 22:35:41, Jamie wrote:
> I don't think I'd bother with the default_locale. It's unlikely ever to
change.

Done.

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

lgtm

[commit-bot: I haz the power, 2012-03-02 13:29:47]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/garykac@chromium.org/9564029/5001

[commit-bot: I haz the power, 2012-03-02 16:51:34]

Change committed as 124666