search

esprehn / chromium-codereview

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

Inexistent patch chunk with extension #88

Closed jchaffraix closed 10 years ago

jchaffraix commented 10 years ago

The first diff on line 1 ('1' added) doesn't exist if you don't use the extension on patchset 1 of:

https://codereview.chromium.org/426373002/

esprehn commented 10 years ago

Yes it does:

https://codereview.chromium.org/426373002/diff/1/content/child/runtime_features.cc

esprehn commented 10 years ago

Also see the raw diff, you definitely added that 1:

https://codereview.chromium.org/download/issue426373002_1.diff