commit | 37b8239bec36c4cba1ad3f89d62babd97e4efd27 | [log] [tgz] |
---|---|---|
author | Kevin Lubick <kjlubick@google.com> | Wed Feb 23 12:17:12 2022 -0500 |
committer | Kevin Lubick <kjlubick@google.com> | Wed Feb 23 22:46:44 2022 +0000 |
tree | 39c90d2514dbaa777ba59e62a22c5146f7d70e16 | |
parent | 96df1183ac4de65bd3a8b3c207418703abdb0b94 [diff] |
[includes] Remove include link between SkPathRef.h and SkRRect.h According to go/chrome-includes [1], this will save about 40MB (0.02%) off the Chrome build. http://screen/4GnPjFaYpwCVHVL I'm not quite sure why the link is so expensive, but we can forward declare it and move the implementation from the .h to the .cpp file easily enough. [1] https://commondatastorage.googleapis.com/chromium-browser-clang/include-analysis.html#view=edges&filter=%5Ethird_party%2Fskia%2Finclude%2Fprivate%2FSkPathRef%5C.h%24&sort=includes&reverse=&includer=%5Ethird_party%2Fskia%2Finclude%2Fprivate%2FSkPathRef%5C.h%24&included=&limit=1000 Canary-Chromium-CL: 3485346 Change-Id: Ie3a5e7a735426f883a6c06ddbd3b8bf148bf1709 Bug: 242216 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/512158 Reviewed-by: Robert Phillips <robertphillips@google.com>