Closed Bug 1731553 Opened 3 years ago Closed 3 years ago

Add support for "stackTrace" to ConsoleLogEntry and JavascriptLogEntry

Categories

(Remote Protocol :: WebDriver BiDi, task, P2)

Product:
Remote Protocol
Component:
WebDriver BiDi
Type:
task
Points:
2

Tracking

(firefox97 fixed)

Status:
RESOLVED FIXED
Milestone:
97 Branch
Tracking Flags:
Tracking Status
firefox97 --- fixed

People

(Reporter: whimboo, Assigned: whimboo)

References

(Depends on 1 open bug, Blocks 1 open bug)

Details

(Whiteboard: [bidi-m2-mvp], [wptsync upstream])

Attachments

(3 files)

Bug 1731553 - [remote] Add shared module to handle stack frames.
(deleted), text/x-phabricator-request
Details
Bug 1731553 - [webdriver-bidi] Add support for "stackTrace" to "log.entryAdded".
(deleted), text/x-phabricator-request
Details
Bug 1731553 - [wdspec] Added tests to support stackTrace entries for log.entryAdded.
(deleted), text/x-phabricator-request
Details

Once support for ConcoleLogEntry and JavascriptLogEntry has been added we have to also support attaching the stacktrace for trace level ConsoleLogEntry entries, and JavascriptLogEntry.

Depends on: 1731548
Summary: Add support for sta for ConsoleLogEntry for log.entryAdded event → Add support for stackTrace to ConsoleLogEntry and JavascriptLogEntry
Summary: Add support for stackTrace to ConsoleLogEntry and JavascriptLogEntry → Add "stack" support to ConsoleLogEntry and JavascriptLogEntry
Summary: Add "stack" support to ConsoleLogEntry and JavascriptLogEntry → Add support for "stackTrace" to ConsoleLogEntry and JavascriptLogEntry

Note that via https://github.com/w3c/webdriver-bidi/pull/135 we might not want to add the stackTrace for every ConsoleLogEntry type.

Points: --- → 2
Priority: -- → P2
No longer depends on: 1731543
Assignee: nobody → hskupin
Status: NEW → ASSIGNED

(In reply to Henrik Skupin (:whimboo) [⌚️UTC+1] from comment #1)

Note that via https://github.com/w3c/webdriver-bidi/pull/135 we might not want to add the stackTrace for every ConsoleLogEntry type.

Based on the merged changes the stacktrace needs to be included for console API calls for the following methods: assert, error, warn and trace.

Blocks: 1679981
Depends on: 1744705
Attachment #9254118 - Attachment description: WIP: Bug 1731553 - [remote] Add shared module to handle stack frames. → Bug 1731553 - [remote] Add shared module to handle stack frames.
Attachment #9254119 - Attachment description: WIP: Bug 1731553 - [webdriver-bidi] Add support for "stackTrace" to "log.entryAdded". → Bug 1731553 - [webdriver-bidi] Add support for "stackTrace" to "log.entryAdded".
Pushed by hskupin@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/4ffae25ab5a9 [remote] Add shared module to handle stack frames. r=webdriver-reviewers,jdescottes https://hg.mozilla.org/integration/autoland/rev/fd8c95f5ea3a [webdriver-bidi] Add support for "stackTrace" to "log.entryAdded". r=webdriver-reviewers,jdescottes https://hg.mozilla.org/integration/autoland/rev/598f068edcbc [wdspec] Added tests to support stackTrace entries for log.entryAdded. r=webdriver-reviewers,jdescottes
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/31985 for changes under testing/web-platform/tests
Whiteboard: [bidi-m2-mvp] → [bidi-m2-mvp], [wptsync upstream]

https://hg.mozilla.org/mozilla-central/rev/4ffae25ab5a9
https://hg.mozilla.org/mozilla-central/rev/fd8c95f5ea3a
https://hg.mozilla.org/mozilla-central/rev/598f068edcbc

Status: ASSIGNED → RESOLVED
Closed: 3 years ago
status-firefox97: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 97 Branch
Upstream PR merged by moz-wptsync-bot
Blocks: 1745440
Blocks: 1747061
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: