Issue 6403 - Updated adblockplusui dependency

lib/prefs.js

Index: lib/prefs.js
===================================================================
--- a/lib/prefs.js
+++ b/lib/prefs.js
@@ -155,7 +155,7 @@
 /**
  * Whether to show tracking warning in options page when both
  * Acceptable Ads and subscription of type "Privacy" are enabled.
- * 
+ *
  * @type {boolean}
  */
 defaults.ui_warn_tracking = true;
@@ -183,7 +183,7 @@
 defaults.show_devtools_panel = true;
 
 /**
- * Whether to suppress the first run page. This preference isn't
+ * Whether to suppress the first run and updates page. This preference isn't
  * set by the extension but can be pre-configured externally.
  *
  * @see https://adblockplus.org/development-builds/suppressing-the-first-run-page-on-chrome
@@ -201,6 +201,14 @@
 defaults.additional_subscriptions = [];
 
 /**
+ * The version of major updates that the user is aware of. If it's too low,
+ * the updates page will be shown to inform the user about intermediate changes.
+ *
+ * @type {number}
+ */
+defaults.updates_version = 0;

[kzar, 2018/01/23 10:07:13]

I think this preference name is kind of confusing, especially since we call the
version of the current update's page updatesVersion. Without the (very useful)
comment above I would be quite confused about what the preference did. How about
we call it last_update_page_displayed and the other variable updatePageVersion?

[Thomas Greiner, 2018/01/25 11:09:45]

I see your point. Just a bit hesitant to make it too specific to the updates
page or otherwise we'd end up with the same issue as with
"suppress_first_run_page" which we're now using also for the updates page. If we
had used a more generic name (e.g. "suppress_info_pages") would've helped us in
that regard.

So what about something along the lines of "updates_awareness" instead to make
it a bit clearer what it's supposed to represent?

Anyway, if you don't think that being too specific to the updates page is an
issue, I'd be happy to change it to "last_update_page_displayed" as you
suggested.

[kzar, 2018/01/25 11:46:25]

Well it was unfortunate about the "suppress_first_run_page" name you're right
but I think being vague when naming this preference as a response is a bad idea.
I'd rather we went with something like "last_update_page_displayed" that was
easy to understand.

[Thomas Greiner, 2018/01/25 13:09:03]

Done.

+
+/**
   * @namespace
   * @static
   */