Open
Bug 1680212
Opened 3 years ago
Rename Watcher classes's watch method to initialize
Categories
(DevTools :: Framework, enhancement)
DevTools
Framework
Tracking
(Not tracked)
NEW
People
(Reporter: ochameau, Unassigned)
References
(Blocks 1 open bug)
Details
See this review comment for more context:
https://phabricator.services.mozilla.com/D87504?id=372538#inline-554598
Today, Watcher classes have to implement two methods: watch
and destroy
.
watch
could, in theory be merged into the constructor, it originaly was.
But because this is async we introduced an async method.
Given that this is still an initialization method that is meant to be called only once, this would probably be better named initialize
.
You need to log in
before you can comment on or make changes to this bug.
Description
•