Page MenuHomePhabricator
Create Task
Maniphest T156917

Replace custom saved pages cache with OkHttp cache
Closed, ResolvedPublic
Actions

Description

  • Add second OkHttp cache for saved pages independent of app cache files (cannot be deleted by clearing app cache)
    • Customize to allow pages to be marked as sticky (not needed)
    • Add as a second data source
  • Remove saved pages cache and update SavedPagesSyncService
  • Figure out what to do about Fresco's special cache and make a ticket
  • Create a ticket for updating the sync service to use the new headers and CacheDelegate.remove() method if it doesn't already exist

Develop and merge to separate branch as necessary

Details

SubjectRepoBranchLines +/-
Update: use the new save page download apiapps/android/wikipediamaster+158 -23
Update: add cache option to PageClientapps/android/wikipediamaster+170 -79
Chore: remove pageCombo endpointapps/android/wikipediamaster+0 -188
Update: remove file-based saved pagesapps/android/wikipediamaster+21 -809
Update: add second OkHttp cache for saved contentapps/android/wikipediamaster+733 -31
Chore: add PageClient testsapps/android/wikipediamaster+97 -0
Update: remove page combo endpoint usageapps/android/wikipediamaster+3 -279
Update: remove saved page image downloading and link remappingapps/android/wikipediamaster+24 -307
Hygiene: remove unused SavedPage methods and infer nullityapps/android/wikipediamaster+12 -29
Fix: save page downloadingapps/android/wikipediamaster+115 -84
Update: expose WikiSite URIapps/android/wikipediamaster+13 -2
Update: replace underlying image URL parserapps/android/wikipediamaster+608 -29
Hygiene: add jsoup v1.10.2apps/android/wikipediamaster+26 -0
Hygiene: rename ImageUrlMap to ImageUrlHtmlParserapps/android/wikipediamaster+26 -21
Update: use OkHttp for WebView requestsapps/android/wikipediamaster+81 -0
Update: do not forbid Fresco image request cachingapps/android/wikipediamaster+44 -3
Hygiene: allow PageClients requestors to specify a/synchronicityapps/android/wikipediamaster+139 -221
Hygiene: replace all Response.isSuccessful() calls with interceptorapps/android/wikipediamaster+255 -369
Hygiene: hold WikiCachedService in clients, not implementationapps/android/wikipediamaster+42 -25
Hygiene: consolidate service caches into WikiCachedServiceapps/android/wikipediamaster+30 -157
Hygiene: isolate the Retrofit *PageClient servicesapps/android/wikipediamaster+177 -164
Hygiene: move Wikipedia Zero header checks from clients to response interceptorapps/android/wikipediamaster+50 -77
Hygiene: clean up empty string checksapps/android/wikipediamaster+7 -6
Hygiene: move MediaWiki fallback checks from clients to response interceptorapps/android/wikipediamaster+89 -34
Hygiene: rename PageDataClient to PageFragmentLoadStateapps/android/wikipediamaster+25 -25
Hygiene: delete PageLoadStrategyapps/android/wikipediamaster+4 -72
Hygiene: rename PageService classes to PageClientsapps/android/wikipediamaster+43 -43
Update: add second OkHttp cache for saved contentapps/android/wikipediasave-cache+494 -5
Hygiene: remove duplicate method in WidgetProviderFeaturedPageapps/android/wikipediasave-cache+1 -7
Hygiene: remove duplicate method in WidgetProviderFeaturedPageapps/android/wikipediamaster+1 -7
Show related patches Customize query in gerrit

Related Objects
Search...

Event Timeline

• Niedzielski created this task.Feb 1 2017, 2:32 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptFeb 1 2017, 2:32 PM
• Niedzielski added a comment.Feb 1 2017, 2:33 PM

@Mholloway, @Dbrant, we talked about this in our meeting but I don't think we ever created the task. Please update if I've missed something

• NHarateh_WMF moved this task from Needs Triage to Product Backlog on the Wikipedia-Android-App-Backlog board.Feb 1 2017, 4:20 PM
• Niedzielski claimed this task.Feb 1 2017, 8:13 PM
• Niedzielski edited projects, added Wikipedia-Android-App-Backlog (Android-app-release-v2.6.19x-Bermuda🌴); removed Wikipedia-Android-App-Backlog.Feb 13 2017, 4:21 PM
• Niedzielski moved this task from To Do to Doing on the Wikipedia-Android-App-Backlog (Android-app-release-v2.6.19x-Bermuda🌴) board.Feb 13 2017, 4:24 PM
• NHarateh_WMF added a subtask: T119082: Fully suppress link preview when content is unavailable (no network connection and page not cached or saved).Feb 13 2017, 4:26 PM
• Niedzielski mentioned this in T149991: Link preview does not search for articles in cache.Feb 16 2017, 11:13 PM
gerritbot subscribed.Feb 23 2017, 10:36 PM

Change 339574 had a related patch set uploaded (by Niedzielski):
Update: add second OkHttp cache for saved content

https://gerrit.wikimedia.org/r/339574

gerritbot added a project: Patch-For-Review.Feb 23 2017, 10:36 PM
• Niedzielski updated the task description. (Show Details)Feb 23 2017, 10:43 PM
• Niedzielski mentioned this in T146944: Mobile-sections replacement.Feb 28 2017, 10:50 PM
gerritbot added a comment.Feb 28 2017, 11:12 PM

Change 340433 had a related patch set uploaded (by Niedzielski; owner: Sniedzielski):
[apps/android/wikipedia] Hygiene: remove duplicate method in WidgetProviderFeaturedPage

https://gerrit.wikimedia.org/r/340433

gerritbot added a comment.Mar 1 2017, 8:25 PM

Change 340563 had a related patch set uploaded (by Niedzielski; owner: Sniedzielski):
[apps/android/wikipedia] Hygiene: remove duplicate method in WidgetProviderFeaturedPage

https://gerrit.wikimedia.org/r/340563

gerritbot added a comment.Mar 2 2017, 2:52 AM

Change 340676 had a related patch set uploaded (by Niedzielski; owner: Sniedzielski):
[apps/android/wikipedia] Hygiene: rename PageService classes to PageClients

https://gerrit.wikimedia.org/r/340676

gerritbot added a comment.Mar 2 2017, 2:52 AM

Change 340677 had a related patch set uploaded (by Niedzielski; owner: Sniedzielski):
[apps/android/wikipedia] Hygiene: delete PageLoadStrategy

https://gerrit.wikimedia.org/r/340677

gerritbot added a comment.Mar 2 2017, 2:52 AM

Change 340678 had a related patch set uploaded (by Niedzielski; owner: Sniedzielski):
[apps/android/wikipedia] Hygiene: rename PageDataClient to PageFragmentLoadState

https://gerrit.wikimedia.org/r/340678

gerritbot added a comment.Mar 2 2017, 2:53 AM

Change 340680 had a related patch set uploaded (by Niedzielski; owner: Sniedzielski):
[apps/android/wikipedia] Hygiene: move MediaWiki fallback checks from clients to response interceptor

https://gerrit.wikimedia.org/r/340680

gerritbot added a comment.Mar 2 2017, 5:07 AM

Change 340687 had a related patch set uploaded (by Niedzielski; owner: Sniedzielski):
[apps/android/wikipedia] Hygiene: clean up empty string checks

https://gerrit.wikimedia.org/r/340687

gerritbot added a comment.Mar 2 2017, 5:07 AM

Change 340688 had a related patch set uploaded (by Niedzielski; owner: Sniedzielski):
[apps/android/wikipedia] Hygiene: move Wikipedia Zero header checks from clients to response interceptor

https://gerrit.wikimedia.org/r/340688

gerritbot added a comment.Mar 2 2017, 4:21 PM

Change 340761 had a related patch set uploaded (by Niedzielski; owner: Sniedzielski):
[apps/android/wikipedia] Hygiene: isolate the Retrofit *PageClient services

https://gerrit.wikimedia.org/r/340761

gerritbot added a comment.Mar 2 2017, 4:21 PM

Change 340762 had a related patch set uploaded (by Niedzielski; owner: Sniedzielski):
[apps/android/wikipedia] Hygiene: consolidate service caches into WikiCachedService

https://gerrit.wikimedia.org/r/340762

gerritbot added a comment.Mar 2 2017, 5:36 PM

Change 340774 had a related patch set uploaded (by Niedzielski; owner: Sniedzielski):
[apps/android/wikipedia] Hygiene: hold WikiCachedService in clients, not implementation

