chromium-codereview
chromium-codereview copied to clipboard
esprehn
A chrome extension that changes the frontend to Rietveld for Chromium
Since the blink-chromium merge chromium-codereview is no longer showing try jobs for my blink CLs.
On a CL, upload 2 patches with titles. How do you modify their titles from the GUI?
Sometimes the timestamps of review times can get skewed where the last modified date of a review is in the future. For example:  This is probably just the difference...
https://chrome.google.com/webstore/detail/chromite-butler/bhcnanendmgjjeghamaccjnochlnhcgj/details?hl=en-US
https://codereview.chromium.org/812413003#ps20001 should link to the second patch in the issue, instead it opens the latest one. The #ps20001 in the URL appears to be ignored.
It's a bit annoying to have to manually remove yourself from the authors list... it's also annoying when you do that and then get re-added automatically :P
Please provide a side-by-side diff mode. It make reading some change list _much_ easier.
The new Rietveld UI has 1em of space between each line of text (Owner, Created, Modified, CQ, etc.) and .65em extra space between each line inside a paragraph, which means...
The review tool should provide a single file view to allow reviewing complex files, or change lists. One use case is to have the ability to see a new API...
In https://codereview.chromium.org/812853002 in new-line-19 of LayoutTests/http/tests/navigatorconnect/connect-cross-origin.html (why can't I link directly to a line?), the line's mysteriously been replaced by itself. It'd be nice to see something like github's ...