Fix 8 tests failures on devtools/client/canvasdebugger due the EventEmitter refactoring

NEW
Unassigned

Status

()

Firefox
Developer Tools: Canvas Debugger
P2
normal
5 months ago
25 days ago

People

(Reporter: zer0, Unassigned)

Tracking

(Blocks: 1 bug)

unspecified
Points:
---
Bug Flags:
qe-verify -

Firefox Tracking Flags

(firefox57 fix-optional)

Details

(Reporter)

Description

5 months ago
Failing tests:

devtools/client/canvasdebugger/test/browser_canvas-frontend-call-stack-01.js
devtools/client/canvasdebugger/test/browser_canvas-frontend-call-stack-02.js
devtools/client/canvasdebugger/test/browser_canvas-frontend-img-thumbnails-02.js
devtools/client/canvasdebugger/test/browser_canvas-frontend-record-01.js
devtools/client/canvasdebugger/test/browser_canvas-frontend-record-02.js
devtools/client/canvasdebugger/test/browser_canvas-frontend-reload-01.js
devtools/client/canvasdebugger/test/browser_canvas-frontend-reload-02.js
devtools/client/canvasdebugger/test/browser_canvas-frontend-snapshot-select-01.js


The refactoring is currently only on:

https://github.com/zer0/gecko/tree/event-emitter-1381542

We need to address the test failures before land this patch in m-c.
(Reporter)

Comment 1

5 months ago
Here the original try build with the failures:

https://treeherder.mozilla.org/#/jobs?repo=try&revision=bba13e27a2371fa8aad68b9b227534b31829cb0d

Those failures are most likely due the breaking change in how the `EventEmitter` emits event.

Previously, the first argument was the event type:

  myEmitter.on("custom-event", (eventType, message) => { ... });

Now the first argument is the message:

  myEmitter.on("custom-event", (message) => { ... });

In the majority of the scenario the `eventType` is ignored by our code, so we should just remove it from the function's signature.

For more details and edge cases, see: https://github.com/devtools-html/snippets-for-removing-the-sdk/#events

Updated

5 months ago
Flags: qe-verify-
Priority: -- → P2
(Reporter)

Updated

5 months ago
No longer blocks: 1381542
(Reporter)

Updated

5 months ago
Blocks: 1384546
(Reporter)

Updated

5 months ago
Whiteboard: [nosdk]

Updated

3 months ago
status-firefox57: --- → fix-optional
Blocks: 1402832
No longer blocks: 1402832
Component: Developer Tools → Developer Tools: Canvas Debugger
You need to log in before you can comment on or make changes to this bug.