823448 - Show exact replacement of code and not only the addition

Status: RESOLVED INCOMPLETE

(bugzilla.mozilla.org :: Splinter, defect, P5)

Description

[Henrik Skupin [:whimboo][⌚️UTC+1]]

As you can see on bug 809431 comment 6 there is only the addition of code shown:

> Review of attachment 693482 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> ::: tests/functional/testAwesomeBar/testGoButton.js
> @@ +50,5 @@
> >    expect.ok(!utils.isDisplayed(controller, goButton), "Go button is hidden");
> >  
> >    // Check if an element with an id of 'organization' exists
> >    var pageElement = new elementslib.ID(controller.tabs.activeTab, "organization");
> > +  assert.ok(pageElement, "'Organization' element has been found");

But it was a replacement actually. So it's important to merge both sides for the final display in the bug comment.

Comment 1

[:glob ✱]

this requires treating each comment as Splinter.CHANGED, and fixing up the @@ lines for the created patch.  i've given it a shot, but as it isn't a trivial change i'll put this back onto the pile for now.

Comment 2

[Kohei Yoshino [:kohei]]

Splinter is retiring.