search

esprehn / chromium-codereview

A chrome extension that changes the frontend to Rietveld for Chromium
Other
35 stars 11 forks source link

Lines duplicated after clicking "show context" #103

Open alexelias opened 9 years ago

alexelias commented 9 years ago

Visit https://codereview.chromium.org/542883005/ patchset #1 and expand "ScrollView.cpp". Click "show context" at the bottom of that file. Observe lines 498/499 (500/501 in new file) appear twice, as does the comment draft history associated with line 499.

esprehn commented 9 years ago

Really need to get a better API from rietveld, parsing it's side by side diff into the unified diff is buggy.