New issue
Advanced search Search tips

Issue 776292 link

Starred by 1 user
Status: Fixed
Owner:
eseckler@chromium.org
Closed: Oct 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

New content rendering timeout interferes with deterministic rendering

Project Member Reported by eseckler@chromium.org, Oct 19 2017 
In e.g. some headless environments, we care about deterministic
rendering results. The new content timeout in RenderWidgetHostImpl can affect the page rendering by removing the active CompositorFrame in its frame sink. This interferes e.g. with the --run-all-compositor-stages-before-draw mode, where we long delays between CompositorFrame submissions are common.

Therefore, we'd like a way to disable the new content timeout, e.g. via a command line flag.
Project Member

Comment 1 by bugdroid1@chromium.org, Oct 19 2017 

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

commit b2158cecac2d6924556991b30b15ff53205b06f9
Author: Eric Seckler <eseckler@chromium.org>
Date: Thu Oct 19 08:11:53 2017

content: Add switch to disable new content rendering timeout.

In e.g. some headless environments, we care about deterministic
rendering results. As the new content rendering timeout can affect page
rendering, we'd like a way to disable it.

This patch adds a command line switch to that effect.

Change-Id: Ib0c09aa57df82665ffb4f1290c8ad5d2761aa7bf
Bug:  776292 
Reviewed-on: https://chromium-review.googlesource.com/725324
Commit-Queue: Eric Seckler <eseckler@chromium.org>
Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#510030}
[modify] https://crrev.com/b2158cecac2d6924556991b30b15ff53205b06f9/content/browser/renderer_host/render_widget_host_impl.cc
[modify] https://crrev.com/b2158cecac2d6924556991b30b15ff53205b06f9/content/public/common/content_switches.cc
[modify] https://crrev.com/b2158cecac2d6924556991b30b15ff53205b06f9/content/public/common/content_switches.h

Comment 2 by eseckler@chromium.org, Oct 19 2017

Status: Fixed (was: Started)
►

Sign in to add a comment