1575343 - Avoid extra copy when calling nsJSUtils::GetCallingLocation()'s nsACString overload

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, task)

Description

[Andrew McCreight [:mccr8]]

It looks like a number of callers of nsJSUtils::GetCallingLocation()'s nsACString overload do this:

nsAutoCString spec;
nsJSUtils::GetCallingLocation(cx, spec, ...);
... NS_ConvertUTF8toUTF16(spec) ...

This copies the file name into the Cstring, then makes another copy to turn it into a cstring. If I'm reading the code correctly, this could be changed to this to avoid that intermediate copy:

nsAutoString spec;
nsJSUtils::GetCallingLocation(cx, spec, ...);
... spec ...

Comment 1

[Andrew McCreight [:mccr8]]

Attachment: Bug 1575343, part 1 - Avoid a gratuitous string copy by calling the nsAString overload of GetCallingLocation.

The nsAString overload of GetCallingLocation directly converts the
original source file name string into an nsAString. A number of
callers that want the source file name in an nsAString are calling the
nsACString overload of GetCallingLocation, then calling
NS_ConvertUTF8toUTF16. This results in an extra intermediate copy of
the original string data.

Comment 2

[Andrew McCreight [:mccr8]]

Attachment: Bug 1575343, part 2 - Avoid string copies when getting calling location in nsCSPContext.

This code grabs the URI, then tries to overwrite it with information
from GetCallingLocation, then possibly overwrites it again with the
original information, then converts the string.

My patch reorders things so that we try GetCallingLocation() first, so
we only set the values once. In the case where GetCallingLocation()
succeeds it avoids a string copy from 8 to 16 bit.

Comment 3

[Andrew McCreight [:mccr8]]

Attachment: Bug 1575343, part 3 - Avoid copies in workers and service workers when getting calling location.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/df88483c2aaf
part 1 - Avoid a gratuitous string copy by calling the nsAString overload of GetCallingLocation. r=smaug
https://hg.mozilla.org/integration/autoland/rev/71bcb9b6ae4a
part 2 - Avoid string copies when getting calling location in nsCSPContext. r=smaug
https://hg.mozilla.org/integration/autoland/rev/cdc9c1fc871e
part 3 - Avoid copies in workers and service workers when getting calling location. r=asuth

Comment 5

[Andrei Ciure[:aciure]]

https://hg.mozilla.org/mozilla-central/rev/df88483c2aaf
https://hg.mozilla.org/mozilla-central/rev/71bcb9b6ae4a
https://hg.mozilla.org/mozilla-central/rev/cdc9c1fc871e