Bug summary change is split up weirdly in bugmail and inline history

RESOLVED INCOMPLETE

Status

()

RESOLVED INCOMPLETE
6 years ago
6 months ago

People

(Reporter: emorley, Unassigned)

Tracking

Production

Details

(Reporter)

Description

6 years ago
In bug 795314, the summary was changed from:

"Intermittent test_resize_move_windows.xul | Assertion failure: _mOwningThread.GetThread() == PR_GetCurrentThread() (nsScriptCacheCleaner not thread-safe) at nsFrameMessageManager.cpp:1019 (After: ASSERTION: containing block height must be constrained...)"

to

"Intermittent test_resize_move_windows.xul test_pluginstream_err.html test_MozEnteredDomFullscreen_event.xul | Assertion failure: _mOwningThread.GetThread() == PR_GetCurrentThread() (nsScriptCacheCleaner not thread-safe), at nsFrameMessageManager.cpp:1019"

However the bugmail splits up the 'from' value into two parts, but not the 'to' (guess due to length?):

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|Intermittent                |Intermittent
                   |test_resize_move_windows.xu |test_resize_move_windows.xu
                   |l | Assertion failure:      |l
                   |_mOwningThread.GetThread()  |test_pluginstream_err.html
                   |== PR_GetCurrentThread()    |test_MozEnteredDomFullscree
                   |(nsScriptCacheCleaner not   |n_event.xul | Assertion
                   |thread-safe)                |failure:
                   |                            |_mOwningThread.GetThread()
                   |                            |== PR_GetCurrentThread()
                   |                            |(nsScriptCacheCleaner not
                   |                            |thread-safe), at
                   |                            |nsFrameMessageManager.cpp:1
                   |                            |019
            Summary|at                          |
                   |nsFrameMessageManager.cpp:1 |
                   |019 (After: ASSERTION:      |
                   |containing block height     |
                   |must be constrained...)     |

Inline history also splits it into two (see bug 795314 comment 2 for history entry), but worse, it applies strikethrough to only one half (and it's pretty hard to grok what actually changed, given we don't have columns like the bugmail, and the arrow ("→") is easy to miss.
(Reporter)

Updated

6 months ago
Status: NEW → RESOLVED
Last Resolved: 6 months ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.