Closed
Bug 1094965
Opened 10 years ago
Closed 10 years ago
move `fetch` from actors/script.js to DevToolsUtils.js
Categories
(DevTools :: Debugger, defect)
DevTools
Debugger
Tracking
(firefox35 fixed, firefox36 fixed)
RESOLVED
FIXED
Firefox 36
People
(Reporter: jlong, Assigned: jlong)
Details
Attachments
(1 file, 2 obsolete files)
12.53 KB,
patch
|
Details | Diff | Splinter Review |
I need to use the same `fetch` in Fever Dream to fetch the HTML of a page, and this seems like a good utility function anyway so lets move it until our utils so everyone can access it.
Assignee | ||
Comment 1•10 years ago
|
||
Attachment #8518267 -
Flags: review?(nfitzgerald)
Comment 2•10 years ago
|
||
Comment on attachment 8518267 [details] [diff] [review] 1094965.patch Review of attachment 8518267 [details] [diff] [review]: ----------------------------------------------------------------- ::: toolkit/devtools/DevToolsUtils.js @@ +411,5 @@ > +/** > + * Performs a request to load the desired URL and returns a promise. > + * > + * @param aURL String > + * The URL we will request. While you're here, can you add an @param comment for aOptions?
Attachment #8518267 -
Flags: review?(nfitzgerald) → review+
Assignee | ||
Comment 4•10 years ago
|
||
Tests failed locally; had to import the `NetUtils` dependency as well (still lazily loads)
Attachment #8518280 -
Attachment is obsolete: true
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → jlong
Comment 5•10 years ago
|
||
Comment on attachment 8518305 [details] [diff] [review] 1094965.patch Review of attachment 8518305 [details] [diff] [review]: ----------------------------------------------------------------- ::: toolkit/devtools/DevToolsUtils.js @@ +407,5 @@ > return temp[aSymbol || aName]; > }); > }; > + > +exports.defineLazyGetter(this, "NetUtil", () => { Why do you need to export NetUtil from this module?
Assignee | ||
Comment 6•10 years ago
|
||
try is green https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=f839dd1db2b1
Assignee | ||
Comment 7•10 years ago
|
||
(In reply to Panos Astithas [:past] (overloaded, please needinfo) from comment #5) > > Why do you need to export NetUtil from this module? I'm not exporting it, but lazy loading it because it's required inside of `fetch`. That's how it was loaded in the other file. Not sure if it's also attaching it to the module object, and effectively exporting it? This depends on that, but I think we want to lazy load it, is there a better way?
Comment 8•10 years ago
|
||
Comment on attachment 8518305 [details] [diff] [review] 1094965.patch Review of attachment 8518305 [details] [diff] [review]: ----------------------------------------------------------------- ::: toolkit/devtools/DevToolsUtils.js @@ +407,5 @@ > return temp[aSymbol || aName]; > }); > }; > + > +exports.defineLazyGetter(this, "NetUtil", () => { defineLazyGetter is on exports, NetUtil is on this.
Comment 9•10 years ago
|
||
^ That was supposed to be a reply to Panos
Comment 10•10 years ago
|
||
D'oh, you are right. Please carry on.
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Comment 11•10 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/4c618071bb26
Keywords: checkin-needed
Whiteboard: [fixed-in-fx-team]
Assignee | ||
Comment 12•10 years ago
|
||
Eddy, just want to make sure this is on the list of patches to apply to gum
https://hg.mozilla.org/mozilla-central/rev/4c618071bb26
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 36
Updated•10 years ago
|
status-firefox35:
--- → fixed
status-firefox36:
--- → fixed
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•