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

Issue 812322 link

Starred by 2 users
Status: Available
Owner: ----
Cc:
danakj@chromium.org
samans@chromium.org
█ fsam...@chromium.org
Components:
EstimatedDays: 21
NextAction: 2018-06-14
OS: ----
Pri: 3
Type: Task

Blocking:
issue 730193



Sign in to add a comment

Cleanup content/browser/renderer_host after VizDisplayCompositor launches

Project Member Reported by kylec...@chromium.org, Feb 14 2018 
There is a lot of very confusing/convoluted CompositorFrame related code in content/browser/renderer_host. There is one code path for browser display compositor and another divergent code path for VizDisplayCompositor. We need to go back and clean this up after VizDisplayCompositor has launched.

Some classes in particular that need attention:
RenderWidgetHostImpl
RenderWidgetHostView*
DelegatedFrameHost

Comment 1 by monor...@bugs.chromium.org, Jun 14 2018 

The NextAction date has arrived: 2018-06-14
Project Member

Comment 2 by bugdroid1@chromium.org, Dec 10 

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

commit 1af3bb6d37fdbb89e5375d3502574cb9e58740e6
Author: Morten Stenshorne <mstensho@chromium.org>
Date: Mon Dec 10 11:16:30 2018

Remove virtual/feature-policy-vibrate leftovers.

The rest got removed in February:
https://chromium-review.googlesource.com/916807

TBR=iclelland@chromium.org

Bug: 812322
Change-Id: I3893988ceea7e7eb7e2dfa91903ea19dd5364b11
Reviewed-on: https://chromium-review.googlesource.com/c/1369929
Reviewed-by: Morten Stenshorne <mstensho@chromium.org>
Commit-Queue: Morten Stenshorne <mstensho@chromium.org>
Cr-Commit-Position: refs/heads/master@{#615086}
[modify] https://crrev.com/1af3bb6d37fdbb89e5375d3502574cb9e58740e6/third_party/blink/web_tests/FlagExpectations/enable-blink-features=BlinkGenPropertyTrees
[modify] https://crrev.com/1af3bb6d37fdbb89e5375d3502574cb9e58740e6/third_party/blink/web_tests/FlagExpectations/enable-blink-features=LayoutNG
[delete] https://crrev.com/46c6aa6b5f23277d624d8999b9987ec719a6f46c/third_party/blink/web_tests/virtual/feature-policy-vibrate/http/tests/feature-policy-vibrate/README.txt
[delete] https://crrev.com/46c6aa6b5f23277d624d8999b9987ec719a6f46c/third_party/blink/web_tests/virtual/feature-policy-vibrate/http/tests/feature-policy-vibrate/vibrate-allowed-by-container-policy-relocate-and-no-reload-expected.txt
[delete] https://crrev.com/46c6aa6b5f23277d624d8999b9987ec719a6f46c/third_party/blink/web_tests/virtual/feature-policy-vibrate/http/tests/feature-policy-vibrate/vibrate-disabled-expected.txt
[delete] https://crrev.com/46c6aa6b5f23277d624d8999b9987ec719a6f46c/third_party/blink/web_tests/virtual/feature-policy-vibrate/http/tests/feature-policy-vibrate/vibrate-enabledforself-expected.txt
►

Sign in to add a comment