Chromium Code Reviews
Help | Chromium Project | Gerrit Changes | Sign in
(72)

Issue 2879533005: Use new skia resource clearing API

Created:
7 months, 1 week ago by ericrk
Modified:
7 months ago
Reviewers:
vmiura
CC:
chromium-reviews, cc-bugs_chromium.org
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Use new skia resource clearing API This will clear most of the same data as the previous approach, but will preserver the glyph cache. Thrashing of the glyph cache appears to be a major source of jank, so hopefully this will improve performance. BUG=656320 CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -2 lines) Patch
M cc/output/context_cache_controller.cc View 1 chunk +3 lines, -2 lines 0 comments Download
Trybot results:  win_chromium_x64_rel_ng   linux_chromium_rel_ng   win_chromium_x64_rel_ng   linux_trusty_blink_rel   win_clang   win_chromium_rel_ng   mac_chromium_compile_dbg_ng   mac_chromium_rel_ng   win_chromium_compile_dbg_ng   ios-simulator-xcode-clang   ios-simulator   ios-device-xcode-clang   ios-device   linux_chromium_tsan_rel_ng   linux_chromium_rel_ng   linux_chromium_chromeos_rel_ng   linux_chromium_compile_dbg_ng   linux_chromium_chromeos_ozone_rel_ng   chromeos_amd64-generic_chromium_compile_only_ng   chromium_presubmit   linux_chromium_asan_rel_ng   chromeos_daisy_chromium_compile_only_ng   linux_android_rel_ng   android_n5x_swarming_rel   cast_shell_linux   cast_shell_android   android_cronet   android_arm64_dbg_recipe   android_clang_dbg_recipe   android_compile_dbg   linux_chromium_rel_ng   mac_chromium_rel_ng   linux_trusty_blink_rel   win_clang   win_chromium_x64_rel_ng   win_chromium_compile_dbg_ng   win_chromium_rel_ng   ios-simulator-xcode-clang   mac_chromium_compile_dbg_ng   mac_chromium_rel_ng   ios-device-xcode-clang   ios-simulator   linux_chromium_tsan_rel_ng   ios-device   linux_chromium_chromeos_rel_ng   linux_chromium_rel_ng   linux_chromium_compile_dbg_ng   linux_chromium_asan_rel_ng   linux_chromium_chromeos_ozone_rel_ng   chromium_presubmit   chromeos_daisy_chromium_compile_only_ng   cast_shell_linux   chromeos_amd64-generic_chromium_compile_only_ng   linux_android_rel_ng   cast_shell_android   android_n5x_swarming_rel   android_compile_dbg   android_cronet   android_clang_dbg_recipe   android_arm64_dbg_recipe 

Messages

Total messages: 11 (10 generated)
ericrk
7 months ago (2017-05-18 21:27:10 UTC) #9
FYI, this is the change - very small... Just want to make sure it behaves as
expected (need to try out the site you mentioned).

Powered by Google App Engine
This is Rietveld 0eb02b776