Issue 2245: ChangeScreen2: trailing whitespace is highlighted in non-changed lines
Status:  WontFix
Owner: ----
Closed:  Jan 2015
Project Member Reported by david.pu...@sonymobile.com, Nov 6, 2013
************************************************************
***** NOTE: THIS BUG TRACKER IS FOR GERRIT CODE REVIEW *****
***** DO NOT SUBMIT BUGS FOR CHROME, ANDROID, INTERNAL *****
***** ISSUES WITH YOUR COMPANY'S GERRIT SETUP, ETC.    *****
***** THOSE ISSUE BELONG IN DIFFERENT ISSUE TRACKERS!  *****
************************************************************

Affected Version: 2.8-rc2-126-g8da6200

What steps will reproduce the problem?
1. Upload a change on a file that has trailing whitespace, but not on any line that is touched by the change.
2. View the file diff using the old change screen
3. View the file diff using the new change screen

What is the expected output? What do you see instead?

With the old change screen [1] the existing whitespace is not highlighted.

With the new change screen [2] the existing whitespace is highlighted.

[1] https://android-review.googlesource.com/#/c/63541/1/core/java/android/webkit/WebViewClassic.java

[2] https://android-review.googlesource.com/#/c/63541/1/core/java/android/webkit/WebViewClassic.java,cm

tws-new-changescreen.png
14.3 KB   View   Download
tws-old-changescreen.png
10.2 KB   View   Download
Nov 26, 2013
Project Member #1 david.pu...@sonymobile.com
Michael, do you have any comments on this?  As I understood from the comments on the review where we noticed this issue, this behaviour is intended.  Correct?

If so, we can reject this issue.

Cc: m...@google.com
Jan 19, 2015
Project Member #2 david.pu...@sonymobile.com
Behaviour is as intended.
Status: WontFix
Cc: -m...@google.com