search

wbamberg / securecontext-checker

2 stars 0 forks source link

Some entries in the output are duplicate #1

Open OnkarRuikar opened 9 months ago

OnkarRuikar commented 9 months ago

Some entries in the output are duplicate: 

PresentationConnection.binaryType
PresentationConnection.send
PresentationConnection.send
PresentationConnection.send
PresentationConnection.send
ServiceWorkerRegistration.pushManager

https://github.com/wbamberg/securecontext-checker/blob/5f2380ead578c975c69c9e8d92b817d7d559ceba/secure-context-analysis.md?plain=1#L367C1-L370C28

Also, the section Secure context in WebIDL, not secure context in MDN doesn't include Cache and CacheStorage entries. Although the subpages under them are listed.

wbamberg commented 9 months ago

Some entries in the output are duplicate:

PresentationConnection.binaryType
PresentationConnection.send
PresentationConnection.send
PresentationConnection.send
PresentationConnection.send
ServiceWorkerRegistration.pushManager

https://github.com/wbamberg/securecontext-checker/blob/5f2380ead578c975c69c9e8d92b817d7d559ceba/secure-context-analysis.md?plain=1#L367C1-L370C28

Hm, interesting. That's because they are defined separately in the IDL: https://w3c.github.io/presentation-api/#dom-presentationconnection , but MDN only has one page for them: https://developer.mozilla.org/en-US/docs/Web/API/PresentationConnection/send.

I don't know why the spec author has written it like this, instead of a single member with a union argument.

It probably wouldn't be too hard to collapse methods with the same name into one - although there is the possibility that some might be SC and others not! But I don't think that is realistic.

Also, the section Secure context in WebIDL, not secure context in MDN doesn't include Cache and CacheStorage entries. Although the subpages under them are listed.

That's because it is marked secure context in MDN: developer.mozilla.org/en-US/docs/Web/API/Cachehttps://developer.mozilla.org/en-US/docs/Web/API/Cache, isn't it?

OnkarRuikar commented 9 months ago

Hm, interesting. That's because they are defined separately in the IDL:

We'll have to collapse overloaded methods for MDN purpose. This package hasn't been published on npmjs.com so I copied and tweaked the query-webref.js script.

Also, the section Secure context in WebIDL, not secure context in MDN doesn't include Cache and CacheStorage entries. Although the subpages under them are listed.

That's because it is marked secure context in MDN: developer.mozilla.org/en-US/docs/Web/API/Cachehttps://developer.mozilla.org/en-US/docs/Web/API/Cache, isn't it?

Ohh, the headers weren't at the top (after front-matter).

wbamberg commented 9 months ago

Some entries in the output are duplicate:

Hm, interesting. That's because they are defined separately in the IDL:

We'll have to collapse overloaded methods for MDN purpose.

I'd be interested to know if you saw any other examples of this?

OnkarRuikar commented 9 months ago

I'd be interested to know if you saw any other examples of this?

I could find the following items:

GPUComputePassEncoder.setBindGroup
GPUComputePassEncoder.setBindGroup

GPURenderPassEncoder.setBindGroup
GPURenderPassEncoder.setBindGroup

GPURenderBundleEncoder.setBindGroup
GPURenderBundleEncoder.setBindGroup

CookieStore.get
CookieStore.get

CookieStore.getAll
CookieStore.getAll

CookieStore.set
CookieStore.set

CookieStore.delete
CookieStore.delete

PasswordCredential.PasswordCredential
PasswordCredential.PasswordCredential

PresentationRequest.PresentationRequest
PresentationRequest.PresentationRequest

PresentationConnection.send
PresentationConnection.send
PresentationConnection.send
PresentationConnection.send

ServiceWorker.postMessage
ServiceWorker.postMessage

LockManager.request
LockManager.request

ML.createContext
ML.createContext

ML.createContextSync
ML.createContextSync

MLGraphBuilder.constant
MLGraphBuilder.constant

MLGraphBuilder.clamp
MLGraphBuilder.clamp

MLGraphBuilder.elu
MLGraphBuilder.elu

MLGraphBuilder.hardSigmoid
MLGraphBuilder.hardSigmoid

MLGraphBuilder.hardSwish
MLGraphBuilder.hardSwish

MLGraphBuilder.leakyRelu
MLGraphBuilder.leakyRelu

MLGraphBuilder.linear
MLGraphBuilder.linear

MLGraphBuilder.relu
MLGraphBuilder.relu

MLGraphBuilder.sigmoid
MLGraphBuilder.sigmoid

MLGraphBuilder.softmax
MLGraphBuilder.softmax

MLGraphBuilder.softplus
MLGraphBuilder.softplus

MLGraphBuilder.softsign
MLGraphBuilder.softsign

MLGraphBuilder.tanh
MLGraphBuilder.tanh

XRRay.XRRay
XRRay.XRRay