https://gerrit.wikimedia.org/r/340774

gerritbot added a comment.Mar 2 2017, 9:51 PM

Change 340834 had a related patch set uploaded (by Niedzielski; owner: Sniedzielski):
[apps/android/wikipedia] Hygiene: replace all Response.isSuccessful() calls with interceptor

https://gerrit.wikimedia.org/r/340834

gerritbot added a comment.Mar 2 2017, 10:18 PM

Change 340563 merged by Niedzielski:
[apps/android/wikipedia] Hygiene: remove duplicate method in WidgetProviderFeaturedPage

https://gerrit.wikimedia.org/r/340563

gerritbot added a comment.Mar 2 2017, 11:55 PM

Change 340904 had a related patch set uploaded (by Niedzielski; owner: Sniedzielski):
[apps/android/wikipedia] Hygiene: allow PageClients requestors to specify a/synchronicity

https://gerrit.wikimedia.org/r/340904

gerritbot added a comment.Mar 3 2017, 12:16 AM

Change 340433 abandoned by Niedzielski:
Hygiene: remove duplicate method in WidgetProviderFeaturedPage

Reason:
merged to master yo!

https://gerrit.wikimedia.org/r/340433

gerritbot added a comment.Mar 3 2017, 12:20 AM

Change 340906 had a related patch set uploaded (by Niedzielski; owner: Sniedzielski):
[apps/android/wikipedia] Update: add second OkHttp cache for saved content

https://gerrit.wikimedia.org/r/340906

gerritbot added a comment.Mar 3 2017, 12:25 AM

Change 339574 abandoned by Niedzielski:
Update: add second OkHttp cache for saved content

Reason:
Moved to I45825777d18cbbc686d757a715002464cf94ae3f

https://gerrit.wikimedia.org/r/339574

gerritbot added a comment.Mar 3 2017, 2:25 PM

Change 340676 merged by jenkins-bot:
[apps/android/wikipedia] Hygiene: rename PageService classes to PageClients

https://gerrit.wikimedia.org/r/340676

gerritbot added a comment.Mar 3 2017, 2:34 PM

Change 340677 merged by jenkins-bot:
[apps/android/wikipedia] Hygiene: delete PageLoadStrategy

https://gerrit.wikimedia.org/r/340677

gerritbot added a comment.Mar 3 2017, 2:49 PM

Change 340678 merged by jenkins-bot:
[apps/android/wikipedia] Hygiene: rename PageDataClient to PageFragmentLoadState

https://gerrit.wikimedia.org/r/340678

gerritbot added a comment.Mar 3 2017, 5:08 PM

Change 340680 merged by jenkins-bot:
[apps/android/wikipedia] Hygiene: move MediaWiki fallback checks from clients to response interceptor

https://gerrit.wikimedia.org/r/340680

gerritbot added a comment.Mar 3 2017, 5:09 PM

Change 340687 merged by jenkins-bot:
[apps/android/wikipedia] Hygiene: clean up empty string checks

https://gerrit.wikimedia.org/r/340687

gerritbot added a comment.Mar 3 2017, 9:30 PM

Change 340688 merged by jenkins-bot:
[apps/android/wikipedia] Hygiene: move Wikipedia Zero header checks from clients to response interceptor

https://gerrit.wikimedia.org/r/340688

gerritbot added a comment.Mar 3 2017, 9:52 PM

Change 340761 merged by jenkins-bot:
[apps/android/wikipedia] Hygiene: isolate the Retrofit *PageClient services

https://gerrit.wikimedia.org/r/340761

gerritbot added a comment.Mar 6 2017, 2:24 PM

Change 340762 merged by jenkins-bot:
[apps/android/wikipedia] Hygiene: consolidate service caches into WikiCachedService

https://gerrit.wikimedia.org/r/340762

gerritbot added a comment.Mar 6 2017, 2:34 PM

Change 340774 merged by jenkins-bot:
[apps/android/wikipedia] Hygiene: hold WikiCachedService in clients, not implementation

https://gerrit.wikimedia.org/r/340774

gerritbot added a comment.Mar 6 2017, 2:50 PM

Change 340834 merged by jenkins-bot:
[apps/android/wikipedia] Hygiene: replace all Response.isSuccessful() calls with interceptor

https://gerrit.wikimedia.org/r/340834

