Closed
Bug 1273609
Opened 9 years ago
Closed 9 years ago
Support manual_subframe transitionType in webNavigation events details
Categories
(WebExtensions :: Untriaged, defect)
WebExtensions
Untriaged
Tracking
(firefox49 fixed)
| Tracking | Status | |
|---|---|---|
| firefox49 | --- | fixed |
People
(Reporter: rpl, Assigned: rpl)
References
Details
(Whiteboard: [webNavigation] triaged)
Attachments
(1 file)
The webNavigation events should have their transitionType set to "manual_subframe" when their related sub-frame navigation has been generated by a user click.
| Assignee | ||
Comment 1•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/53306/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/53306/
Attachment #8753471 -
Flags: review?(gkrizsanits)
| Assignee | ||
Updated•9 years ago
|
Assignee: nobody → lgreco
Status: NEW → ASSIGNED
| Assignee | ||
Updated•9 years ago
|
Whiteboard: [webNavigation]
| Assignee | ||
Comment 2•9 years ago
|
||
Comment on attachment 8753471 [details]
MozReview Request: Bug 1273609 - [webext] Support manual_subframe transitionType in webNavigation events details. r?krizsa
Review request updated; see interdiff: https://reviewboard.mozilla.org/r/53306/diff/1-2/
| Assignee | ||
Comment 3•9 years ago
|
||
(In reply to Luca Greco [:rpl] from comment #2)
> Comment on attachment 8753471 [details]
> MozReview Request: Bug 1273609 - [webext] Support manual_subframe
> transitionType in webNavigation events details. r?krizsa
>
> Review request updated; see interdiff:
> https://reviewboard.mozilla.org/r/53306/diff/1-2/
added missing test assets.
Comment 4•9 years ago
|
||
Comment on attachment 8753471 [details]
MozReview Request: Bug 1273609 - [webext] Support manual_subframe transitionType in webNavigation events details. r?krizsa
https://reviewboard.mozilla.org/r/53306/#review50262
Attachment #8753471 -
Flags: review?(gkrizsanits) → review+
| Assignee | ||
Comment 5•9 years ago
|
||
Push to try of the attached patch:
- https://treeherder.mozilla.org/#/jobs?repo=try&revision=ffbe6d429a2b3e07cee38c6e3419d05d443da7a1
| Assignee | ||
Updated•9 years ago
|
Iteration: --- → 49.2 - May 23
Keywords: checkin-needed
Updated•9 years ago
|
Whiteboard: [webNavigation] → [webNavigation] triaged
Keywords: checkin-needed
Comment 7•9 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox49:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
Updated•7 years ago
|
Product: Toolkit → WebExtensions
You need to log in
before you can comment on or make changes to this bug.
Description
•