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

Issue 803243 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Jan 2018
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

Opening DevTools cause the page to stop rendering

Project Member Reported by einbinder@chromium.org, Jan 17 2018

Issue description

What steps will reproduce the problem?
(1) Visit http://todomvc.com/examples/vanillajs/
(2) Type something
(3) Open DevTools with f12
(4) Try to type another thing, the page stops showing your changes

I bisected this to https://chromium.googlesource.com/chromium/src/+/966d29e791e4811124037ae509754c4277391fd1

Reverting the patch fixes the problem for me locally.
 

Comment 1 by fsamuel@google.com, Jan 17 2018

I can't repro the bug. Could you please capture a video?
video2.ogv
3.7 MB View Download

Comment 3 by fsamuel@google.com, Jan 18 2018

Owner: fsam...@chromium.org
Taking ownership of this bug because I think it's surface sync related.
Project Member

Comment 4 by bugdroid1@chromium.org, Jan 18 2018

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

commit 3cb232c1a3c7776aa0ddb18088eda0d5701cb9f5
Author: Fady Samuel <fsamuel@chromium.org>
Date: Thu Jan 18 19:51:46 2018

Surface synchronization: Fix devtools bug

GetLatestInFlightSurface can return the primary surface ID. This can
cause a race where occasionally the primary surface will be closed making
a page unusable. This CL fixes the issue by skipping the primary surface
in GetLatestInFlightSurface.

Bug:  803243 ,  672962 
Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel
Change-Id: Icfc0b96847962e94002b8d4e2885684348232b03
Reviewed-on: https://chromium-review.googlesource.com/872454
Commit-Queue: Fady Samuel <fsamuel@chromium.org>
Reviewed-by: Saman Sami <samans@chromium.org>
Cr-Commit-Position: refs/heads/master@{#530233}
[modify] https://crrev.com/3cb232c1a3c7776aa0ddb18088eda0d5701cb9f5/components/viz/service/frame_sinks/surface_synchronization_unittest.cc
[modify] https://crrev.com/3cb232c1a3c7776aa0ddb18088eda0d5701cb9f5/components/viz/service/surfaces/surface_manager.cc

Status: Verified (was: Assigned)
Looks fixed. Thanks!

Sign in to add a comment