Bug 1680440 - [devtools] Remove backward compatibility comments for descriptor watcher traits. r=jdescottes.

The watcher trait might not be set on other descriptors, so
we want to keep them until we can get rid of the generic client
code that is handling them. This can probably be done as part of Bug 1680280.

Differential Revision: https://phabricator.services.mozilla.com/D98737
This commit is contained in:
Nicolas Chevobbe 2020-12-11 14:20:16 +00:00
Родитель 6d5ae59617
Коммит 9cd6389ce8
3 изменённых файлов: 6 добавлений и 6 удалений

Просмотреть файл

@ -154,7 +154,7 @@ const ProcessDescriptorActor = ActorClassWithSpec(processDescriptorSpec, {
id: this.id,
isParent: this.isParent,
traits: {
// @backward-compat { version 77 } supports the Watcher actor
// Supports the Watcher actor. Can be removed as part of Bug 1680280.
watcher: true,
},
};

Просмотреть файл

@ -60,7 +60,7 @@ const TabDescriptorActor = ActorClassWithSpec(tabDescriptorSpec, {
selected: this.selected,
title: this._getTitle(),
traits: {
// @backward-compat { version 77 }
// Supports the Watcher actor. Can be removed as part of Bug 1680280.
watcher: true,
},
url: this._getUrl(),

Просмотреть файл

@ -296,8 +296,8 @@ class TargetList extends EventEmitter {
}
this._setListening(type, true);
// @backward-compat { version 77 } We now support frames watching via watchTargets
// for Tab and Process descriptors.
// Only a few top level targets support the watcher actor at the moment (see WatcherActor
// traits in the _form method). Bug 1675763 tracks watcher actor support for all targets.
if (this.hasTargetWatcherSupport(type)) {
// When we switch to a new top level target, we don't have to stop and restart
// Watcher listener as it is independant from the top level target.
@ -336,8 +336,8 @@ class TargetList extends EventEmitter {
}
this._setListening(type, false);
// @backward-compat { version 77 } We now support frames watching via watchTargets
// for Tab and Process descriptors.
// Only a few top level targets support the watcher actor at the moment (see WatcherActor
// traits in the _form method). Bug 1675763 tracks watcher actor support for all targets.
if (this.hasTargetWatcherSupport(type)) {
// When we switch to a new top level target, we don't have to stop and restart
// Watcher listener as it is independant from the top level target.