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

Status: RESOLVED INCOMPLETE

(bugzilla.mozilla.org :: User Interface, defect)

Description

[Ed Morley [:emorley]]

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.