Update adblockplusui dependency to hg:0da3ee4abcad (release-2018-4.2)

[greiner]

Background

This imports the follow changes:

Ticket	Status	Resolution	Summary	Component	Owner
#6960	closed	fixed	Reopening the options page causes issues	User-Interface	geo
#6963	closed	fixed	Regression: Filter source missing in issue reports	User-Interface	greiner

And also ​import of the translations from the crowdin.

What to change

Change adblockplusui dependency to hg:0da3ee4abcad git:682d4d6

Hints for testers

• Test that the options page works properly after it has been closed and reopened multiple times (#6960). See the testing instructions here.
• Ensure that the issue reporter includes the Sources for filters - which is the filter list that they come from - in the final report it produces (#6963). ​See this issue for more information.
• Test that ​the imported translations haven't broken the extension for other languages. It seems that the translations for most of the UI have been touched (Issue reporter, mobile options page, options page, popup, first run page, etc.)
• Verify that the large "Block ads on" toggle button shows up in the popup window, on both Firefox and Chrome.
• Verify that the numbered steps ("1 - Select issue" etc) show up at the top of the Issue reporter page, on both Firefox and Chrome.

[kzar]

How can the testers test ​this commit, or does it have no effect on the extension?

[agiammarchi]

Replying to kzar:

How can the testers test ​this commit, or does it have no effect on the extension?

If that wasn't in, Firefox wouldn't work (and neither Edge) so, basically, testing in both Chrome and Firefox would already implicitly validate that change was needed and it's working as meant.

The extension itself is not affected, the DOM / Custom Elements are.

The issue in a nutshell: polyfilled Custom Elements were not properly initialized due an issue with that library that has been fixed, and also, it's already 100% test covered.

Here to answer other questions, if needed.

[abpbot]

A commit referencing this issue has landed:
​Issue 6962 - Update adblockplusui dependency to hg:0da3ee4abcad

[kzar]

Could you give an example of something that wouldn't work in Adblock Plus for Firefox/Edge without that change? I'm still not too clear what it's about to be honest.

[agiammarchi]

Replying to kzar:

Could you give an example of something that wouldn't work in Adblock Plus for Firefox/Edge without that change? I'm still not too clear what it's about to be honest.

Actually, I forgot I've also updated the underlying render library in there, because the one used in 3.0.1 had issues with the infinite scroll table so the core has been updated to be reliable in all cases.

As summary, 3.0.1 already fixed Custom Element issues, 3.1.0 made random thousands of dynamic changes reliable so there's really nothing new to test here, but having a more reliable render core shouldn't hurt anyone, right?

Tests to do:

• be sure the big toggle button is visible in every target browser
• verify the issue reporter top steps and canvas drawer show up

But if these tests were done already, nothing should've changed (and it has been tested that it shouldn't have so maybe you can ignore that change)

[kzar]

Brilliant, thanks I've added those.

[Ross]

Done. The Edge options page issue still needs testing when a build is available but the other issues (translations, popup toggle, issue reporter source/steps) are working as expected.

ABP 3.3.2.2175
Firefox 62 / 51 / Windows 10
Chrome 69 / 49 / Windows 10
Opera 56 / 36 / Windows 10