Navigation transitions (web to native app): Revert exit transition (Chrome side)

Navigation transitions (web to native app): Revert exit transition (Chrome side)

We need a way to revert exit transition and show hidden transition elements when coming back from native app (when back button is pressed).

This is the Chrome side of the CL. Blink side CL is here: https://codereview.chromium.org/653223002/

BUG=370696
Committed: https://crrev.com/5a70789ceb6b216e0835e36d3838635f45ba3589
Cr-Commit-Position: refs/heads/master@{#307621}

[Zhen Wang, 2014-12-08 05:02:45]

zhenw@chromium.org changed reviewers:
+ simonhatch@chromium.org

[Zhen Wang, 2014-12-08 05:02:46]

Hi Simon, can you take a look before I send to Daniel and Nasko? Thanks!

[shatch, 2014-12-08 14:10:16]

lg2m

https://codereview.chromium.org/775843004/diff/40001/content/public/android/j...
File
content/public/android/java/src/org/chromium/content_public/browser/WebContents.java
(right):

https://codereview.chromium.org/775843004/diff/40001/content/public/android/j...
content/public/android/java/src/org/chromium/content_public/browser/WebContents.java:166:
* Revert the effect of exit transition after it transits to activity.
nit: transitions?

[Zhen Wang, 2014-12-08 16:19:56]

zhenw@chromium.org changed reviewers:
+ nasko@chromium.org, sievers@chromium.org

[Zhen Wang, 2014-12-08 16:19:56]

Hi Daniel, can you take a look at the Java part?
Hi Nasko, can you take a look at the C++ part?

Thanks!

https://codereview.chromium.org/775843004/diff/40001/content/public/android/j...
File
content/public/android/java/src/org/chromium/content_public/browser/WebContents.java
(right):

https://codereview.chromium.org/775843004/diff/40001/content/public/android/j...
content/public/android/java/src/org/chromium/content_public/browser/WebContents.java:166:
* Revert the effect of exit transition after it transits to activity.
On 2014/12/08 14:10:15, shatch wrote:
> nit: transitions?

Done.

[nasko, 2014-12-10 00:08:55]

C++ part LGTM

[no sievers, 2014-12-10 00:17:17]

java lgtm

[Zhen Wang, 2014-12-10 01:44:09]

The CQ bit was checked by zhenw@chromium.org

[commit-bot: I haz the power, 2014-12-10 01:44:44]

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

[commit-bot: I haz the power, 2014-12-10 02:31:20]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2014-12-10 02:32:13]

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