Navigation between <body> and another element breaks the animation grid

VERIFIED FIXED in Firefox 62

Status

defect
P3
normal
VERIFIED FIXED
a year ago
a year ago

People

(Reporter: gyula.palko, Assigned: daisuke)

Tracking

(Blocks 1 bug, {regression, regressionwindow-wanted})

Dependency tree / graph

Firefox Tracking Flags

(firefox-esr52 unaffected, firefox-esr60 unaffected, firefox60 unaffected, firefox61 disabled, firefox62 verified)

Details

Attachments

(4 attachments, 3 obsolete attachments)

Reporter

Description

a year ago
Posted image grid.PNG
[Environments:]
Windows 10, Windows 7, Ubuntu 16.04, OsX10.12
62.0a1 2018-05-25

[Steps:]
1. Open Firefox and open a new tab with https://rawgit.com/dadaa/3b73f847427025b51ba1ab7333013d0c/raw/77f3f0bb884875a179c3407f73bf8a8dd54751c9/doc_custom_playback_rate.html
2. Press F12 and select Animations tab.
3. Select <body> and press Pause on the animation player.
4. Select a div
5. Select <body> again
6. Repeat steps 4-5 switching between the same div and <body>
7. Inspect the animation grid

[Actual Result:]
The second div's animation bar is moved to the right(see attached screenshot)

[Expected Result:]
The bar's starting point should remain at 0.0s 

[Note:]
On Fx 62.0a1 20180513220424, this issue is not reproducible.
On Fx 61 (old animation inspector), the problem is not reproducible.
Assignee

Updated

a year ago
Assignee: nobody → dakatsuka
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Status: NEW → ASSIGNED
Priority: -- → P3
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Assignee

Updated

a year ago
Attachment #8980635 - Attachment is obsolete: true
Attachment #8980635 - Flags: review?(pbrosset)
Assignee

Updated

a year ago
Attachment #8980636 - Attachment is obsolete: true
Attachment #8980636 - Flags: review?(pbrosset)
Assignee

Updated

a year ago
Attachment #8980637 - Attachment is obsolete: true
Attachment #8980637 - Flags: review?(pbrosset)

Comment 9

a year ago
mozreview-review
Comment on attachment 8981036 [details]
Bug 1464396 - Part 1: Hold all animation's created time.

https://reviewboard.mozilla.org/r/247148/#review253248

::: devtools/server/actors/animation.js:968
(Diff revision 1)
> +  },
> +
> +  /**
> +   * Update all animation created time map.
> +   */
> +  updateCreatedTimeMapAll() {

I suggest renaming this to `updateAllAnimationsCreatedTime`

::: devtools/server/actors/animation.js:971
(Diff revision 1)
> +   * Update all animation created time map.
> +   */
> +  updateCreatedTimeMapAll() {
> +    const currentAnimations = this.getAllAnimations(this.tabActor.window.document);
> +
> +    // Remove invalid animation.

Remove invalid animation*s*

::: devtools/server/actors/animation.js:988
(Diff revision 1)
> +  /**
> +   * Update animation created time map.
> +   *
> +   * @param {Object} animation
> +   */
> +  updateCreatedTimeMap(animation) {

I suggest renaming this to `updateAnimationCreatedTime`
Attachment #8981036 - Flags: review?(pbrosset) → review+

Comment 10

a year ago
mozreview-review
Comment on attachment 8981037 [details]
Bug 1464396 - Part 2: Update current time at first.

https://reviewboard.mozilla.org/r/247150/#review253252
Attachment #8981037 - Flags: review?(pbrosset) → review+

Comment 11

a year ago
mozreview-review
Comment on attachment 8981038 [details]
Bug 1464396 - Part 3: Add whether the created time of animation unchanged even if change node.

https://reviewboard.mozilla.org/r/247152/#review253260
Attachment #8981038 - Flags: review?(pbrosset) → review+
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Assignee

Comment 15

a year ago
Thanks Patrick!

I have updated the patch.
Will land if the try was green.
https://treeherder.mozilla.org/#/jobs?repo=try&revision=36e98ffa330cd73fbaa07b760668b920b10ea535

Comment 16

a year ago
Pushed by dakatsuka@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/732a958aa5cd
Part 1: Hold all animation's created time. r=pbro
https://hg.mozilla.org/integration/autoland/rev/9353259c9adf
Part 2: Update current time at first. r=pbro
https://hg.mozilla.org/integration/autoland/rev/97aee105338e
Part 3: Add whether the created time of animation unchanged even if change node. r=pbro
Reporter

Comment 18

a year ago
Verified as fixed on Nightly 62.0a1(20180610220159) on Win10 x64, Ubuntu 16.04, MacOS 10.12
Status: RESOLVED → VERIFIED
Reporter

Updated

a year ago
Depends on: 1468475

Updated

a year ago
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.