Commit 879b37c6 authored by Alex Newcomer's avatar Alex Newcomer Committed by Commit Bot

cros: Fix unresponsive app list button

|back_and_app_list_background_| is interfering with event targeting and
causing the app list button to be unresponsive, specifically in the
right side of the button. NOTE: This only repros on device, not on the
emulator.

Prevent |back_and_app_list_background_| from recieving events to prevent
this.

Bug: 900632
Change-Id: Ib2cd8e51cb560d84307904c86ae46ecc388e2424
Reviewed-on: https://chromium-review.googlesource.com/c/1338723Reviewed-by: default avatarAlex Newcomer <newcomer@chromium.org>
Reviewed-by: default avatarMichael Wasserman <msw@chromium.org>
Commit-Queue: Alex Newcomer <newcomer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#608796}
parent 768c7e94
......@@ -374,6 +374,7 @@ void ShelfView::Init() {
// Add the background view behind the app list and back buttons first, so
// that other views will appear above it.
back_and_app_list_background_ = new views::View();
back_and_app_list_background_->set_can_process_events_within_subtree(false);
back_and_app_list_background_->SetBackground(
CreateBackgroundFromPainter(views::Painter::CreateSolidRoundRectPainter(
kShelfControlPermanentHighlightBackground,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment