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

Issue 850546 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 732555



Sign in to add a comment

Viz/Android: Chrome refuses to update the page until you reopen the app

Project Member Reported by samans@chromium.org, Jun 7 2018

Issue description

Please see the attached video.
 
20180607_111642.mp4
11.9 MB View Download
Blocking: 732555

Comment 2 by samans@chromium.org, Jun 12 2018

Owner: samans@chromium.org
Status: Started (was: Available)
We get this error and no further DrawAndSwap happens. I'm investigating.

"glPostSubBufferCHROMIUM: command not supported by surface"

Comment 3 by samans@chromium.org, Jun 12 2018

Seems like GLES2DecoderImpl::HandlePostSubBufferCHROMIUM is never called in non-viz mode at all. I'm not sure why what viz does differently that causes it to be called.
Project Member

Comment 4 by bugdroid1@chromium.org, Jun 13 2018

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

commit ff8129e257e2f9963302e76c69011d5400205793
Author: Saman Sami <samans@chromium.org>
Date: Wed Jun 13 23:26:11 2018

Android/viz: Properly handle empty swaps

Android does not support partial swaps. Instead, when an empty swap
is requested we're supposed to commit overlay planes.

Bug:  850546 
Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel
Change-Id: If77156732b4f43872855a8ad676c42a5e9842678
Reviewed-on: https://chromium-review.googlesource.com/1099266
Reviewed-by: kylechar <kylechar@chromium.org>
Commit-Queue: Saman Sami <samans@chromium.org>
Cr-Commit-Position: refs/heads/master@{#567042}
[modify] https://crrev.com/ff8129e257e2f9963302e76c69011d5400205793/components/viz/service/BUILD.gn
[modify] https://crrev.com/ff8129e257e2f9963302e76c69011d5400205793/components/viz/service/display_embedder/gl_output_surface.cc
[modify] https://crrev.com/ff8129e257e2f9963302e76c69011d5400205793/components/viz/service/display_embedder/gl_output_surface.h
[add] https://crrev.com/ff8129e257e2f9963302e76c69011d5400205793/components/viz/service/display_embedder/gl_output_surface_android.cc
[add] https://crrev.com/ff8129e257e2f9963302e76c69011d5400205793/components/viz/service/display_embedder/gl_output_surface_android.h
[modify] https://crrev.com/ff8129e257e2f9963302e76c69011d5400205793/components/viz/service/display_embedder/gpu_display_provider.cc

Comment 5 by samans@chromium.org, Jun 13 2018

Status: Fixed (was: Started)

Sign in to add a comment