Opened 3 years ago

Last modified 3 years ago

#5929 closed change

Skip update subscription URLs in AndroidWebRequest if not required — at Version 1

Reported by: asmirnov Assignee:
Priority: Unknown Milestone:
Module: Unknown Keywords:
Cc: Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: no Confidential: no
Tester: Unknown Verified working: no
Review URL(s):



In order to save memory AndroidWebRequest ( filters out not required filters from subscriptions. For example if element hiding is disabled (enableElemhide =falsein ctor it leaves lines from not listed subscriptions ( and removes lines with elemhide rules.

If element hiding is enabled (enableElemhide =truein ctor, this.elemhideEnabled ( is set to true and we need no listed subscription URLs at all as condition is always true. So there will be no isListedSubscriptionUrl() invocations and we don't need subscription URLs list at all.

Thus we can skip calling androidWebRequest.updateSubscriptionURLs(engine.filterEngine); in if element hiding is enabled (by default it is enabled and improve start-up time.

It's good because it happens in UI thread and UI is frozen for the user, so we can make UI more responsive with no cost at all (just skip doing what is not required).

What to change

Skip calling androidwebRequest.updateSubscriptionURLs() if element hiding is enabled.

Change History (1)

comment:1 Changed 3 years ago by asmirnov

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing
Note: See TracTickets for help on using tickets.