See the link to graphs below.
All graphs for this bug: https://chromeperf.appspot.com/group_report?bug_id=895377 (For debugging:) Original alerts at time of bug-filing: https://chromeperf.appspot.com/group_report?sid=f87deb3a948365a63c74b9ed00a15b49edab9c741b346b2414108bc2671aa5fc Bot(s) for this bug's original alert(s): mac-10_13_laptop_high_end-perf power.desktop - Benchmark documentation link: https://bit.ly/power-benchmarks
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/15c4f732e40000
📍 Found a significant difference after 1 commit. https://pinpoint-dot-chromeperf.appspot.com/job/15c4f732e40000 Skia: Enable ExplicitlyAllocateGPUResources & SortRenderTargets. by penghuang@chromium.org https://chromium.googlesource.com/chromium/src/+/0cb762482ab99e54be75368d8da949d52af00059 cpu_time_percentage: 0.1879 → 0.2398 (+0.05188) Understanding performance regressions: http://g.co/ChromePerformanceRegressions Benchmark documentation link: https://bit.ly/power-benchmarks
Hi Robert and Greg, could you please take a look? Thanks.
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/e1fc3396bdc0b6a7091e3ad318a0085cd0989a75 commit e1fc3396bdc0b6a7091e3ad318a0085cd0989a75 Author: Peng Huang <penghuang@chromium.org> Date: Mon Oct 15 20:50:31 2018 Revert "Skia: Enable ExplicitlyAllocateGPUResources & SortRenderTargets." This reverts commit 0cb762482ab99e54be75368d8da949d52af00059. Reason for revert: performance regressions Original change's description: > Skia: Enable ExplicitlyAllocateGPUResources & SortRenderTargets. > > Enable those two options which may improve skia performance. > > Bug: 894492 > > Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel > Change-Id: I80a1b9d1ce018d90ab91e5e676b6d09a7dc23d8d > Reviewed-on: https://chromium-review.googlesource.com/c/1273267 > Commit-Queue: Peng Huang <penghuang@chromium.org> > Reviewed-by: Antoine Labour <piman@chromium.org> > Reviewed-by: Robert Phillips <robertphillips@google.com> > Cr-Commit-Position: refs/heads/master@{#599321} TBR=penghuang@chromium.org,robertphillips@google.com,piman@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 894492 ,895408, 895384 , 895377 , 895369 , 895280 Change-Id: I657077091716a618d18a3d1109531e6b3819108c Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel Reviewed-on: https://chromium-review.googlesource.com/c/1281055 Commit-Queue: Peng Huang <penghuang@chromium.org> Reviewed-by: Peng Huang <penghuang@chromium.org> Cr-Commit-Position: refs/heads/master@{#599741} [modify] https://crrev.com/e1fc3396bdc0b6a7091e3ad318a0085cd0989a75/components/viz/service/gl/gpu_service_impl.cc [modify] https://crrev.com/e1fc3396bdc0b6a7091e3ad318a0085cd0989a75/components/viz/test/data/rotated_drop_shadow_filter_gl.png [modify] https://crrev.com/e1fc3396bdc0b6a7091e3ad318a0085cd0989a75/components/viz/test/data/rotated_filter_gl.png [modify] https://crrev.com/e1fc3396bdc0b6a7091e3ad318a0085cd0989a75/gpu/skia_bindings/grcontext_for_gles2_interface.cc [modify] https://crrev.com/e1fc3396bdc0b6a7091e3ad318a0085cd0989a75/third_party/WebKit/LayoutTests/platform/linux/compositing/masks/mask-with-added-filters-expected.png [modify] https://crrev.com/e1fc3396bdc0b6a7091e3ad318a0085cd0989a75/third_party/WebKit/LayoutTests/platform/linux/compositing/reflections/nested-reflection-anchor-point-expected.png [modify] https://crrev.com/e1fc3396bdc0b6a7091e3ad318a0085cd0989a75/third_party/WebKit/LayoutTests/platform/linux/compositing/reflections/nested-reflection-animated-expected.png [modify] https://crrev.com/e1fc3396bdc0b6a7091e3ad318a0085cd0989a75/third_party/WebKit/LayoutTests/platform/linux/media/video-layer-crash-expected.png [modify] https://crrev.com/e1fc3396bdc0b6a7091e3ad318a0085cd0989a75/third_party/WebKit/LayoutTests/platform/linux/virtual/video-surface-layer/media/video-layer-crash-expected.png [modify] https://crrev.com/e1fc3396bdc0b6a7091e3ad318a0085cd0989a75/third_party/WebKit/LayoutTests/platform/mac/compositing/reflections/nested-reflection-anchor-point-expected.png [modify] https://crrev.com/e1fc3396bdc0b6a7091e3ad318a0085cd0989a75/third_party/WebKit/LayoutTests/platform/mac/compositing/reflections/nested-reflection-animated-expected.png [modify] https://crrev.com/e1fc3396bdc0b6a7091e3ad318a0085cd0989a75/third_party/WebKit/LayoutTests/platform/mac/media/video-layer-crash-expected.png [delete] https://crrev.com/382f3853b1ed90c29d95225d30b2c021fec66c96/third_party/WebKit/LayoutTests/platform/mac/virtual/video-surface-layer/media/video-layer-crash-expected.png [modify] https://crrev.com/e1fc3396bdc0b6a7091e3ad318a0085cd0989a75/third_party/WebKit/LayoutTests/platform/win/compositing/masks/mask-with-added-filters-expected.png [modify] https://crrev.com/e1fc3396bdc0b6a7091e3ad318a0085cd0989a75/third_party/WebKit/LayoutTests/platform/win/compositing/reflections/nested-reflection-anchor-point-expected.png [modify] https://crrev.com/e1fc3396bdc0b6a7091e3ad318a0085cd0989a75/third_party/WebKit/LayoutTests/platform/win/compositing/reflections/nested-reflection-animated-expected.png [delete] https://crrev.com/382f3853b1ed90c29d95225d30b2c021fec66c96/third_party/WebKit/LayoutTests/platform/win/images/color-profile-layer-filter-expected.png [modify] https://crrev.com/e1fc3396bdc0b6a7091e3ad318a0085cd0989a75/third_party/WebKit/LayoutTests/platform/win/media/video-layer-crash-expected.png [modify] https://crrev.com/e1fc3396bdc0b6a7091e3ad318a0085cd0989a75/third_party/WebKit/LayoutTests/platform/win/virtual/exotic-color-space/images/color-profile-layer-filter-expected.png [modify] https://crrev.com/e1fc3396bdc0b6a7091e3ad318a0085cd0989a75/third_party/WebKit/LayoutTests/platform/win/virtual/video-surface-layer/media/video-layer-crash-expected.png [add] https://crrev.com/e1fc3396bdc0b6a7091e3ad318a0085cd0989a75/third_party/WebKit/LayoutTests/virtual/video-surface-layer/media/video-layer-crash-expected.png
Comment 1 by 42576172...@developer.gserviceaccount.com
, Oct 15