Navigation transitions (web to native app): Hide/Show transition elements (Chrome side)

Navigation transitions (web to native app): Hide/Show transition elements (Chrome side)

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

This is the Chrome side of the CL. The Blink side has been committed here: https://codereview.chromium.org/715653004/

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

[Zhen Wang, 2014-11-12 15:33:37]

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

[Zhen Wang, 2014-11-12 15:33:38]

ptal

[nasko, 2014-11-12 22:16:23]

On 2014/11/12 15:33:38, Zhen Wang wrote:
> ptal

Please have someone with more knowledge on android side to review the android
specific files. I'll stamp the review afterwards.

[Zhen Wang, 2014-11-12 22:18:47]

zhenw@chromium.org changed reviewers:
+ oysteine@chromium.org, sievers@chromium.org, simonhatch@google.com

[Zhen Wang, 2014-11-12 22:18:48]

Hi Daniel, can you take a look at the android part? Thanks!

[no sievers, 2014-11-12 23:21:10]

On 2014/11/12 22:18:48, Zhen Wang wrote:
> Hi Daniel, can you take a look at the android part? Thanks!

Seems fine. But can you let someone whose familiar with the overall architecture
of the feature review first?

[Zhen Wang, 2014-11-12 23:23:46]

Sure.

Hi Simon, can you take a look? I will ask Daniel and Nasko to take a look
afterwards.

[shatch, 2014-11-13 14:42:33]

On 2014/11/12 23:23:46, Zhen Wang wrote:
> Sure.
> 
> Hi Simon, can you take a look? I will ask Daniel and Nasko to take a look
> afterwards.

lg2m, I commented on the blink side of the patch, please take a look there.

[no sievers, 2014-11-13 20:37:36]

On 2014/11/13 14:42:33, shatch wrote:
> On 2014/11/12 23:23:46, Zhen Wang wrote:
> > Sure.
> > 
> > Hi Simon, can you take a look? I will ask Daniel and Nasko to take a look
> > afterwards.
> 
> lg2m, I commented on the blink side of the patch, please take a look there.

lgtm

[Zhen Wang, 2014-11-14 15:22:51]

The CQ bit was checked by zhenw@chromium.org

[commit-bot: I haz the power, 2014-11-14 15:23:43]

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

[commit-bot: I haz the power, 2014-11-14 15:27:50]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-11-14 15:27:51]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Zhen Wang, 2014-11-14 15:31:55]

Hi Nasko, can you take a look? Simon and Daniel are ok with it now. Thanks!

[nasko, 2014-11-15 00:07:42]

LGTM

[Zhen Wang, 2014-11-15 04:00:30]

The CQ bit was checked by zhenw@chromium.org

[commit-bot: I haz the power, 2014-11-15 04:01:16]

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

[commit-bot: I haz the power, 2014-11-15 04:04:26]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2014-11-15 04:05:18]

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