Commit 00813839 authored by Becca Hughes's avatar Becca Hughes Committed by Commit Bot

[Picture in Picture] Fix controls timer issue

PiP was having issues with hiding the controls because
we have a timer that will hide the controls. This timer
gets reset every time we interact with it. In one case
we actually stop the timer so it does not hide the controls
anymore.

BUG=897246

Change-Id: I4965b5c6cae0047ee0260131a16c9b346003bcec
Reviewed-on: https://chromium-review.googlesource.com/c/1292518Reviewed-by: default avatarapacible <apacible@chromium.org>
Commit-Queue: Becca Hughes <beccahughes@chromium.org>
Cr-Commit-Position: refs/heads/master@{#601302}
parent 62f7e225
......@@ -747,7 +747,7 @@ void OverlayWindowViews::OnMouseEvent(ui::MouseEvent* event) {
// If the user interacts with the window using a mouse, stop the timer to
// automatically hide the controls.
hide_controls_timer_.Stop();
hide_controls_timer_.Reset();
views::Widget::OnMouseEvent(event);
}
......
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