Closed
Bug 1983606
Opened 1 year ago
Closed 1 year ago
Replace remaining Cu.now() calls with ChromeUtils.now()
Categories
(Firefox :: General, task)
Firefox
General
Tracking
()
RESOLVED
FIXED
144 Branch
| Tracking | Status | |
|---|---|---|
| firefox144 | --- | fixed |
People
(Reporter: florian, Assigned: florian)
References
(Blocks 1 open bug)
Details
Attachments
(1 external file, 1 obsolete file)
ChromeUtils.now() was implemented in bug 1982887. To avoid possible future confusion, let's remove the old Cu.now() that wasn't available in worker contexts.
| Assignee | ||
Comment 1•1 year ago
|
||
Patch generated using this command: rg "Cu.now()" --files-with-matches | grep -v "obj-" | xargs sed -i 's/Cu.now()/ChromeUtils.now()/g'
| Assignee | ||
Comment 2•1 year ago
|
||
Updated•1 year ago
|
Summary: Remove Cu.now() ni favor of ChromeUtils.now() → Remove Cu.now() in favor of ChromeUtils.now()
| Assignee | ||
Updated•1 year ago
|
No longer blocks: 1985100
Component: XPConnect → General
Product: Core → Firefox
Summary: Remove Cu.now() in favor of ChromeUtils.now() → Replace remaining Cu.now() callls with ChromeUtils.now()
Comment 3•1 year ago
|
||
Comment on attachment 9507697 [details]
Bug 1983606 - remove the Cu.now implementation, r=mccr8.
Revision D261475 was moved to bug 1985100. Setting attachment 9507697 [details] to obsolete.
Attachment #9507697 -
Attachment is obsolete: true
Pushed by fqueze@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/16b346576d40
https://hg.mozilla.org/integration/autoland/rev/d2a6fe1c195d
replace Cu.now() calls with ChromeUtils.now() calls, r=Gijs,perftest-reviewers,geckoview-reviewers,devtools-reviewers,sync-reviewers,places-reviewers,dom-storage-reviewers,translations-reviewers,backup-reviewers,home-newtab-reviewers,urlbar-reviewers,janv,nalexander,skhamis,nchevobbe,timhuang,mconley,sparky.
Comment 5•1 year ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
status-firefox144:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 144 Branch
Updated•1 year ago
|
Summary: Replace remaining Cu.now() callls with ChromeUtils.now() → Replace remaining Cu.now() calls with ChromeUtils.now()
Updated•1 year ago
|
QA Whiteboard: [qa-triage-done-c145/b144]
You need to log in
before you can comment on or make changes to this bug.
Description
•