Closed Bug 1142347 Opened 9 years ago Closed 9 years ago

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

Categories

(Firefox OS Graveyard :: Gaia::E-Mail, defect)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jrburke, Assigned: jrburke)

Details

Attachments

(1 file)

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 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+
Keywords: checkin-needed
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.
Keywords: checkin-needed
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.
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
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: