Page MenuHomePhabricator

Rename inline comment views to "PHUIDiff" and give them a base class
ClosedPublic

Authored by epriestley on Mar 6 2015, 1:13 PM.
Tags
None
Referenced Files
F18939872: D11996.diff
Nov 11 2025, 7:33 AM
F18844097: D11996.diff
Oct 29 2025, 3:07 AM
F18818250: D11996.id.diff
Oct 21 2025, 10:36 PM
F18812643: D11996.id28884.diff
Oct 20 2025, 11:00 AM
F18784203: D11996.id.diff
Oct 13 2025, 12:06 PM
F18784192: D11996.id28884.diff
Oct 13 2025, 12:06 PM
F18784182: D11996.id28894.diff
Oct 13 2025, 12:06 PM
F18784111: D11996.diff
Oct 13 2025, 11:53 AM
Subscribers

Details

Summary

Ref T2009. These classes are "Differential" now, but are used elsewhere in diff infrastructure (e.g., Diffusion).

  • Rename them to "PHUIDiff".
  • Move them to "src/infrastructure/".
  • Give them a base class.
Test Plan

Interacted with inlines in unified and side-by-side views.

Diff Detail

Repository
rP Phabricator
Branch
uni16
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 4783
Build 4799: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley retitled this revision from to Rename inline comment views to "PHUIDiff" and give them a base class.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Mar 6 2015, 10:57 PM
This revision was automatically updated to reflect the committed changes.