chromium-codereview
chromium-codereview copied to clipboard
esprehn
A chrome extension that changes the frontend to Rietveld for Chromium
ex. https://codereview.chromium.org/554283003 Here a line with the start of a multi line comment was removed, and then a code line was added, but the syntax state from the removal line...
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...
e.g. on https://codereview.chromium.org/476833003/#msg6 you first have to click on "7 older comments", and then comment 6 is highlighted. It should just expand immediately.
e.g. links like this https://codereview.chromium.org/422593003/diff/500001/include/v8.h#newcode4359 (from the comments) drop you out of the review tool into rietveld. I'd like the diff/patchset of that file to expand and scroll up instead.
It shouldn't let you write a message in the first place I think or have a better error message
e.g. in https://codereview.chromium.org/393123005/ when you click on show diffs for patchset 1, the comment in line 35 is not shown
I don't think it's possible to have a value other than 'me' there, so it would be nice to show the reviewer's list instead. It might be nice to show...
It would be helpful if we could have a quick re-run action associated with failed/stalled bot status bubble. Sometimes there would be problem only with one of the bots and...
