New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 904617 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Nov 13
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Quick window switch does not load the web content

Project Member Reported by malaykeshav@chromium.org, Nov 13

Issue description

Open multiple windows in tablet mode.
Use the window switcher twice consecutively to switch window.

Expectation:
Window content loads immediately

Actual:
Window content takes about 5 seconds to load.
 
Delayed WebContent Loading.webm
4.0 MB View Download
Expectation - Switch window.webm
2.6 MB View Download
Cc: omrilio@chromium.org
Cc: bhthompson@chromium.org
Labels: Merge-Request-71
@bhthompson 
The change is only a reduction in the time delay and has been tested manually on device.
Project Member

Comment 4 by sheriffbot@chromium.org, Nov 13

Labels: -Merge-Request-71 Hotlist-Merge-Review Merge-Review-71
This bug requires manual review: M71 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), kbleicher@(ChromeOS), govind@(Desktop)

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

Comment 5 by bugdroid1@chromium.org, Nov 13

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

commit fde52fa598322a672348cbe06251c5b8d486a140
Author: Malay Keshav <malaykeshav@chromium.org>
Date: Tue Nov 13 01:27:09 2018

Reduce delay to unpause occlusion tracker after overview mode ends

This patch is a temporary fix to reduce the delay before occlusion
tracker is unpaused after overview mode is dismissed. It is a temporary
fix to load web contents immediately instead of waiting on the occlusion
tracker for 5 seconds.

Bug:  904617 
Change-Id: Ib1d4957638965fd34ffa719c057d234a058d7e08
Component: Window selector controller
Reviewed-on: https://chromium-review.googlesource.com/c/1332717
Reviewed-by: Bernie Thompson <bhthompson@chromium.org>
Cr-Commit-Position: refs/branch-heads/3578@{#647}
Cr-Branched-From: 4226ddf99103e493d7afb23a4c7902ee496108b6-refs/heads/master@{#599034}
[modify] https://crrev.com/fde52fa598322a672348cbe06251c5b8d486a140/ash/wm/overview/window_selector_controller.cc

Labels: Merge-Merged-71-3578
The following revision refers to this bug: 
https://chromium.googlesource.com/chromium/src.git/+/fde52fa598322a672348cbe06251c5b8d486a140

Commit: fde52fa598322a672348cbe06251c5b8d486a140
Author: malaykeshav@chromium.org
Commiter: bhthompson@chromium.org
Date: 2018-11-13 01:27:09 +0000 UTC

Reduce delay to unpause occlusion tracker after overview mode ends

This patch is a temporary fix to reduce the delay before occlusion
tracker is unpaused after overview mode is dismissed. It is a temporary
fix to load web contents immediately instead of waiting on the occlusion
tracker for 5 seconds.

Bug:  904617 
Change-Id: Ib1d4957638965fd34ffa719c057d234a058d7e08
Component: Window selector controller
Reviewed-on: https://chromium-review.googlesource.com/c/1332717
Reviewed-by: Bernie Thompson <bhthompson@chromium.org>
Cr-Commit-Position: refs/branch-heads/3578@{#647}
Cr-Branched-From: 4226ddf99103e493d7afb23a4c7902ee496108b6-refs/heads/master@{#599034}
Labels: -Hotlist-Merge-Review -Merge-Review-71 Merge-Approved-71
Project Member

Comment 8 by bugdroid1@chromium.org, Nov 13

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

commit 1d59705d647db7544f4d8e86ffd8f00870adeac6
Author: Malay Keshav <malaykeshav@chromium.org>
Date: Tue Nov 13 01:31:22 2018

Reduce delay to unpause occlusion tracker after overview mode ends

This patch is a temporary fix to reduce the delay before occlusion
tracker is unpaused after overview mode is dismissed. It is a temporary
fix to load web contents immediately instead of waiting on the occlusion
tracker for 5 seconds.

Bug:  904617 
Change-Id: Ib1d4957638965fd34ffa719c057d234a058d7e08
Component: Window selector controller
Reviewed-on: https://chromium-review.googlesource.com/c/1332778
Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
Commit-Queue: Malay Keshav <malaykeshav@chromium.org>
Cr-Commit-Position: refs/heads/master@{#607409}
[modify] https://crrev.com/1d59705d647db7544f4d8e86ffd8f00870adeac6/ash/wm/overview/window_selector_controller.cc

Status: Fixed (was: Assigned)
Project Member

Comment 10 by sheriffbot@chromium.org, Nov 16

Cc: bhthompson@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 11 by sheriffbot@chromium.org, Nov 19

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-71

Sign in to add a comment