Closed
Bug 1382576
Opened 7 years ago
Closed 7 years ago
Fix 47 tests failures on devtools/client/animationinspector due the EventEmitter refactoring
Categories
(DevTools :: General, enhancement, P2)
DevTools
General
Tracking
(firefox57 fixed)
RESOLVED
FIXED
Firefox 57
Tracking | Status | |
---|---|---|
firefox57 | --- | fixed |
People
(Reporter: zer0, Assigned: pbro)
References
Details
Attachments
(1 file)
Failing tests: devtools/client/animationinspector/test/browser_animation_animated_properties_displayed.js devtools/client/animationinspector/test/browser_animation_animated_properties_for_delayed_starttime_animations.js devtools/client/animationinspector/test/browser_animation_animated_properties_path.js devtools/client/animationinspector/test/browser_animation_animated_properties_progress_indicator.js devtools/client/animationinspector/test/browser_animation_click_selects_animation.js devtools/client/animationinspector/test/browser_animation_controller_exposes_document_currentTime.js devtools/client/animationinspector/test/browser_animation_empty_on_invalid_nodes.js devtools/client/animationinspector/test/browser_animation_keyframe_markers.js devtools/client/animationinspector/test/browser_animation_mutations_with_same_names.js devtools/client/animationinspector/test/browser_animation_participate_in_inspector_update.js devtools/client/animationinspector/test/browser_animation_playerFronts_are_refreshed.js devtools/client/animationinspector/test/browser_animation_playerWidgets_appear_on_panel_init.js devtools/client/animationinspector/test/browser_animation_playerWidgets_target_nodes.js devtools/client/animationinspector/test/browser_animation_refresh_on_added_animation.js devtools/client/animationinspector/test/browser_animation_refresh_when_active.js devtools/client/animationinspector/test/browser_animation_running_on_compositor.js devtools/client/animationinspector/test/browser_animation_same_nb_of_playerWidgets_and_playerFronts.js devtools/client/animationinspector/test/browser_animation_shows_player_on_valid_node.js devtools/client/animationinspector/test/browser_animation_spacebar_toggles_animations.js devtools/client/animationinspector/test/browser_animation_spacebar_toggles_node_animations.js devtools/client/animationinspector/test/browser_animation_summarygraph_for_multiple_easings.js devtools/client/animationinspector/test/browser_animation_target_highlight_select.js devtools/client/animationinspector/test/browser_animation_target_highlighter_lock.js devtools/client/animationinspector/test/browser_animation_timeline_currentTime.js devtools/client/animationinspector/test/browser_animation_timeline_header.js devtools/client/animationinspector/test/browser_animation_timeline_iterationStart.js devtools/client/animationinspector/test/browser_animation_timeline_pause_button_01.js devtools/client/animationinspector/test/browser_animation_timeline_pause_button_02.js devtools/client/animationinspector/test/browser_animation_timeline_pause_button_03.js devtools/client/animationinspector/test/browser_animation_timeline_rate_selector.js devtools/client/animationinspector/test/browser_animation_timeline_rewind_button.js devtools/client/animationinspector/test/browser_animation_timeline_scrubber_exists.js devtools/client/animationinspector/test/browser_animation_timeline_scrubber_movable.js devtools/client/animationinspector/test/browser_animation_timeline_scrubber_moves.js devtools/client/animationinspector/test/browser_animation_timeline_setCurrentTime.js devtools/client/animationinspector/test/browser_animation_timeline_short_duration.js devtools/client/animationinspector/test/browser_animation_timeline_shows_delay.js devtools/client/animationinspector/test/browser_animation_timeline_shows_endDelay.js devtools/client/animationinspector/test/browser_animation_timeline_shows_iterations.js devtools/client/animationinspector/test/browser_animation_timeline_shows_name_label.js devtools/client/animationinspector/test/browser_animation_timeline_shows_time_info.js devtools/client/animationinspector/test/browser_animation_timeline_takes_rate_into_account.js devtools/client/animationinspector/test/browser_animation_timeline_ui.js devtools/client/animationinspector/test/browser_animation_toggle_button_resets_on_navigate.js devtools/client/animationinspector/test/browser_animation_toggle_button_toggles_animations.js devtools/client/animationinspector/test/browser_animation_toolbar_exists.js devtools/client/animationinspector/test/browser_animation_ui_updates_when_animation_data_changes.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•7 years 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•7 years ago
|
Flags: qe-verify-
Priority: -- → P2
Reporter | ||
Updated•7 years ago
|
Whiteboard: [nosdk]
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → pbrosset
Status: NEW → ASSIGNED
Comment hidden (mozreview-request) |
Assignee | ||
Comment 3•7 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=61c3cd86b28061a6c0288303821a38e4ef4ad717&group_state=expanded
Comment 4•7 years ago
|
||
mozreview-review |
Comment on attachment 8905066 [details] Bug 1382576 - Switch over from the old-event-emitter to event-emitter in the animation inspector; https://reviewboard.mozilla.org/r/176878/#review181848 Looks good to me, thanks for the cleanup!
Attachment #8905066 -
Flags: review?(jdescottes) → review+
Pushed by pbrosset@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/9383659b5adc Switch over from the old-event-emitter to event-emitter in the animation inspector; r=jdescottes
Comment 6•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/9383659b5adc
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox57:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 57
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•