Fail to run devtools network monitor component at browser

RESOLVED FIXED in Firefox 59

Status

defect
P2
normal
RESOLVED FIXED
a year ago
11 months ago

People

(Reporter: pd4d10, Assigned: pd4d10)

Tracking

unspecified
Firefox 59

Firefox Tracking Flags

(firefox59 fixed)

Details

Attachments

(2 attachments)

(Assignee)

Description

a year ago
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_13_2) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/63.0.3239.108 Safari/537.36

Steps to reproduce:

1. Clone `mozilla-central` repo
2. Go to `devtools/client/netmonitor` folder
3. run `yarn install; yarn start`
4. Open `http://localhost:8000`


Actual results:

Browser throw an error `loader is not defined`


Expected results:

It should throw no error and works correctly
Comment hidden (mozreview-request)
Component: Untriaged → Developer Tools: Netmonitor
Thanks for the patch! We just come back from the year-end vacation, will review it soon.

Ricky, could you take a look on this patch?
Assignee: nobody → pd4d10
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Flags: needinfo?(rchien)

Comment 3

a year ago
mozreview-review
Comment on attachment 8938860 [details]
Bug 1427077 - Make devtools/client/netmonitor work correctly at browser

https://reviewboard.mozilla.org/r/209360/#review215318

Thanks for the patch @pd4d10! Using rewrite-lazy-getter.js is great for solving this issue. Patch has verfifed and launchpad works right now. \o/

I'm going to approve this patch since right now rewrite-lazy-getter makes sense in our case. Please fix the issue before landing this patch.

::: devtools/client/netmonitor/src/components/PropertiesView.js:18
(Diff revision 1)
>  const { FILTER_SEARCH_DELAY } = require("../constants");
>  
>  // Components
>  const TreeViewClass = require("devtools/client/shared/components/tree/TreeView");
>  const TreeView = createFactory(TreeViewClass);
>  const SearchBox = createFactory(require("devtools/client/shared/components/SearchBox"));

Searchbox rendering will be determined by this.shouldRenderSearchBox(), so it's necessary to be imported via loader.lazyGetter.
Attachment #8938860 - Flags: review+
Flags: needinfo?(rchien)
Comment hidden (mozreview-request)

Comment 5

a year ago
Pushed by rchien@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/591cf97d4777
Make devtools/client/netmonitor work correctly at browser r=rickychien

Comment 7

a year ago
Pushed by apavel@mozilla.com:
https://hg.mozilla.org/mozilla-central/rev/379d966c93d2
Make devtools/client/netmonitor work correctly at browser r=rickychien
https://hg.mozilla.org/mozilla-central/rev/e6aed81c3f04
Backed out changeset 591cf97d4777 for ES failures in /builds/worker/checkouts/gecko/devtools/client/shared/webpack/rewrite-lazy-getter.js r=backout on a CLOSED TREE

Comment 8

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/379d966c93d2
Status: ASSIGNED → RESOLVED
Last Resolved: a year ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 59
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Reopen this bug since patch has been backout for ES linter failures.

pd4d10, could you take a look? thanks

Comment 10

a year ago
mozreview-review
Comment on attachment 8938860 [details]
Bug 1427077 - Make devtools/client/netmonitor work correctly at browser

https://reviewboard.mozilla.org/r/209360/#review216058

::: devtools/client/shared/webpack/rewrite-lazy-getter.js:5
(Diff revision 2)
> +/* This Source Code Form is subject to the terms of the Mozilla Public
> + * License, v. 2.0. If a copy of the MPL was not distributed with this
> + * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
> +
> + "use strict";

nit: remove whitespace before "use strict";
Attachment #8938860 - Flags: review+
(Assignee)

Comment 11

a year ago
(In reply to Ricky Chien [:rickychien] from comment #10)
> Comment on attachment 8938860 [details]
> Bug 1427077 - Make devtools/client/netmonitor work correctly at browser
> 
> https://reviewboard.mozilla.org/r/209360/#review216058
> 
> ::: devtools/client/shared/webpack/rewrite-lazy-getter.js:5
> (Diff revision 2)
> > +/* This Source Code Form is subject to the terms of the Mozilla Public
> > + * License, v. 2.0. If a copy of the MPL was not distributed with this
> > + * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
> > +
> > + "use strict";
> 
> nit: remove whitespace before "use strict";

Hi Ricky, I remove whitespace and run `hg commit --amend`, then `hg push review`, receiving messages below:

remote: adding changesets
remote: adding manifests
remote: adding file changes
remote: added 1 changesets with 1 changes to 4 files (+1 heads)
remote: recorded push in pushlog
submitting 1 changesets for review
abort: Review request is submitted or discarded.
You must reopen the review request before it can be updated.
Review requests should only be reopened if your changes have not landed or have
been backed out - file new bugs for follow-up work.

Seems that the new commit has been pushed to remote successfully, but the review request failed. Did I miss something?
Hi pd4d10,

The only thing you need to do for re-submitting to a closed review is to reopen the reviewboard, and then you can do `hg push review` again to overwrite the original commit as usual.
For how to reopen closed review, see attachment 8940648 [details].
Comment hidden (mozreview-request)
Flags: needinfo?(pd4d10)

Comment 15

a year ago
Pushed by rchien@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7c76e6168da7
Make devtools/client/netmonitor work correctly at browser r=rickychien
https://hg.mozilla.org/mozilla-central/rev/7c76e6168da7
Status: REOPENED → RESOLVED
Last Resolved: a year agoa year ago
Resolution: --- → FIXED

Updated

11 months ago
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.