1288125 - Diffs for big patches take too long to load

Status: RESOLVED INVALID

(MozReview Graveyard :: Review Board: DiffViewer, defect)

Description

[:Gijs (he/him)]

+++ This bug was initially created as a clone of Bug #1288112 +++

STR:

load https://reviewboard.mozilla.org/r/28271/diff/1#index_header

This is not even a terribly large patch, it's even worse with something like:

https://reviewboard.mozilla.org/r/40859/diff/4/#index_header


ER:

the patch loads promptly, with a minimum of obvious stalling/spinner-ing/loading


AR:
every individual file at the top of the page has a spinner that takes multiple seconds per file to disappear. All the files themselves are also loaded dynamically, each taking ages to load. Even now that the cache on the server is hot and so a ctrl-shift-reload is faster than the initial load when I filed this bug, it takes a full 20 seconds before the page has finished. It was more like several minutes the first time. That's not acceptable performance.

Comment 1

[BMO Automation]

MozReview is now obsolete. Please use Phabricator instead. Closing this bug.