1133900 - Only collect JS telemetry for web content (not add-on or chrome JS)

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Chris Peterson [:cpeterson]]

Attachment: http-telemetry.patch

The telemetry probe in bug 1130123 was adding console.whatever() calls (where "whatever" is an unknown property) in .html files to our __noSuchMethod__ telemetry. Even though this is likely a content bug, we don't want to count these calls as __noSuchMethod__ uses.

This patch adds back the "http" filename check. Our Nightly 38 telemetry counts have been polluted, but next week's Aurora 38 telemetry should be accurate.

Comment 1

[Jan de Mooij [:jandem]]

Comment on attachment 8565546 [details] [diff] [review]
http-telemetry.patch

Review of attachment 8565546 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good to me, r=me with nits below addressed.

::: js/src/frontend/Parser.cpp
@@ +8383,5 @@
>  {
>      JSContext* cx = context->maybeJSContext();
>      if (!cx)
>          return;
> +    const char* filename = getFilename();

Nit: SM style is const char *filename

::: js/src/jscompartment.cpp
@@ +860,5 @@
> +void
> +JSCompartment::addTelemetry(const char* filename, DeprecatedLanguageExtension e)
> +{
> +    // Only report telemetry for web content, not add-ons or chrome JS.
> +    if (addonId || isSystem || !filename || strcmp(filename, "http") != 0)

This strcmp will always return non-zero because it compares the full filename. I think you want strncmp (we also used this function before the refactoring):

strncmp(filename, "http", 4) != 0

::: js/src/jscompartment.h
@@ +557,5 @@
>  
>      void reportTelemetry();
>  
>    public:
> +    void addTelemetry(const char* filename, DeprecatedLanguageExtension e);

Nit: * before the name here too, also in the cpp file.

Comment 2

[Chris Peterson [:cpeterson]]

Thanks! I fixed the nits.

https://hg.mozilla.org/integration/mozilla-inbound/rev/d563f8e2d157

Comment 3

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/d563f8e2d157