Skip to content

Remove singleton dependency from base::Clock#1463

Merged
hiroyuki-komatsu merged 1 commit intogoogle:masterfrom
ciceroaware:remove_singleton_from_clock
Apr 21, 2026
Merged

Remove singleton dependency from base::Clock#1463
hiroyuki-komatsu merged 1 commit intogoogle:masterfrom
ciceroaware:remove_singleton_from_clock

Conversation

@yukawa
Copy link
Copy Markdown
Collaborator

@yukawa yukawa commented Apr 7, 2026

Description

As part of our on-going efforts to remove the dependency on Singleton, this commit rewrites base::Clock without the Singleton class dependency.

This is purely mechanical refactoring, and there must be no observable behavioral change in production.

Issue IDs

N/A

Steps to test new behaviors (if any)

  • OS: All
  • Steps:
    1. All GitHub Actions still pass

Comment thread src/base/clock.cc Outdated
As part of our on-going efforts to remove the dependency on Singleton,
this commit rewrites base::Clock without the Singleton class dependency.

This is purely mechanical refactoring, and there must be no observable
behavioral change in production.
@yukawa yukawa force-pushed the remove_singleton_from_clock branch from c23f7eb to 3ea9687 Compare April 9, 2026 11:08
@hiroyuki-komatsu hiroyuki-komatsu merged commit 6700f7a into google:master Apr 21, 2026
1 check passed
@yukawa yukawa deleted the remove_singleton_from_clock branch April 21, 2026 10:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants