HomePhabricator

Fix an issue with symbol lookup identifying path names in Diffusion

Description

Fix an issue with symbol lookup identifying path names in Diffusion

Summary:
Depends on D18939. Ref T13047. Symbol lookup can be activated from a diff (in Differential or Diffusion) or from the static view of a file at a particular commit.

In the latter case, we need to figure out the path a little differently. The character and line number approaches still work as written.

Test Plan:

  • Command-clicked symbols in the Diffusion browse view with blame on and off; saw path, line and char populate properly.
  • Command-clicked symbols in Differential diff view to check I didn't break anything.

Reviewers: amckinley

Reviewed By: amckinley

Maniphest Tasks: T13047

Differential Revision: https://secure.phabricator.com/D18940

Details

Provenance
epriestleyAuthored on Jan 26 2018, 2:42 PM
epriestleyPushed on Jan 26 2018, 9:02 PM
Reviewer
amckinley
Differential Revision
D18940: Fix an issue with symbol lookup identifying path names in Diffusion
Parents
rPfdc36677ba4f: Provide character position information to symbol queries
Branches
Unknown
Tags
Unknown
Tasks
T13047: Plans: Symbol Indexes
Build Status
Buildable 19215
Build 25954: Run Core Tests