Issue 5360 - Whitelist subscription shown as '-user-35236'

Issue 5360 - User added whitelist shown as '-user-35236' and not the URL name

[anton, 2017-08-07 09:40:16]

https://codereview.adblockplus.org/29505606/diff/29505607/mobile/android/thir...
File
mobile/android/thirdparty/org/adblockplus/browser/MoreSubscriptionsPreferenceGroup.java
(right):

https://codereview.adblockplus.org/29505606/diff/29505607/mobile/android/thir...
mobile/android/thirdparty/org/adblockplus/browser/MoreSubscriptionsPreferenceGroup.java:284:
ActiveSubscriptionContainer(Context context) {
`{` should be in the next line

https://codereview.adblockplus.org/29505606/diff/29505607/mobile/android/thir...
mobile/android/thirdparty/org/adblockplus/browser/MoreSubscriptionsPreferenceGroup.java:308:
if(title.startsWith("~user~")) {
space expected between `if` and `(`

https://codereview.adblockplus.org/29505606/diff/29505607/mobile/android/thir...
mobile/android/thirdparty/org/adblockplus/browser/MoreSubscriptionsPreferenceGroup.java:308:
if(title.startsWith("~user~")) {
`{` is expected to be in the next line

[diegocarloslima, 2017-08-07 11:07:34]

https://codereview.adblockplus.org/29505606/diff/29505607/mobile/android/thir...
File
mobile/android/thirdparty/org/adblockplus/browser/MoreSubscriptionsPreferenceGroup.java
(right):

https://codereview.adblockplus.org/29505606/diff/29505607/mobile/android/thir...
mobile/android/thirdparty/org/adblockplus/browser/MoreSubscriptionsPreferenceGroup.java:284:
ActiveSubscriptionContainer(Context context) {
On 2017/08/07 09:40:16, anton wrote:
> `{` should be in the next line

Acknowledged.

https://codereview.adblockplus.org/29505606/diff/29505607/mobile/android/thir...
mobile/android/thirdparty/org/adblockplus/browser/MoreSubscriptionsPreferenceGroup.java:308:
if(title.startsWith("~user~")) {
On 2017/08/07 09:40:16, anton wrote:
> `{` is expected to be in the next line

Acknowledged.

[anton, 2017-08-08 05:34:16]

On 2017/08/07 11:07:34, diegocarloslima wrote:
>
https://codereview.adblockplus.org/29505606/diff/29505607/mobile/android/thir...
> File
>
mobile/android/thirdparty/org/adblockplus/browser/MoreSubscriptionsPreferenceGroup.java
> (right):
> 
>
https://codereview.adblockplus.org/29505606/diff/29505607/mobile/android/thir...
>
mobile/android/thirdparty/org/adblockplus/browser/MoreSubscriptionsPreferenceGroup.java:284:
> ActiveSubscriptionContainer(Context context) {
> On 2017/08/07 09:40:16, anton wrote:
> > `{` should be in the next line
> 
> Acknowledged.
> 
>
https://codereview.adblockplus.org/29505606/diff/29505607/mobile/android/thir...
>
mobile/android/thirdparty/org/adblockplus/browser/MoreSubscriptionsPreferenceGroup.java:308:
> if(title.startsWith("~user~")) {
> On 2017/08/07 09:40:16, anton wrote:
> > `{` is expected to be in the next line
> 
> Acknowledged.

LGTM

[jens, 2017-08-08 07:48:58]

On 2017/08/08 05:34:16, anton wrote:
> On 2017/08/07 11:07:34, diegocarloslima wrote:
> >
>
https://codereview.adblockplus.org/29505606/diff/29505607/mobile/android/thir...
> > File
> >
>
mobile/android/thirdparty/org/adblockplus/browser/MoreSubscriptionsPreferenceGroup.java
> > (right):
> > 
> >
>
https://codereview.adblockplus.org/29505606/diff/29505607/mobile/android/thir...
> >
>
mobile/android/thirdparty/org/adblockplus/browser/MoreSubscriptionsPreferenceGroup.java:284:
> > ActiveSubscriptionContainer(Context context) {
> > On 2017/08/07 09:40:16, anton wrote:
> > > `{` should be in the next line
> > 
> > Acknowledged.
> > 
> >
>
https://codereview.adblockplus.org/29505606/diff/29505607/mobile/android/thir...
> >
>
mobile/android/thirdparty/org/adblockplus/browser/MoreSubscriptionsPreferenceGroup.java:308:
> > if(title.startsWith("~user~")) {
> > On 2017/08/07 09:40:16, anton wrote:
> > > `{` is expected to be in the next line
> > 
> > Acknowledged.
> 
> LGTM

LGTM