Opened on 03/09/2016 at 12:44:29 PM

Closed on 07/05/2017 at 08:03:49 AM

Last modified on 07/05/2017 at 08:05:54 AM

#3766 closed defect (fixed)

Restore default ABP bookmarks

Reported by: rjeschke Assignee: diegocarloslima
Priority: P2 Milestone: Adblock-Browser-for-Android-1.2.0
Module: Adblock-Browser-for-Android Keywords:
Cc: Blocked By:
Blocking: Platform: Adblock Browser for Android
Ready: yes Confidential: no
Tester: Scheer Verified working: yes
Review URL(s):

https://codereview.adblockplus.org/29346676/

Description (last modified by diegocarloslima)

Mozilla changed the way mobile bookmarks work, i.e. removed those and is now using the desktop ones. This change currently breaks the tests for building Fennec. A temporary solution to get the build working again is to disable the tests by adding ac_add_options --disable-tests to your mozconfig.

Previously bookmarks were defined via a locale specific bookmarks.inc file in the mobile folder. See here. The corresponding string definitions were to be found here (Those strings are currently commented out in branch abb_fennec_44_0_2_merge).

With the latest merge, Mozilla has removed the mobile specific bookmarks.inc file.

We need a way to get the bookmarks and their strings working again.

Furthermore we need to remove the predefined Mozilla bookmarks.

Hints for Testers

The project should build without the need of adding ac_add_options --disable-tests to the mozconfig file

After installing the app, check if these bookmarks are added by default:
About your browser (about:)
Acceptable Ads (https://acceptableads.org)
Adblock Plus (https://adblockplus.org)
Beta community (https://adblockplus.org/redirect?link=adblock_browser_android_beta_community)

After installing the app, check if these bookmarks are not added:
Firefox: About your browser (about:firefox)
Firefox: Customize with add-ons (https://addons.mozilla.org/android/)
Firefox: Support (https://support.mozilla.org/products/mobile)
Learn the Web: Mozilla Webmaker (https://webmaker.org/)
Firefox Help and Support for restricted profiles on Android tablets (https://support.mozilla.org/kb/controlledaccess)

Attachments (0)

Change History (10)

comment:1 Changed on 05/09/2016 at 09:55:26 AM by rjeschke

  • Description modified (diff)

comment:2 Changed on 05/10/2016 at 09:08:59 AM by diegocarloslima

  • Owner set to diegocarloslima

comment:3 Changed on 05/10/2016 at 01:57:21 PM by rjeschke

  • Description modified (diff)

comment:4 Changed on 05/10/2016 at 03:52:01 PM by diegocarloslima

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

comment:5 Changed on 06/20/2016 at 09:24:32 PM by diegocarloslima

  • Review URL(s) modified (diff)

comment:6 Changed on 10/19/2016 at 02:32:36 PM by abpbot

A commit referencing this issue has landed:
Issue 3766 - Restore default ABP bookmarks

comment:7 Changed on 10/19/2016 at 02:33:44 PM by diegocarloslima

  • Milestone set to Adblock-Browser-for-Android-next
  • Resolution set to fixed
  • Status changed from reviewing to closed

comment:8 Changed on 06/19/2017 at 11:13:29 AM by scheer

  • Resolution fixed deleted
  • Status changed from closed to reopened
  • Tester changed from Unknown to Scheer

Is there any more updates on the change here. Did we use this method in the end, to get the build to function or did we revert since then? could we have some more information to the issue and maybe add some hints for testers?

Thanks

comment:9 Changed on 06/30/2017 at 03:10:19 PM by diegocarloslima

  • Description modified (diff)

comment:10 Changed on 07/05/2017 at 08:03:49 AM by scheer

  • Resolution set to fixed
  • Status changed from reopened to closed
  • Verified working set

adblockbrowser-1.1.1.20170612182010-arm
Galaxy S7
Android 7.0

Last edited on 07/05/2017 at 08:05:54 AM by scheer

Add Comment

Modify Ticket

Change Properties
Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from diegocarloslima.
 
Note: See TracTickets for help on using tickets.