[Webapp Refactor] Implement AppRemotingActivity.

[Webapp Refactor] Implement AppRemotingActivity.

This CL separates the session state changes handling logic from
AppRemoting.js into AppRemotingActivity, to make it more consistent with
CRD.

In short,
1. Extension logic are moved into AppConnectedView, as a protocol extension
   is only valid when the session is connected, which has exactly the same
   lifetime characteristics of AppConnectedView.
2. Connection State logic are moved into AppRemotingActivity.

BUG=476132
NOTRY=true

Committed: https://crrev.com/4a6581f6f483fb307aa041cca81284f945cb81bd
Cr-Commit-Position: refs/heads/master@{#325175}

[kelvinp, 2015-04-14 00:39:26]

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

[kelvinp, 2015-04-14 00:44:10]

Patchset #1 (id:1) has been deleted

[kelvinp, 2015-04-14 00:46:34]

PTAL

https://codereview.chromium.org/1082973002/diff/20001/remoting/webapp/app_rem...
File remoting/webapp/app_remoting/js/app_remoting.js (left):

https://codereview.chromium.org/1082973002/diff/20001/remoting/webapp/app_rem...
remoting/webapp/app_remoting/js/app_remoting.js:108: // TODO(jamiewalch): Remove
ClientSession's dependency on remoting.fullscreen
No longer needed, as we did such a good job cleanup globals :)

https://codereview.chromium.org/1082973002/diff/20001/remoting/webapp/app_rem...
remoting/webapp/app_remoting/js/app_remoting.js:130:
remoting.AppRemoting.prototype.startApplication_ = function(token) {
Moved to AppRemotingActivity

https://codereview.chromium.org/1082973002/diff/20001/remoting/webapp/app_rem...
remoting/webapp/app_remoting/js/app_remoting.js:207: * @param
{remoting.ConnectionInfo} connectionInfo
Moved to AppRemotingActivity

https://codereview.chromium.org/1082973002/diff/20001/remoting/webapp/app_rem...
remoting/webapp/app_remoting/js/app_remoting.js:252: /**
Moved to AppConnectedView

[garykac, 2015-04-15 00:52:37]

https://codereview.chromium.org/1082973002/diff/20001/remoting/webapp/app_rem...
File remoting/webapp/app_remoting/js/app_remoting.js (left):

https://codereview.chromium.org/1082973002/diff/20001/remoting/webapp/app_rem...
remoting/webapp/app_remoting/js/app_remoting.js:108: // TODO(jamiewalch): Remove
ClientSession's dependency on remoting.fullscreen
On 2015/04/14 00:46:33, kelvinp wrote:
> No longer needed, as we did such a good job cleanup globals :)

Wahoo!

https://codereview.chromium.org/1082973002/diff/20001/remoting/webapp/app_rem...
File remoting/webapp/app_remoting/js/app_remoting_activity.js (right):

https://codereview.chromium.org/1082973002/diff/20001/remoting/webapp/app_rem...
remoting/webapp/app_remoting/js/app_remoting_activity.js:113:
remoting.app_capabilities(), this);
Having this call out to the global might make this a little bit harder to test.
I think it's better to pass this in via AppRemoting (but I'm willing to be
convinced otherwise).

[garykac, 2015-04-15 01:01:23]

lgtm once the app_capabilities has been moved back as a param for AppRemoting.

https://codereview.chromium.org/1082973002/diff/20001/remoting/webapp/app_rem...
File remoting/webapp/app_remoting/js/ar_main.js (right):

https://codereview.chromium.org/1082973002/diff/20001/remoting/webapp/app_rem...
remoting/webapp/app_remoting/js/ar_main.js:14: remoting.app = new
remoting.AppRemoting();
Since this is the only "public" entry point, we need the capabilities to be
passed in here. We don't want clients to have to implement
remoting.app_capabilities() - that's a workaround so that we can share a single
ar_main.js file for the various AppRemoting apps.

[kelvinp, 2015-04-15 02:21:22]

The CQ bit was checked by kelvinp@chromium.org

[kelvinp, 2015-04-15 02:21:22]

The patchset sent to the CQ was uploaded after l-g-t-m from garykac@chromium.org
Link to the patchset: https://codereview.chromium.org/1082973002/#ps40001
(title: "Reviewer's feedback")

[commit-bot: I haz the power, 2015-04-15 02:21:27]

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

[commit-bot: I haz the power, 2015-04-15 02:25:24]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2015-04-15 02:26:22]

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