Closed
Bug 1509804
Opened 7 years ago
Closed 6 years ago
Automatic export of HAR is broken due to EventEmitter refactor from last year
Categories
(DevTools :: Netmonitor, defect, P5)
Tracking
(firefox73 fixed)
RESOLVED
FIXED
Firefox 73
Tracking | Status | |
---|---|---|
firefox73 | --- | fixed |
People
(Reporter: jordan, Assigned: jdescottes)
References
(Blocks 1 open bug)
Details
Attachments
(3 files)
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_14) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/12.0 Safari/605.1.15
Steps to reproduce:
1. Set devtools.netmonitor.har.enableAutoExportToFile = true
2. Opened dev tools
3. Browsed around
Actual results:
No har files showed up in profile directory
Expected results:
Har files should have showed up.
This appears to be due to a bug in har-automation.js. The onTabNavigated event listener is trying to check type for 'navigated' and 'will-navigate', but that is no longer passed in so HAR files are never created.
I've attached a quick patch. I haven't tested it so it may be buggy.
Reporter | ||
Updated•7 years ago
|
Component: Untriaged → Netmonitor
OS: Unspecified → Mac OS X
Product: Firefox → DevTools
Hardware: Unspecified → Desktop
Updated•6 years ago
|
Status: UNCONFIRMED → NEW
Ever confirmed: true
Priority: -- → P5
Updated•6 years ago
|
Blocks: netmonitor-har
Assignee | ||
Comment 1•6 years ago
|
||
This fixes several gaps:
- event api changes
- networkEvent no longer emitted by debugger client
- shape of expected HAR items changed
- use the connector from the netmon api
This is the minimum set of changes I could do to get HAR automation to save valid HAR files again.
Updated•6 years ago
|
Assignee: nobody → jdescottes
Assignee | ||
Comment 2•6 years ago
|
||
Depends on D55204
This is just a smoke test to verify that the HAR file is created.
We should probably expand this and test the content as well, but we can handle that in follow up bugs
Updated•6 years ago
|
Attachment #9112531 -
Attachment description: Bug 1509804 - Add smoke test for har automation → Bug 1509804 - Add smoke test for HAR automation
Pushed by jdescottes@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/a4459904376f
Temporary fix for HAR automation r=Honza
https://hg.mozilla.org/integration/autoland/rev/76944c9ad43c
Add smoke test for HAR automation r=Honza
Comment 4•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/a4459904376f
https://hg.mozilla.org/mozilla-central/rev/76944c9ad43c
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox73:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 73
You need to log in
before you can comment on or make changes to this bug.
Description
•