[Contextual Search] Fix Content visibility regression.

[Contextual Search] Fix Content visibility regression.

BUG=545604
BUG=545608
Committed: https://crrev.com/449c5aa0e9ab9cc209c1d1b72b34df93ad5e56bb
Cr-Commit-Position: refs/heads/master@{#355146}

[pedro (no code reviews), 2015-10-20 19:10:05]

Description was changed from

==========
[Contextual Search] Fix Content visibility regression.

BUG=545604
BUG=545608
==========

to

==========
[Contextual Search] Fix Content visibility regression.

BUG=545604
BUG=545608
==========

[pedro (no code reviews), 2015-10-20 19:10:05]

pedrosimonetti@chromium.org changed reviewers:
+ aurimas@chromium.org, donnd@chromium.org

[pedro (no code reviews), 2015-10-20 19:11:16]

Hey guys, please take a look at this change. It fixes a recently introduced
regression I want to merge this into M47.

I will add tests in a followup CL just for M48.

[Donn Denman, 2015-10-20 20:32:44]

lgtm

[aurimas (slooooooooow), 2015-10-20 20:45:01]

LGTM.

Developers usually list affected bugs with a single line like:
BUG=545604,545608

https://chromiumcodereview.appspot.com/1412223004/diff/1/chrome/android/java/...
File
chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/contextualsearch/ContextualSearchPanel.java
(left):

https://chromiumcodereview.appspot.com/1412223004/diff/1/chrome/android/java/...
chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/contextualsearch/ContextualSearchPanel.java:35:
private static final long INTERCEPT_NAVIGATION_PROMOTION_ANIMATION_DURATION_MS =
40;
Is this removal related to the rest of the patch?

[pedro (no code reviews), 2015-10-20 20:47:46]

https://codereview.chromium.org/1412223004/diff/1/chrome/android/java/src/org...
File
chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/contextualsearch/ContextualSearchPanel.java
(left):

https://codereview.chromium.org/1412223004/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/contextualsearch/ContextualSearchPanel.java:35:
private static final long INTERCEPT_NAVIGATION_PROMOTION_ANIMATION_DURATION_MS =
40;
On 2015/10/20 20:45:01, aurimas wrote:
> Is this removal related to the rest of the patch?

No, I just noticed this was not being used. This was added by the recent
refactoring.

[pedro (no code reviews), 2015-10-20 20:50:07]

The CQ bit was checked by pedrosimonetti@chromium.org

[commit-bot: I haz the power, 2015-10-20 20:51:07]

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

[commit-bot: I haz the power, 2015-10-20 20:57:51]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-10-20 20:58:35]

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