Beginning on October 25th, 2016, Persona will no longer be an option for authentication on BMO. For more details see Persona Deprecated.
Last Comment Bug 806644 - Back out bug 778993
: Back out bug 778993
Product: Core
Classification: Components
Component: JavaScript Engine (show other bugs)
: unspecified
: All All
: -- normal with 1 vote (vote)
: mozilla19
Assigned To: Bill McCloskey (:billm)
: general
: Jason Orendorff [:jorendorff]
Depends on:
  Show dependency treegraph
Reported: 2012-10-29 17:34 PDT by Bill McCloskey (:billm)
Modified: 2012-10-30 08:03 PDT (History)
4 users (show)
ryanvm: in‑testsuite-
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

backout patch (1.83 KB, patch)
2012-10-29 17:34 PDT, Bill McCloskey (:billm)
terrence.d.cole: review+
lukasblakk+bugs: approval‑mozilla‑aurora+
lukasblakk+bugs: approval‑mozilla‑beta+
Details | Diff | Splinter Review

Description Bill McCloskey (:billm) 2012-10-29 17:34:25 PDT
Created attachment 676409 [details] [diff] [review]
backout patch

This is causing a regression in bug 806087. It also may be related to bug 782072.
Comment 2 Bill McCloskey (:billm) 2012-10-29 17:42:42 PDT
Comment on attachment 676409 [details] [diff] [review]
backout patch

[Approval Request Comment]
Bug caused by (feature/regressing bug #): bug 778993
User impact if declined: OOM during syncs of big profiles.
Testing completed (on m-c, etc.): On m-c.
Risk to taking this patch (and alternatives if risky): Low. It's purely a heuristic change and it reverts to FF16 behavior.
String or UUID changes made by this patch: None.
Comment 3 Lukas Blakk [:lsblakk] use ?needinfo 2012-10-29 17:59:10 PDT
Comment on attachment 676409 [details] [diff] [review]
backout patch

yes to this backout, pre-approving for aurora/beta even though this is only on inbound right now so that there's time to get this landed as early as it can be (ie: before tomorrow at noon PT) and we get this backout in our 17 Beta 4 builds.
Comment 5 Ryan VanderMeulen [:RyanVM] 2012-10-30 08:02:51 PDT

Note You need to log in before you can comment on or make changes to this bug.