[Chromoting] Fix telemetry error in remoting.It2MeActivity

[Chromoting] Fix telemetry error in remoting.It2MeActivity

1. Excludes the access code entry time from the session duration.
2. Reports cancellation/error to the telemetry service if the
   main promise in It2meActivity.start() rejects.
3. Disposes DesktopRemotingActivity if the session fails to connect.

BUG=549658
Committed: https://crrev.com/cb61f37e5f706260e86e7e95e91596408cf78e9c
Cr-Commit-Position: refs/heads/master@{#358874}

[kelvinp, 2015-11-07 05:15:50]

Description was changed from

==========
[Chromoting] Fix telemetry error in remoting.It2MeActivity

1. Exclude the access code entry time from the session duration.
2. Report cancelation/error to the telemetry service if the
   promise returned in It2meActivity.start() rejects.
3. Dispose DesktopRemotingActivity if the session fails to connect.

BUG=549658
==========

to

==========
[Chromoting] Fix telemetry error in remoting.It2MeActivity

1. Excludes the access code entry time from the session duration.
2. Reports cancellation/error to the telemetry service if the
   main promise in It2meActivity.start() rejects.
3. Disposes DesktopRemotingActivity if the session fails to connect.

BUG=549658
==========

[kelvinp, 2015-11-07 05:15:51]

kelvinp@chromium.org changed reviewers:
+ garykac@chromium.org, jamiewalch@chromium.org

[kelvinp, 2015-11-07 05:15:58]

Patchset #1 (id:1) has been deleted

[kelvinp, 2015-11-07 05:18:45]

Jamie, PTAL
Adding Gary as Jamie will be OOO soon.

[Jamie, 2015-11-09 18:16:46]

lgtm

https://codereview.chromium.org/1413923015/diff/20001/remoting/webapp/crd/js/...
File remoting/webapp/crd/js/it2me_activity.js (right):

https://codereview.chromium.org/1413923015/diff/20001/remoting/webapp/crd/js/...
remoting/webapp/crd/js/it2me_activity.js:54:
base.dispose(this.desktopActivity_);
Do you ever expect desktopActivity to be non-null? If not, then consider
console.assert here, as it leaves the reader fewer combinations to think about.

[kelvinp, 2015-11-10 18:42:56]

The CQ bit was checked by kelvinp@chromium.org

[kelvinp, 2015-11-10 18:42:57]

The patchset sent to the CQ was uploaded after l-g-t-m from
jamiewalch@chromium.org
Link to the patchset: https://codereview.chromium.org/1413923015/#ps40001
(title: " ")

[commit-bot: I haz the power, 2015-11-10 18:44:15]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1413923015/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1413923015/40001

[commit-bot: I haz the power, 2015-11-10 19:27:01]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2015-11-10 19:28:01]

Patchset 2 (id:??) landed as
https://crrev.com/cb61f37e5f706260e86e7e95e91596408cf78e9c
Cr-Commit-Position: refs/heads/master@{#358874}