Closed Bug 1378864 Opened 7 years ago Closed 7 years ago

Stop using sdk/core/heritage in DevTools shared specs/string

Categories

(DevTools :: General, enhancement, P1)

enhancement

Tracking

(firefox57 fixed)

RESOLVED FIXED
Firefox 57
Tracking Status
firefox57 --- fixed

People

(Reporter: sole, Assigned: sole)

References

Details

(Whiteboard: [nosdk])

Attachments

(1 file)

Used in: devtools/shared/specs/string.js

More details to follow as we triage.
Flags: qe-verify-
Priority: -- → P2
Whiteboard: [nosdk]
Taking this one
Assignee: nobody → sole
Status: NEW → ASSIGNED
Priority: P2 → P1
Target Milestone: --- → Firefox 56
Comment on attachment 8891333 [details]
Bug 1378864 - Stop using sdk/core/heritage in DevTools shared specs/string.

https://reviewboard.mozilla.org/r/162530/#review168318

Looks great! Just one nit. Thanks for working on this, sole!

::: devtools/shared/specs/string.js:32
(Diff revision 1)
>  /**
>   * When a caller is expecting a LongString actor but the string is already available on
>   * client, the SimpleStringFront can be used as it shares the same API as a
>   * LongStringFront but will not make unnecessary trips to the server.
>   */
> -const SimpleStringFront = Class({
> +const SimpleStringFront = class SimpleStringFront {

I think you can just use:
```class SimpleStringFront {
…
}
```

Without the `const`. It's generally more consistent with the usage of `class` we've until now in our codebase. Unless there any specific reason you want to keep like this.
Attachment #8891333 - Flags: review?(zer0) → review+
Comment on attachment 8891333 [details]
Bug 1378864 - Stop using sdk/core/heritage in DevTools shared specs/string.

https://reviewboard.mozilla.org/r/162530/#review168318

Fixed the nit!
Comment on attachment 8891333 [details]
Bug 1378864 - Stop using sdk/core/heritage in DevTools shared specs/string.

https://reviewboard.mozilla.org/r/162530/#review168788
Attachment #8891333 - Flags: review+
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/fb2dff3fff69
Stop using sdk/core/heritage in DevTools shared specs/string. r=zer0
Keywords: checkin-needed
Target Milestone: Firefox 56 → Firefox 57
https://hg.mozilla.org/mozilla-central/rev/fb2dff3fff69
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: