Remove unused code in AppActivityProxy

Remove unused code in AppActivityProxy

BUG=None
TEST=None

Committed: https://crrev.com/e1cfa8469e59a23f68eeadae32c4f78cc89296cb
Cr-Commit-Position: refs/heads/master@{#299111}

[pkotwicz, 2014-10-03 21:37:49]

Patchset #1 (id:1) has been deleted

[pkotwicz, 2014-10-03 21:42:25]

pkotwicz@chromium.org changed reviewers:
+ skuhne@chromium.org

[pkotwicz, 2014-10-03 21:42:26]

Stefan, PTAL. I believe that this is no longer necessary as a result of
https://codereview.chromium.org/596623005

[pkotwicz, 2014-10-03 21:43:22]

Patchset #1 (id:20001) has been deleted

[Mr4D (OOO till 08-26), 2014-10-03 23:26:18]

https://codereview.chromium.org/626913003/diff/40001/athena/content/app_activ...
File athena/content/app_activity_proxy.cc (left):

https://codereview.chromium.org/626913003/diff/40001/athena/content/app_activ...
athena/content/app_activity_proxy.cc:75:
wm::ActivateWindow(window_list_provider->GetWindowList().back());
Are you sure this works now as intended? I have a browser test queued to test
this condition. Please wait...

[pkotwicz, 2014-10-08 02:20:07]

Ping?

[Mr4D (OOO till 08-26), 2014-10-08 03:48:00]

I have sent out the browser tests today and await their review. I will test
tomorrow morning if this can be removed and let you know.

[Mr4D (OOO till 08-26), 2014-10-08 14:06:29]

lgtm

With my upcoming change (CL 631333003) these lines can go. However in its
current setup this case can still happen under extreme conditions. I would
therefore not advice to land that yet.

https://codereview.chromium.org/626913003/diff/40001/athena/content/app_activ...
File athena/content/app_activity_proxy.cc (left):

https://codereview.chromium.org/626913003/diff/40001/athena/content/app_activ...
athena/content/app_activity_proxy.cc:75:
wm::ActivateWindow(window_list_provider->GetWindowList().back());
The way it is set up at the moment you cannot remove it without breaking certain
situations. However - after my browser test and other fixes have landed this can
be removed.

Problem: When the system is bogged down (over 90% resources used), you go into
overview mode and you select a previous application activity, it will not show,
but the app behind will (which is caused by the fact that the proxy's
destruction will cause an activation change).

[pkotwicz, 2014-10-10 14:02:40]

The CQ bit was checked by pkotwicz@chromium.org

[commit-bot: I haz the power, 2014-10-10 14:03:25]

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

[commit-bot: I haz the power, 2014-10-10 14:43:12]

Committed patchset #1 (id:40001)

[commit-bot: I haz the power, 2014-10-10 14:43:52]

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