Closed Bug 1451019 Opened 6 years ago Closed 5 years ago

Convert LongStringActor to protocol.js

Categories

(DevTools :: General, enhancement, P2)

enhancement

Tracking

(Not tracked)

RESOLVED DUPLICATE of bug 1450977

People

(Reporter: yulia, Unassigned)

References

Details

As part of refactoring ObjectActor to protocol.js, we need to update LongStringActor. it is instantiated via: 
-> via createValueGrip, so similar to ObjectActor and PauseScopedObjectActor
-> StyleSheetActor.getText, NodeActor.getNodeValue, Device.getWallpaper/screenshotToDataURL
-> From many methods of WebConsoleActor, StorageActors
Note that there already is a protocol.js version of LongStringActor:
  https://searchfox.org/mozilla-central/source/devtools/server/actors/string.js
So it is mostly about switching from LongStringActor from actors/object/long-string.js to this one.

bug 1450977 looks like a duplicated of this bug.
comment 0 helps identifying all the current usages of LongStringActor to be refactored.
Severity: normal → enhancement
Priority: -- → P2
Product: Firefox → DevTools

Marked as duplicate to Bug 1450977. I think there's even less places where the old actor is used now.

Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.