commit | 3bf4670c97f449dcb41007944c3fa57cedd410e0 | [log] [tgz] |
---|---|---|
author | Glen Robertson <glenrob@chromium.org> | Thu May 16 03:13:42 2024 |
committer | Chromium LUCI CQ <chromium-scoped@luci-project-accounts.iam.gserviceaccount.com> | Thu May 16 03:13:42 2024 |
tree | 81f49992669b6a8be8f49d719867bf7c9c55446d | |
parent | e1353db2bad5d8603a1f7d53b5fccb113d629af7 [diff] |
WebApps: Use unique_ptr<WebAppInstallInfo> in manifest update code Previously this was using std::optional and moving stack-allocated WebAppInstallInfos around, which is costly (WebAppInstallInfo is a big struct) and different from our normal usage. This also helps with a wider refactor to remove the default constructor of WebAppInstallInfo. POC for this change: dmurph@ if glenrob@ is unavailable. Bug: b/280862254 Change-Id: I3950d10ba649cdf2cf5fba12281ffd4a0fc0736e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5534806 Auto-Submit: Glen Robertson <glenrob@chromium.org> Reviewed-by: Daniel Murphy <dmurph@chromium.org> Commit-Queue: Glen Robertson <glenrob@chromium.org> Cr-Commit-Position: refs/heads/main@{#1301735}
Chromium is an open-source browser project that aims to build a safer, faster, and more stable way for all users to experience the web.
The project's web site is https://www.chromium.org.
To check out the source code locally, don't use git clone
! Instead, follow the instructions on how to get the code.
Documentation in the source is rooted in docs/README.md.
Learn how to Get Around the Chromium Source Code Directory Structure.
For historical reasons, there are some small top level directories. Now the guidance is that new top level directories are for product (e.g. Chrome, Android WebView, Ash). Even if these products have multiple executables, the code should be in subdirectories of the product.
If you found a bug, please file it at https://crbug.com/new.