New issue
Advanced search Search tips

Issue 881510 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Sep 17
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: ----



Sign in to add a comment

The peeking launcher shows high on the screen

Project Member Reported by newcomer@chromium.org, Sep 6

Issue description

Chrome Version: ToT from 06AUG2018

What steps will reproduce the problem?
(1) With zero state enabled.
(2) open peeking launcher
(3) Show emojii keyboard.
(4) Click x in search box.
(5) Close onscreen keyboard.

What is the expected result?
launcher is peeking at regular height.

What happens instead?
Launcher is peeking at "Half" height



Please use labels and text to provide additional information.

If this is a regression (i.e., worked before), please consider using the
bisect tool (https://www.chromium.org/developers/bisect-builds-py) to help
us identify the root cause and more rapidly triage the issue.

For graphics-related bugs, please copy/paste the contents of the about:gpu
page at the end of this report.


 
Cc: newcomer@chromium.org
Owner: ginko@chromium.org
Similar to 882666
Status: Started (was: Untriaged)
Project Member

Comment 3 by bugdroid1@chromium.org, Sep 17

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/392f9653da8d1492beae5a27303934c7cfcf6aaf

commit 392f9653da8d1492beae5a27303934c7cfcf6aaf
Author: Kevin Strohbehn <ginko@google.com>
Date: Mon Sep 17 22:19:40 2018

a11y keyboard no longer causes peeking launcher to appear at half-height after close

Bug:  881510 
Change-Id: Ic75fb402df6dad4fd1c863a3475a09f45092ed6e
Reviewed-on: https://chromium-review.googlesource.com/1225798
Commit-Queue: Kevin Strohbehn <ginko@google.com>
Reviewed-by: Weidong Guo <weidongg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#591840}
[modify] https://crrev.com/392f9653da8d1492beae5a27303934c7cfcf6aaf/ash/app_list/views/app_list_view.cc
[modify] https://crrev.com/392f9653da8d1492beae5a27303934c7cfcf6aaf/ash/app_list/views/app_list_view.h
[modify] https://crrev.com/392f9653da8d1492beae5a27303934c7cfcf6aaf/ash/app_list/views/search_box_view.cc
[modify] https://crrev.com/392f9653da8d1492beae5a27303934c7cfcf6aaf/ash/app_list/views/search_box_view_unittest.cc

Status: Fixed (was: Started)
Labels: Merge-Request-70
Project Member

Comment 6 by sheriffbot@chromium.org, Sep 20

Labels: -Merge-Request-70 Merge-Review-70 Hotlist-Merge-Review
This bug requires manual review: M70 has already been promoted to the beta branch, so this requires manual review
Please contact the milestone owner if you have questions.
Owners: benmason@(Android), kariahda@(iOS), geohsu@(ChromeOS), abdulsyed@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 7 by bugdroid1@chromium.org, Sep 21

Labels: merge-merged-3538
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/7503e1b5362f49e83731a9562f1a20bf0eccd4ad

commit 7503e1b5362f49e83731a9562f1a20bf0eccd4ad
Author: Kevin Strohbehn <ginko@google.com>
Date: Fri Sep 21 22:04:00 2018

a11y keyboard no longer causes peeking launcher to appear at half-height after close

Bug:  881510 
Change-Id: Ic75fb402df6dad4fd1c863a3475a09f45092ed6e
Reviewed-on: https://chromium-review.googlesource.com/1225798
Commit-Queue: Kevin Strohbehn <ginko@google.com>
Reviewed-by: Weidong Guo <weidongg@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#591840}(cherry picked from commit 392f9653da8d1492beae5a27303934c7cfcf6aaf)
Reviewed-on: https://chromium-review.googlesource.com/1239264
Reviewed-by: Alex Newcomer <newcomer@chromium.org>
Cr-Commit-Position: refs/branch-heads/3538@{#569}
Cr-Branched-From: 79f7c91a2b2a2932cd447fa6f865cb6662fa8fa6-refs/heads/master@{#587811}
[modify] https://crrev.com/7503e1b5362f49e83731a9562f1a20bf0eccd4ad/ash/app_list/views/app_list_view.cc
[modify] https://crrev.com/7503e1b5362f49e83731a9562f1a20bf0eccd4ad/ash/app_list/views/app_list_view.h
[modify] https://crrev.com/7503e1b5362f49e83731a9562f1a20bf0eccd4ad/ash/app_list/views/search_box_view.cc
[modify] https://crrev.com/7503e1b5362f49e83731a9562f1a20bf0eccd4ad/ash/app_list/views/search_box_view_unittest.cc

Labels: CommitLog-Audit-Violation Merge-Without-Approval
Here's a summary of the rules that were executed: 
 - OnlyMergeApprovedChange: Rule Failed -- Revision 7503e1b5362f49e83731a9562f1a20bf0eccd4ad was merged to refs/branch-heads/3538 branch with no merge approval from a TPM! 
Please explain why this change was merged to the branch!
 - AcknowledgeMerge: Notification Required -- 
Labels: Merge-Merged-70-3538
The following revision refers to this bug: 
https://chromium.googlesource.com/chromium/src.git/+/7503e1b5362f49e83731a9562f1a20bf0eccd4ad

Commit: 7503e1b5362f49e83731a9562f1a20bf0eccd4ad
Author: ginko@google.com
Commiter: newcomer@chromium.org
Date: 2018-09-21 22:04:00 +0000 UTC

a11y keyboard no longer causes peeking launcher to appear at half-height after close

Bug:  881510 
Change-Id: Ic75fb402df6dad4fd1c863a3475a09f45092ed6e
Reviewed-on: https://chromium-review.googlesource.com/1225798
Commit-Queue: Kevin Strohbehn <ginko@google.com>
Reviewed-by: Weidong Guo <weidongg@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#591840}(cherry picked from commit 392f9653da8d1492beae5a27303934c7cfcf6aaf)
Reviewed-on: https://chromium-review.googlesource.com/1239264
Reviewed-by: Alex Newcomer <newcomer@chromium.org>
Cr-Commit-Position: refs/branch-heads/3538@{#569}
Cr-Branched-From: 79f7c91a2b2a2932cd447fa6f865cb6662fa8fa6-refs/heads/master@{#587811}
Labels: -Merge-Review-70 Merge-Approved-70
Project Member

Comment 11 by sheriffbot@chromium.org, Sep 28

Cc: geo...@google.com
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 12 by sheriffbot@chromium.org, Oct 2

This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Merge-Approved-70

Sign in to add a comment