1490627 - browser-captivePortal.js leaks cps variable into the browser window's global scope

Status: RESOLVED FIXED

(Firefox :: General, defect, P3)

Description

[Dão Gottwald [:dao]]

Attachment: patch

Comment 1

[(inactive) Nihanth Subramanya [:nhnt11]]

Comment on attachment 9008363 [details] [diff] [review]
patch

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

Nice catch, thanks.

::: browser/base/content/browser-captivePortal.js
@@ -1,5 @@
>  /* This Source Code Form is subject to the terms of the Mozilla Public
>   * License, v. 2.0. If a copy of the MPL was not distributed with this
>   * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
>  
> -XPCOMUtils.defineLazyServiceGetter(this, "cps",

What do you think about keeping this lazy service getter but s/this/CaptivePortalWatcher/ so that the property is defined on the CaptivePortalWatcher object?

Comment 2

[Dão Gottwald [:dao]]

(In reply to Nihanth Subramanya [:nhnt11] from comment #1)
> What do you think about keeping this lazy service getter but
> s/this/CaptivePortalWatcher/ so that the property is defined on the
> CaptivePortalWatcher object?

This is accessed in init so there's no point in keeping this lazy. It would just add overhead.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/6c2e6f132f46
browser-captivePortal.js leaks cps variable into the browser window's global scope. r=nhnt11

Comment 4

[Guilherme Lima]

https://hg.mozilla.org/mozilla-central/rev/6c2e6f132f46 - landed 22 hours ago