diff options
| author |  Vladimir Levin <vmpstr@chromium.org> | 2018-01-18 12:35:11 -0800 |
|---|---|---|
| committer |  Skia Commit-Bot <skia-commit-bot@chromium.org> | 2018-01-19 21:17:56 +0000 |
| commit | ad6660a639aba236c9d30485da1cd2bb1292ef83 (patch) | |
| tree | 623609cd34aad6aa66e865fc58871a21b6d8be8b /src/core/SkGlobalInitialization_core.cpp | |
| parent | 16d0ad06b46841c78cce816406574314fab95c13 (diff) | |
Use equal_range for factory lookups
This patch uses equal_range instead of linear search to look up a
factory entry by name. This does require a sort, but the expected usage
is that the sort happens once and look ups happen many times.
This improves performance on Chromium's oop deserialization of
flattenables by about 10%
R=reed@chromium.org
Change-Id: I907f457a2ffb7d5b6d8261343099d982260b8415
Reviewed-on: https://skia-review.googlesource.com/96820
Reviewed-by: Mike Klein <mtklein@chromium.org>
Commit-Queue: Mike Klein <mtklein@chromium.org>
Diffstat (limited to 'src/core/SkGlobalInitialization_core.cpp')
| -rw-r--r-- | src/core/SkGlobalInitialization_core.cpp | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/core/SkGlobalInitialization_core.cpp b/src/core/SkGlobalInitialization_core.cpp index bc3e254bd0..f4f2ec8cb7 100644 --- a/src/core/SkGlobalInitialization_core.cpp +++ b/src/core/SkGlobalInitialization_core.cpp @@ -50,6 +50,9 @@ void SkFlattenable::PrivateInitializer::InitCore() { // Now initialize any optional/additional effects (implemented in src/ports) InitEffects(); + + // Finalize flattenable initialization. + SkFlattenable::Finalize(); }; void SkFlattenable::InitializeFlattenablesIfNeeded() { |