1454827 - Source actor spec should use `source` instead of `onSource`

Status: RESOLVED FIXED

(DevTools :: General, enhancement, P3)

Description

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

Source actor's spec includes:

    onSource: {
      request: { type: "source" },
      response: RetVal("json")
    },

which is unusual, since the request's type would be "source" if the method were renamed to that.  We should explore renaming the method and removing the type to match typical style.

Comment 1

[Micah [:mtigley] (she/her)]

Attachment: Bug 1454827 - Source actor spec should use `source` instead of `onSource`.

Review commit: https://reviewboard.mozilla.org/r/256090/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/256090/

Comment 2

[Yulia Startsev [:yulia]]

Comment on attachment 8991100 [details]
Bug 1454827 - Source actor spec should use `source` instead of `onSource`.

https://reviewboard.mozilla.org/r/256090/#review263048

Thanks a lot for thaking this on Micah! I realized too late that the name I suggested to you wasn't the best one, so I wrote a comment regarding that. Otherwise looks really good.

::: commit-message-6073e:1
(Diff revision 1)
> +Bug 1454827 - Source actor spec should use `source` instead of `onSource`. r=jryans,yulia

nit: we usually write what was done in the commit rather than the bug name: in this case "replace onSource with source in Source Actor and Source Actor Spec"

::: devtools/server/actors/source.js:190
(Diff revision 1)
>      return this._threadActor.sources;
>    },
>    get dbg() {
>      return this.threadActor.dbg;
>    },
> -  get source() {
> +  get sourceData() {

I just realized that a better name for this is `originalSource`. 

We have two types of source that we can get. `generatedSource` is a compiled source, like something you get out of webpack when you bundle. `originalSource` is a representation of the file that a programmer was working on when they were writing their code.

I would propose that we rename sourceData to originalSource, because then we are consistant with generatedSource. Sorry it took me a while to think of that!

Comment 3

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

Comment on attachment 8991100 [details]
Bug 1454827 - Source actor spec should use `source` instead of `onSource`.

https://reviewboard.mozilla.org/r/256090/#review263062

Thanks for working on this! :) I agree with Yulia's comments.

Comment 4

[Patrick Brosset <:pbro>]

This bug has not been updated in the last 6 months. Resetting the assignee field.
Please, feel free to pick it up again and add a comment outlining your plans for it if you do still intend to work on it.
This is just trying to clean our backlog of bugs and make bugs available for people.

Comment 5

[malikanshul29]

Attachment: Rename onSource to source in source actor

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/5f00d5f48cb2
Rename onSource to source in source actor r=yulia

Comment 7

[Alexandru Michis [:malexandru]]

https://hg.mozilla.org/mozilla-central/rev/5f00d5f48cb2