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

Issue 793212 link

Starred by 2 users

Issue metadata

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

Blocking:
issue 738190



Sign in to add a comment

Move CreateResource to LayerTreeResourceProvider

Project Member Reported by xing...@intel.com, Dec 8 2017

Issue description

Comment 1 by xing...@intel.com, Dec 8 2017

Owner: xing...@intel.com
Status: Assigned (was: Untriaged)

Comment 2 by xing...@intel.com, Dec 8 2017

Components: Internals>Compositing

Comment 3 by xing...@intel.com, Dec 11 2017

 Issue 793211  has been merged into this issue.
Project Member

Comment 4 by bugdroid1@chromium.org, Dec 21 2017

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

commit c9a1fec5f65f3613d10d3d799d81c3b4f35e925f
Author: Xu Xing <xing.xu@intel.com>
Date: Thu Dec 21 00:29:34 2017

[viz] Move CreateResource to LayerTree in ResourceProviderTest

Reference document:
https://docs.google.com/document/d/1MDrKY4ebi4kQMKOsSbVtDPQnMckja8PjVWut0tC3ynk

BUG= 793212 

Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel
Change-Id: If9cd17cc8f99a97574cdbd6dbd956a05dd38e710
Reviewed-on: https://chromium-review.googlesource.com/833750
Reviewed-by: danakj <danakj@chromium.org>
Commit-Queue: Xing Xu <xing.xu@intel.com>
Cr-Commit-Position: refs/heads/master@{#525532}
[modify] https://crrev.com/c9a1fec5f65f3613d10d3d799d81c3b4f35e925f/cc/resources/resource_provider_unittest.cc

Project Member

Comment 5 by bugdroid1@chromium.org, Jan 3 2018

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

commit 9339c8f545a812878a67584f9c0021306bb1f810
Author: Xu Xing <xing.xu@intel.com>
Date: Wed Jan 03 06:31:40 2018

[viz] Move CreateResource to LayerTree in RendererPixelTest and GLRendererTest

Goal is to create resource in LayerTreeResourceProvider and use mapped id in
DisplayResourceProvider.

Reference document:
https://docs.google.com/document/d/1MDrKY4ebi4kQMKOsSbVtDPQnMckja8PjVWut0tC3ynk

BUG= 793212 

Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel
Change-Id: I0b23f3e473eff1c6da45b6ed0ed9ab04f9e16eea
Reviewed-on: https://chromium-review.googlesource.com/816555
Reviewed-by: danakj <danakj@chromium.org>
Reviewed-by: Sunny Sachanandani <sunnyps@chromium.org>
Commit-Queue: Xing Xu <xing.xu@intel.com>
Cr-Commit-Position: refs/heads/master@{#526637}
[modify] https://crrev.com/9339c8f545a812878a67584f9c0021306bb1f810/cc/BUILD.gn
[modify] https://crrev.com/9339c8f545a812878a67584f9c0021306bb1f810/cc/test/pixel_test.cc
[add] https://crrev.com/9339c8f545a812878a67584f9c0021306bb1f810/cc/test/resource_provider_test_utils.cc
[add] https://crrev.com/9339c8f545a812878a67584f9c0021306bb1f810/cc/test/resource_provider_test_utils.h
[modify] https://crrev.com/9339c8f545a812878a67584f9c0021306bb1f810/components/viz/service/display/gl_renderer_unittest.cc
[modify] https://crrev.com/9339c8f545a812878a67584f9c0021306bb1f810/components/viz/service/display/renderer_pixeltest.cc

Project Member

Comment 6 by bugdroid1@chromium.org, Jan 8 2018

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

commit d3da46a330d25a4225440615734f43baff421de4
Author: Xu Xing <xing.xu@intel.com>
Date: Mon Jan 08 10:32:51 2018

[viz] Move resource's create and write APIs into LayerTree

Goal is to create and write resource in LayerTreeResourceProvider and use mapped id in
DisplayResourceProvider.

Reference document:
https://docs.google.com/document/d/1MDrKY4ebi4kQMKOsSbVtDPQnMckja8PjVWut0tC3ynk

BUG= 793212 

Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel
Change-Id: Ia9131f6dc1acd584b459b8f3d18640a07e9f8f73
Reviewed-on: https://chromium-review.googlesource.com/851843
Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Reviewed-by: danakj <danakj@chromium.org>
Commit-Queue: Xing Xu <xing.xu@intel.com>
Cr-Commit-Position: refs/heads/master@{#527599}
[modify] https://crrev.com/d3da46a330d25a4225440615734f43baff421de4/android_webview/browser/surfaces_instance.cc
[modify] https://crrev.com/d3da46a330d25a4225440615734f43baff421de4/cc/layers/heads_up_display_layer_impl.cc
[modify] https://crrev.com/d3da46a330d25a4225440615734f43baff421de4/cc/raster/bitmap_raster_buffer_provider.cc
[modify] https://crrev.com/d3da46a330d25a4225440615734f43baff421de4/cc/raster/gpu_raster_buffer_provider.cc
[modify] https://crrev.com/d3da46a330d25a4225440615734f43baff421de4/cc/raster/gpu_raster_buffer_provider.h
[modify] https://crrev.com/d3da46a330d25a4225440615734f43baff421de4/cc/raster/one_copy_raster_buffer_provider.cc
[modify] https://crrev.com/d3da46a330d25a4225440615734f43baff421de4/cc/raster/one_copy_raster_buffer_provider.h
[modify] https://crrev.com/d3da46a330d25a4225440615734f43baff421de4/cc/resources/display_resource_provider.cc
[modify] https://crrev.com/d3da46a330d25a4225440615734f43baff421de4/cc/resources/display_resource_provider.h
[modify] https://crrev.com/d3da46a330d25a4225440615734f43baff421de4/cc/resources/layer_tree_resource_provider.cc
[modify] https://crrev.com/d3da46a330d25a4225440615734f43baff421de4/cc/resources/layer_tree_resource_provider.h
[modify] https://crrev.com/d3da46a330d25a4225440615734f43baff421de4/cc/resources/resource_provider.cc
[modify] https://crrev.com/d3da46a330d25a4225440615734f43baff421de4/cc/resources/resource_provider.h
[modify] https://crrev.com/d3da46a330d25a4225440615734f43baff421de4/cc/resources/resource_provider_unittest.cc
[modify] https://crrev.com/d3da46a330d25a4225440615734f43baff421de4/cc/resources/video_resource_updater.cc
[modify] https://crrev.com/d3da46a330d25a4225440615734f43baff421de4/cc/test/fake_resource_provider.h
[modify] https://crrev.com/d3da46a330d25a4225440615734f43baff421de4/cc/test/pixel_test.cc
[modify] https://crrev.com/d3da46a330d25a4225440615734f43baff421de4/cc/tiles/tile_manager_unittest.cc
[modify] https://crrev.com/d3da46a330d25a4225440615734f43baff421de4/cc/trees/layer_tree_host_impl_unittest.cc
[modify] https://crrev.com/d3da46a330d25a4225440615734f43baff421de4/components/viz/service/display/display.cc
[modify] https://crrev.com/d3da46a330d25a4225440615734f43baff421de4/components/viz/service/display/display.h
[modify] https://crrev.com/d3da46a330d25a4225440615734f43baff421de4/components/viz/service/display/display_perftest.cc
[modify] https://crrev.com/d3da46a330d25a4225440615734f43baff421de4/components/viz/service/display/display_unittest.cc
[modify] https://crrev.com/d3da46a330d25a4225440615734f43baff421de4/components/viz/service/display/software_renderer_unittest.cc
[modify] https://crrev.com/d3da46a330d25a4225440615734f43baff421de4/components/viz/service/display_embedder/gpu_display_provider.cc
[modify] https://crrev.com/d3da46a330d25a4225440615734f43baff421de4/components/viz/service/frame_sinks/direct_layer_tree_frame_sink_unittest.cc
[modify] https://crrev.com/d3da46a330d25a4225440615734f43baff421de4/components/viz/test/test_layer_tree_frame_sink.cc
[modify] https://crrev.com/d3da46a330d25a4225440615734f43baff421de4/content/browser/compositor/gpu_process_transport_factory.cc
[modify] https://crrev.com/d3da46a330d25a4225440615734f43baff421de4/content/browser/renderer_host/compositor_impl_android.cc
[modify] https://crrev.com/d3da46a330d25a4225440615734f43baff421de4/content/renderer/android/synchronous_layer_tree_frame_sink.cc
[modify] https://crrev.com/d3da46a330d25a4225440615734f43baff421de4/ui/compositor/test/in_process_context_factory.cc

Comment 7 by xing...@intel.com, Jan 29 2018

Status: Fixed (was: Assigned)

Sign in to add a comment