gerritbot added a comment.Mar 6 2017, 8:26 PM

Change 341386 had a related patch set uploaded (by niedzielski; owner: sniedzielski):
[apps/android/wikipedia] Remove page combo endpoint usage

https://gerrit.wikimedia.org/r/341386

gerritbot added a comment.Mar 6 2017, 8:26 PM

Change 341387 had a related patch set uploaded (by niedzielski; owner: sniedzielski):
[apps/android/wikipedia] Add cache option to PageClient

https://gerrit.wikimedia.org/r/341387

gerritbot added a comment.Mar 7 2017, 2:00 PM

Change 340904 merged by jenkins-bot:
[apps/android/wikipedia] Hygiene: allow PageClients requestors to specify a/synchronicity

https://gerrit.wikimedia.org/r/340904

gerritbot added a comment.Mar 17 2017, 12:53 PM

Change 343287 had a related patch set uploaded (by Niedzielski; owner: Sniedzielski):
[apps/android/wikipedia] Hygiene: remove unused SavedPage methods and infer nullity

https://gerrit.wikimedia.org/r/343287

gerritbot added a comment.Mar 17 2017, 12:53 PM

Change 343288 had a related patch set uploaded (by Niedzielski; owner: Sniedzielski):
[apps/android/wikipedia] Hygiene: rename ImageUrlMap to ImageUrlHtmlParser

https://gerrit.wikimedia.org/r/343288

gerritbot added a comment.Mar 17 2017, 12:53 PM

Change 343289 had a related patch set uploaded (by Niedzielski; owner: Sniedzielski):
[apps/android/wikipedia] Update: remove saved page image downloading and link remapping

https://gerrit.wikimedia.org/r/343289

gerritbot added a comment.Mar 17 2017, 12:53 PM

Change 343290 had a related patch set uploaded (by Niedzielski; owner: Sniedzielski):
[apps/android/wikipedia] Update: use the new save page download api

https://gerrit.wikimedia.org/r/343290

gerritbot added a comment.Mar 17 2017, 12:53 PM

Change 343291 had a related patch set uploaded (by Niedzielski; owner: Sniedzielski):
[apps/android/wikipedia] Hygiene: add jsoup v1.10.2

https://gerrit.wikimedia.org/r/343291

gerritbot added a comment.Mar 17 2017, 12:53 PM

Change 343292 had a related patch set uploaded (by Niedzielski; owner: Sniedzielski):
[apps/android/wikipedia] Update: prepare to replace image URL parser

https://gerrit.wikimedia.org/r/343292

gerritbot added a comment.Mar 17 2017, 12:54 PM

Change 343293 had a related patch set uploaded (by Niedzielski; owner: Sniedzielski):
[apps/android/wikipedia] Update: expose WikiSite URI

https://gerrit.wikimedia.org/r/343293

gerritbot added a comment.Mar 17 2017, 12:54 PM

Change 343294 had a related patch set uploaded (by Niedzielski; owner: Sniedzielski):
[apps/android/wikipedia] Update: use OkHttp for WebView requests

https://gerrit.wikimedia.org/r/343294

gerritbot added a comment.Mar 17 2017, 12:54 PM

Change 343295 had a related patch set uploaded (by Niedzielski; owner: Sniedzielski):
[apps/android/wikipedia] Update: do not forbid Fresco image request caching

https://gerrit.wikimedia.org/r/343295

gerritbot added a comment.Mar 17 2017, 12:54 PM

Change 343296 had a related patch set uploaded (by Niedzielski; owner: Sniedzielski):
[apps/android/wikipedia] Fix: save page downloading

https://gerrit.wikimedia.org/r/343296

• Mholloway added a comment.Mar 20 2017, 4:17 PM

I have a couple of questions about the relationship between the two caches that I'm not seeing clear answers to just from looking at the implementation.

  1. Can a page be in both the http and saved pages cache, or are they mutually exclusive?
  2. If a page is in both caches and fetched from the network while online, are both caches updated?
  3. What's the order of preference among { network, http_cache, saved_pages_cache } If the user is online and loads a page that's in the saved page cache?
  4. Is the reading list page database table updated when the saved pages cache is updated, so that getting the mtime for the page's database row will always result in an accurate last-modified time?

For context, I'm trying to figure out how T156082 should be done (particularly retrieving the last-modified timestamp), which is the motivating factor behind (4). The rest are just to firm up my understanding of how this works generally.

