[go: nahoru, domu]

Reland "[Resource file sharing]: Initialize BrowserInitParams before PreSandboxStartup in browser process"

This is a reland of commit fadc113a593e30ca2a3a0c00f5c8dd04af88af31

The diff between the original reverted change and the fix change can be
checked by Patchset1 -> Patchset 2.
BrowserInitParams class should be declared as COMPOENT_EXPORT since it is
used from chromeos/lacros library.

Original change's description:
> [Resource file sharing]: Initialize BrowserInitParams before PreSandboxStartup in browser process
>
> This CL changes when to initialize BrowserInitParams to
> PreSandboxStartup().
>
> This is a part of resource file sharing work which requires ash resource
> dir inside PreSandboxStartup().
> Ash resource dir path is given by BrowserInitParams, so needs to be
> initialized before or inside PreSandboxStartup() while it was
> initialized on PostEarlyInitialization() as a part of LacrosService in the previous implementation.
>
> This CL doesn't affect the behavior.
> This change will be used in the following CLs.
>
> Bug: 1253280
> Change-Id: I64c2454f09e2b4a667005d7eea9565f12c626f32
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3580292
> Reviewed-by: Hidehiko Abe <hidehiko@chromium.org>
> Reviewed-by: Anqing Zhao <anqing@chromium.org>
> Reviewed-by: Scott Violet <sky@chromium.org>
> Reviewed-by: Ryo Hashimoto <hashimoto@chromium.org>
> Commit-Queue: Eriko Kurimoto <elkurin@chromium.org>
> Cr-Commit-Position: refs/heads/main@{#993594}

Bug: 1253280
Change-Id: Idab9069fa704be42da641275d78e739e8c6cd10d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3592116
Reviewed-by: Hidehiko Abe <hidehiko@chromium.org>
Reviewed-by: Anqing Zhao <anqing@chromium.org>
Reviewed-by: Ryo Hashimoto <hashimoto@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: Eriko Kurimoto <elkurin@chromium.org>
Cr-Commit-Position: refs/heads/main@{#994159}
28 files changed
tree: 7d4ec6592b4239131d0861a51bcb22d988748678
  1. android_webview/
  2. apps/
  3. ash/
  4. base/
  5. build/
  6. build_overrides/
  7. buildtools/
  8. cc/
  9. chrome/
  10. chromecast/
  11. chromeos/
  12. codelabs/
  13. components/
  14. content/
  15. courgette/
  16. crypto/
  17. dbus/
  18. device/
  19. docs/
  20. extensions/
  21. fuchsia/
  22. gin/
  23. google_apis/
  24. google_update/
  25. gpu/
  26. headless/
  27. infra/
  28. ios/
  29. ipc/
  30. media/
  31. mojo/
  32. native_client_sdk/
  33. net/
  34. pdf/
  35. ppapi/
  36. printing/
  37. remoting/
  38. rlz/
  39. sandbox/
  40. services/
  41. skia/
  42. sql/
  43. storage/
  44. styleguide/
  45. testing/
  46. third_party/
  47. tools/
  48. ui/
  49. url/
  50. weblayer/
  51. .clang-format
  52. .clang-tidy
  53. .eslintrc.js
  54. .git-blame-ignore-revs
  55. .gitattributes
  56. .gitignore
  57. .gn
  58. .mailmap
  59. .rustfmt.toml
  60. .vpython
  61. .vpython3
  62. .yapfignore
  63. AUTHORS
  64. BUILD.gn
  65. CODE_OF_CONDUCT.md
  66. codereview.settings
  67. DEPS
  68. DIR_METADATA
  69. ENG_REVIEW_OWNERS
  70. LICENSE
  71. LICENSE.chromium_os
  72. OWNERS
  73. PRESUBMIT.py
  74. PRESUBMIT_test.py
  75. PRESUBMIT_test_mocks.py
  76. README.md
  77. WATCHLISTS
README.md

Logo Chromium

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.