Page MenuHomePhabricator

Prioritize loading of files clicked in ToC
Needs ReviewPublic

Authored by vrana on Nov 23 2013, 7:44 PM.
Tags
None
Referenced Files
F19062245: D7641.id.diff
Nov 29 2025, 3:17 PM
F19053370: D7641.diff
Nov 28 2025, 8:24 AM
F18819180: D7641.id.diff
Oct 22 2025, 4:47 AM
F18817029: D7641.diff
Oct 21 2025, 11:34 AM
F18808810: D7641.diff
Oct 19 2025, 8:47 AM
F18771305: D7641.id17253.diff
Oct 8 2025, 7:00 PM
F18702732: D7641.id.diff
Sep 28 2025, 1:15 AM
F18659749: D7641.id17253.diff
Sep 23 2025, 3:29 PM
This revision needs review, but there are no reviewers specified.

Details

Reviewers
None
Summary

Also fix double loading of files clicked in ToC before they were loaded.

In future, we can detect which file is looked at by user and load that with priority too.

Array is not the best data structure for priority queue but it's not a big deal as we'd usually reorder it just once.

Test Plan

Added sleep(5) to DifferentialChangesetViewController, decreased load_limit to 1, clicked on third file in ToC, saw it loaded before the second.

Diff Detail

Branch
master
Lint
Lint Passed
Unit
No Test Coverage

Event Timeline

vrana updated this revision to Unknown Object (????).Nov 23 2013, 11:32 PM

Generalize

epriestley removed a reviewer: epriestley.

This is obsoleted by D8979, which adds a flexible global priority queue for AJAX requests and covers this use case specifically.