Closed Bug 964964 Opened 10 years ago Closed 10 years ago

Try to remove window.pkcs11

Categories

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

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla29

People

(Reporter: Ms2ger, Assigned: emk)

References

Details

(Keywords: dev-doc-complete, site-compat, Whiteboard: [qa-])

Attachments

(1 file)

It's been returning null since bug 326628 (Fx 3.0.14). Maybe sniffing code doesn't use it.
Comment on attachment 8367352 [details] [diff] [review]
Remove window.pkcs11

Not it!
Attachment #8367352 - Flags: review?(Ms2ger) → review?(bzbarsky)
Comment on attachment 8367352 [details] [diff] [review]
Remove window.pkcs11

Need to rev nsIDOMWindow IID.

r=me with that, but be _very_ careful of object-sniffing bogosity on sites.  :(
Attachment #8367352 - Flags: review?(bzbarsky) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/c57cf5f7cc48
Assignee: nobody → VYV03354
Status: NEW → ASSIGNED
(In reply to :Ms2ger from comment #0)
> Maybe sniffing code doesn't use it.

It was somewhat (not much) popular in Germany because recommended here:
http://www.lipfert-malik.de/webdesign/tutorial/bsp/browser_js_test.html
© K.Lipfert Düsseldorf 2002-2008

Haven't seen it recently, but heads up if there are regressions on German sites please.
(In reply to j.j. (mostly inactive) from comment #5)
> (In reply to :Ms2ger from comment #0)
> > Maybe sniffing code doesn't use it.
> 
> It was somewhat (not much) popular in Germany because recommended here:
> http://www.lipfert-malik.de/webdesign/tutorial/bsp/browser_js_test.html
> © K.Lipfert Düsseldorf 2002-2008
> 
> Haven't seen it recently, but heads up if there are regressions on German
> sites please.

Don't worry. The check |if(window.pkcs11)| is already broken because window.pkcs11 will return a falsy value (null) since Firefox 3.0.14.
https://hg.mozilla.org/mozilla-central/rev/c57cf5f7cc48
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
Whiteboard: [qa-]
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.