Don't use resolved url for instance identity in ApplicationManager

Don't use resolved url for instance identity in ApplicationManager

R=yzshen@chromium.org
http://crbug.com/533085

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

Committed: https://crrev.com/79341ad47c83799fbb10382c9599eeaa781f3620
Cr-Commit-Position: refs/heads/master@{#349608}

[Ben Goodger (Google), 2015-09-17 21:02:28]

ben@chromium.org changed reviewers:
+ yzshen@chromium.org

[yzshen1, 2015-09-17 22:38:11]

So happy to see this change. :)

LGTM

https://codereview.chromium.org/1346143004/diff/40001/mojo/shell/application_...
File mojo/shell/application_manager.cc (right):

https://codereview.chromium.org/1346143004/diff/40001/mojo/shell/application_...
mojo/shell/application_manager.cc:85: // NOTE: Set URL info using
|original_url_request| instead of |original_url|
Please remove this comment.

https://codereview.chromium.org/1346143004/diff/40001/mojo/shell/application_...
mojo/shell/application_manager.cc:167: // Also, it's possible the original URL
was redirected to an app that is
This part of comment could be removed -- the redirect case is handled on line
149 instead of here.

[Ben Goodger (Google), 2015-09-17 22:40:35]

The CQ bit was checked by ben@chromium.org

[Ben Goodger (Google), 2015-09-17 22:40:35]

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

[commit-bot: I haz the power, 2015-09-17 22:41:42]

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

[commit-bot: I haz the power, 2015-09-17 23:41:07]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-09-17 23:41:40]

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

[Justin Donnelly, 2015-09-18 00:42:39]

A revert of this CL (patchset #4 id:60001) has been created in
https://codereview.chromium.org/1354813002/ by jdonnelly@chromium.org.

The reason for reverting is: Need to revert in order to revert 

https://codereview.chromium.org/1351963002/

which broke

https://build.chromium.org/p/chromium.linux/builders/Android%20GN.

[Ben Goodger (Google), 2015-09-18 02:57:49]

The CQ bit was checked by ben@chromium.org

[Ben Goodger (Google), 2015-09-18 02:57:49]

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

[commit-bot: I haz the power, 2015-09-18 02:57:58]

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

[commit-bot: I haz the power, 2015-09-18 03:59:43]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2015-09-18 04:00:07]

Patchset 5 (id:??) landed as
https://crrev.com/79341ad47c83799fbb10382c9599eeaa781f3620
Cr-Commit-Position: refs/heads/master@{#349608}