• Mholloway closed subtask T119082: Fully suppress link preview when content is unavailable (no network connection and page not cached or saved) as Declined.Mar 20 2017, 4:33 PM
• Niedzielski added a comment.Mar 20 2017, 5:07 PM

I'm still working out bugs but:

1

A saved page may exist in any combination of caches. Consider the following scenarios:

  1. [✓Saved cache] [✓ Network cache] A user marks a page as saved and both caches have space
  2. [✓Saved cache] [✗ Network cache] A user marks a page as saved, then clears the app cache
  3. [✗ Saved cache] [✓ Network cache] A rare but supported case that might occur because the disk cache is corrupted or the disk cache is set to be smaller than the app cache
  4. [✗Saved cache] [✗ Network cache] A user marks many pages as saved and one of them falls out of both LRU caches

A non-saved page may only appear in the network cache (no change in behavior).

2

There's been no change to how the default caching works. The network and network cache are always attempted when permitted by headers and request type. The saved page cache is also always attempted when permitted by the same headers and request type. For every network request, the saved page cache is only modified when permitted by the same headers and request type AND any of the following circumstances: 1) new: the custom X-Save header is true, 2) update: the page was previously saved. The saved page cache can also drop entries through an LRU disk overflow mechanism and a couple exposed APIs for removing and retrieving a cache entry.

3

The normal cache invalidation rules apply. The response returned is either: network only, updated network cached version, updated network / network cached saved cached version, saved cached version. The network cache is updated by network responses. The saved page cache is updated by network + network cached responses.

4

There hasn't been any change in this functionality except which APIs are used to download a page (lead + sections with X-Save header instead of pageCombo). The behavior I see before and after the patches is that any time a page is reloaded, the row is marked for update and the save page sync service runs which updates the saved page cache. Always use the database to get saved page status as the saved page cache can drop pages when disk space is exhausted.

gerritbot added a comment.Mar 21 2017, 8:59 PM

Change 343295 merged by jenkins-bot:
[apps/android/wikipedia] Update: do not forbid Fresco image request caching

https://gerrit.wikimedia.org/r/343295

gerritbot added a comment.Mar 21 2017, 9:01 PM

Change 343294 merged by jenkins-bot:
[apps/android/wikipedia] Update: use OkHttp for WebView requests

https://gerrit.wikimedia.org/r/343294

gerritbot added a comment.Mar 21 2017, 9:06 PM

Change 343288 merged by jenkins-bot:
[apps/android/wikipedia] Hygiene: rename ImageUrlMap to ImageUrlHtmlParser

https://gerrit.wikimedia.org/r/343288

gerritbot added a comment.Mar 21 2017, 9:09 PM

Change 343291 merged by jenkins-bot:
[apps/android/wikipedia] Hygiene: add jsoup v1.10.2

https://gerrit.wikimedia.org/r/343291

gerritbot added a comment.Mar 22 2017, 4:50 PM

Change 343292 merged by jenkins-bot:
[apps/android/wikipedia@master] Update: replace underlying image URL parser

https://gerrit.wikimedia.org/r/343292

gerritbot added a comment.Mar 22 2017, 11:12 PM

Change 340907 had a related patch set uploaded (by Niedzielski; owner: Sniedzielski):
[apps/android/wikipedia@master] Update: remove file-based saved pages

https://gerrit.wikimedia.org/r/340907

gerritbot added a comment.Mar 22 2017, 11:12 PM

Change 344261 had a related patch set uploaded (by Niedzielski; owner: Sniedzielski):
[apps/android/wikipedia@master] Chore: add PageClient tests

https://gerrit.wikimedia.org/r/344261

gerritbot added a comment.Mar 22 2017, 11:12 PM

Change 344262 had a related patch set uploaded (by Niedzielski; owner: Sniedzielski):
[apps/android/wikipedia@master] Chore: remove pageCombo endpoint

https://gerrit.wikimedia.org/r/344262

gerritbot added a comment.Mar 22 2017, 11:15 PM

Change 343293 abandoned by Niedzielski:
Update: expose WikiSite URI

Reason:
Replaced

https://gerrit.wikimedia.org/r/343293

gerritbot added a comment.Mar 22 2017, 11:16 PM

