1157702 - Re-disable readinglist on mozilla-beta

Status: RESOLVED FIXED

(Firefox Graveyard :: Reading List, defect)

Description

[:Gijs (he/him)]

So reading list and reader mode got disabled in bug 1155191.

Then bug 1157197 actually backed that out on mozilla-beta again so currently in the m-b repo, reading list has been re-enabled. It needs to be *off* for 38.0.5, so we should undo part of:

https://hg.mozilla.org/releases/mozilla-beta/rev/ceb1b9ba4a5a

Comment 1

[:Gijs (he/him)]

Attachment: Patch

Dolske, can you check this is righteous? Doing a build as we speak to doublecheck as well...

Approval Request Comment
[Feature/regressing bug #]: bug 1157197 / bug 1155191
[User impact if declined]: we don't want reading list on 38.0.5
[Describe test coverage new/current, TreeHerder]: this patch messes with some tests. We've landed it before so it should be OK.
[Risks and why]: turning off features makes people sad. I think that about sums it up.
[String/UUID change made/needed]: no.

Comment 2

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-beta/rev/02c94c36cce5