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
F15454180: D7641.id.diff
Sat, Mar 29, 4:49 PM
F15446447: D7641.diff
Thu, Mar 27, 7:08 PM
F15444601: D7641.diff
Thu, Mar 27, 10:17 AM
F15441853: D7641.id.diff
Wed, Mar 26, 8:24 PM
F15385375: D7641.id17252.diff
Mar 14 2025, 10:25 PM
F15283117: D7641.diff
Mar 4 2025, 5:26 AM
Unknown Object (File)
Feb 26 2025, 6:09 PM
Unknown Object (File)
Feb 24 2025, 8:56 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.