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)
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 1•9 years ago
|
||
Assignee | ||
Comment 2•9 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)
Updated•9 years ago
|
Attachment #8576353 -
Flags: review?(bugmail) → review+
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Updated•9 years ago
|
Keywords: checkin-needed
Comment 3•9 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•9 years ago
|
Keywords: checkin-needed
Updated•9 years ago
|
Keywords: checkin-needed
Comment 4•9 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•9 years ago
|
Keywords: checkin-needed
Updated•9 years ago
|
Keywords: checkin-needed
Comment 5•9 years ago
|
||
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.
Description
•