Change 343296 abandoned by Niedzielski:
Fix: save page downloading

Reason:
Replaced. @Mholloway, I will transplant your comment

https://gerrit.wikimedia.org/r/343296

gerritbot added a comment.Mar 22 2017, 11:18 PM

Change 343287 abandoned by Niedzielski:
Hygiene: remove unused SavedPage methods and infer nullity

Reason:
Replaced

https://gerrit.wikimedia.org/r/343287

gerritbot added a comment.Mar 22 2017, 11:18 PM

Change 343289 abandoned by Niedzielski:
Update: remove saved page image downloading and link remapping

Reason:
Replaced

https://gerrit.wikimedia.org/r/343289

gerritbot added a comment.Mar 22 2017, 11:19 PM

Change 341386 abandoned by Niedzielski:
Update: remove page combo endpoint usage

Reason:
Replaced

https://gerrit.wikimedia.org/r/341386

• Niedzielski moved this task from Doing to Code Review on the Wikipedia-Android-App-Backlog (Android-app-release-v2.6.19x-Bermuda🌴) board.Mar 22 2017, 11:20 PM
• Niedzielski added a comment.Mar 22 2017, 11:24 PM

@Dbrant @Mholloway, ok this guy is finally ready! Well, there was one tiny URI improvement from Michael I had to transplant over from a previous patchset but that can be captured in a new ticket if you want to merge them sooner rather than later (sorry, I gotta pack for an early morning flight!). I've added a buttload of tests so hopefully things are working right but please also test on your own devices. I don't recommend this change for the release but I'll leave it to you both to decide. \o/

gerritbot added a comment.Mar 23 2017, 2:01 PM

Change 344261 merged by jenkins-bot:
[apps/android/wikipedia@master] Chore: add PageClient tests

https://gerrit.wikimedia.org/r/344261

gerritbot added a comment.Apr 5 2017, 9:09 PM

Change 340906 merged by jenkins-bot:
[apps/android/wikipedia@master] Update: add second OkHttp cache for saved content

https://gerrit.wikimedia.org/r/340906

gerritbot added a comment.Apr 5 2017, 10:41 PM

Change 340907 merged by jenkins-bot:
[apps/android/wikipedia@master] Update: remove file-based saved pages

https://gerrit.wikimedia.org/r/340907

gerritbot added a comment.Apr 5 2017, 10:41 PM

Change 344262 merged by jenkins-bot:
[apps/android/wikipedia@master] Chore: remove pageCombo endpoint

https://gerrit.wikimedia.org/r/344262

gerritbot added a comment.Apr 6 2017, 3:53 PM

Change 341387 merged by jenkins-bot:
[apps/android/wikipedia@master] Update: add cache option to PageClient

https://gerrit.wikimedia.org/r/341387

gerritbot added a comment.Apr 6 2017, 5:08 PM

Change 343290 merged by jenkins-bot:
[apps/android/wikipedia@master] Update: use the new save page download api

https://gerrit.wikimedia.org/r/343290

• Niedzielski moved this task from Code Review to Design Signoff on the Wikipedia-Android-App-Backlog (Android-app-release-v2.6.19x-Bermuda🌴) board.Apr 10 2017, 3:05 PM
RHo moved this task from Design Signoff to QA Signoff on the Wikipedia-Android-App-Backlog (Android-app-release-v2.6.19x-Bermuda🌴) board.Apr 17 2017, 1:40 PM
ABorbaWMF moved this task from QA Signoff to Ready for Signoff on the Wikipedia-Android-App-Backlog (Android-app-release-v2.6.19x-Bermuda🌴) board.Apr 19 2017, 7:27 PM
ABorbaWMF subscribed.

Tested on the App Build 2.5.194-alpha-2017-04-19
Devices - Nexus 4 Android 5.1 & Pixel Android 7.1.1

Added reading lists and built up some history. Tried some offline testing. Cleared the app cache and did more testing. Everything appears to be working.

Dbrant moved this task from Ready for Signoff to Done on the Wikipedia-Android-App-Backlog (Android-app-release-v2.6.19x-Bermuda🌴) board.Apr 24 2017, 2:42 PM
• Niedzielski closed this task as Resolved.Apr 24 2017, 7:59 PM
• Mholloway mentioned this in T158349: Isolate page data client responsibilities.Sep 22 2017, 7:44 PM
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL