Page MenuHomePhabricator

Remove some low-hanging buildStandardPageResponse() methods
ClosedPublic

Authored by epriestley on Nov 2 2015, 8:52 PM.
Tags
None
Referenced Files
F18856124: D14385.id.diff
Sat, Nov 1, 8:11 AM
F18851809: D14385.diff
Fri, Oct 31, 4:23 AM
F18845723: D14385.diff
Wed, Oct 29, 2:10 PM
F18737014: D14385.id.diff
Oct 1 2025, 8:42 AM
F18702109: D14385.id34748.diff
Sep 27 2025, 10:38 PM
F18628073: D14385.diff
Sep 16 2025, 5:31 AM
F18506644: D14385.id.diff
Sep 5 2025, 1:40 AM
F18499264: D14385.diff
Sep 4 2025, 7:35 PM
Subscribers
None

Details

Summary

Ref T9690. I wanted to do an example of how to do these but it looks like most of them are trivial (no callsites) and the rest are a little tricky (weird interaction with frames, or in Releeph).

Test Plan
  • Used grep to look for callsites.
  • Hit all applications locally, everything worked.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Remove some low-hanging buildStandardPageResponse() methods.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Nov 3 2015, 12:11 AM
This revision was automatically updated to reflect the committed changes.