[email] Remove vscroll mitigation for bug 1135960 now that bug 1141362 has landed

RESOLVED FIXED

Status

Firefox OS
Gaia::E-Mail
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: jrburke, Assigned: jrburke)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
The fix in bug 1141362 is a more general fix for the issue in bug 1135960, so remove the _initing mitigation that was done in vscroll.js to avoid the issue, since it was a specific fix, and no longer needed with the general mitigation in bug 1141362.

This is only an issue on master, not any branch yet. The specific _initing mitigation was introduced as part of bug 1128739 which is only on master.

Comment 1

3 years ago
Created attachment 8576353 [details] [review]
[gaia] jrburke:bug1142347-email-vscroll-initing > mozilla-b2g:master
(Assignee)

Comment 2

3 years ago
Comment on attachment 8576353 [details] [review]
[gaia] jrburke:bug1142347-email-vscroll-initing > mozilla-b2g:master

Tested on flame device and confirmed that bug 1135960 did not show up with bug 1141362 applied on and with this change in place.
Attachment #8576353 - Flags: review?(bugmail)
Attachment #8576353 - Flags: review?(bugmail) → review+
(Assignee)

Updated

3 years ago
Keywords: checkin-needed

Updated

3 years ago
Keywords: checkin-needed

Comment 3

3 years ago
http://docs.taskcluster.net/tools/task-graph-inspector/#M2_SAMg3Tme3JaqMN3n_pQ

The pull request failed to pass integration tests. It could not be landed, please try again.
(Assignee)

Updated

3 years ago
Keywords: checkin-needed

Updated

3 years ago
Keywords: checkin-needed

Comment 4

3 years ago
http://docs.taskcluster.net/tools/task-graph-inspector/#Jdz1N3IqQYSmrWAI8Hi7Rg

The pull request failed to pass integration tests. It could not be landed, please try again.
(Assignee)

Updated

3 years ago
Keywords: checkin-needed

Updated

3 years ago
Keywords: checkin-needed
Hey - I noticed that Autolander was having some trouble communicating to github today. I need to surface these messages and notify users. For now let's land this manually. Sorry about the confusion here =(

In maseter: https://github.com/mozilla-b2g/gaia/commit/da743ed18d7cd699649bc00d37ba0bfbe846b167
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.