Skip to content
This repository has been archived by the owner on Sep 26, 2023. It is now read-only.

Fix custom test for ProcessingInstruction API #1198

Merged
merged 2 commits into from
Jun 3, 2021
Merged

Conversation

queengooborg
Copy link
Collaborator

This PR fixes an issue with the custom test for the ProcessingInstruction API. When I initially wrote the custom test, I thought that it had to say xml, when in fact, it was the exact opposite.

@foolip
Copy link
Owner

foolip commented Jun 3, 2021

Both document.createProcessingInstruction('xml', "version='1.0' encoding='UTF-8'") and document.createProcessingInstruction('bcd', "version='1.0' encoding='UTF-8'") work for me in Chrome. Is there some browser which throws an exception?

In any case, it wasn't a great example, because <?xml version='1.0' encoding='UTF-8'?> isn't a processing instruction. Maybe the example from https://developer.mozilla.org/en-US/docs/Web/API/Document/createProcessingInstruction#example would be better?

@queengooborg
Copy link
Collaborator Author

Earlier versions of Safari and Firefox aren't too happy about using xml as the processing instruction's namespace. This was causing some testing issues when working on mdn/browser-compat-data#10724.

We could use the example direct from MDN web docs as well! So long as we're not using xml for the first argument, we should be good.

custom-tests.json Outdated Show resolved Hide resolved
@foolip foolip merged commit 4c940b3 into main Jun 3, 2021
@foolip foolip deleted the ProcessingInstruction branch June 3, 2021 08:20
foolip added a commit that referenced this pull request Jun 4, 2021
Notable changes:

#1194
#1195
#1198
#1203

Added tests:

api.GPUSupportedLimits
api.GPUSupportedLimits.maxBindGroups
api.GPUSupportedLimits.maxDynamicStorageBuffersPerPipelineLayout
api.GPUSupportedLimits.maxDynamicUniformBuffersPerPipelineLayout
api.GPUSupportedLimits.maxSampledTexturesPerShaderStage
api.GPUSupportedLimits.maxSamplersPerShaderStage
api.GPUSupportedLimits.maxStorageBufferBindingSize
api.GPUSupportedLimits.maxStorageBuffersPerShaderStage
api.GPUSupportedLimits.maxStorageTexturesPerShaderStage
api.GPUSupportedLimits.maxTextureArrayLayers
api.GPUSupportedLimits.maxTextureDimension1D
api.GPUSupportedLimits.maxTextureDimension2D
api.GPUSupportedLimits.maxTextureDimension3D
api.GPUSupportedLimits.maxUniformBufferBindingSize
api.GPUSupportedLimits.maxUniformBuffersPerShaderStage
api.GPUSupportedLimits.maxVertexAttributes
api.GPUSupportedLimits.maxVertexBufferArrayStride
api.GPUSupportedLimits.maxVertexBuffers

Removed tests:

api.GPUAdapterLimits
api.GPUAdapterLimits.maxBindGroups
api.GPUAdapterLimits.maxDynamicStorageBuffersPerPipelineLayout
api.GPUAdapterLimits.maxDynamicUniformBuffersPerPipelineLayout
api.GPUAdapterLimits.maxSampledTexturesPerShaderStage
api.GPUAdapterLimits.maxSamplersPerShaderStage
api.GPUAdapterLimits.maxStorageBufferBindingSize
api.GPUAdapterLimits.maxStorageBuffersPerShaderStage
api.GPUAdapterLimits.maxStorageTexturesPerShaderStage
api.GPUAdapterLimits.maxTextureArrayLayers
api.GPUAdapterLimits.maxTextureDimension1D
api.GPUAdapterLimits.maxTextureDimension2D
api.GPUAdapterLimits.maxTextureDimension3D
api.GPUAdapterLimits.maxUniformBufferBindingSize
api.GPUAdapterLimits.maxUniformBuffersPerShaderStage
api.GPUAdapterLimits.maxVertexAttributes
api.GPUAdapterLimits.maxVertexBufferArrayStride
api.GPUAdapterLimits.maxVertexBuffers
@foolip foolip mentioned this pull request Jun 4, 2021
foolip added a commit that referenced this pull request Jun 4, 2021
Notable changes:

#1194
#1195
#1198
#1203

Added tests:

api.GPUSupportedLimits
api.GPUSupportedLimits.maxBindGroups
api.GPUSupportedLimits.maxDynamicStorageBuffersPerPipelineLayout
api.GPUSupportedLimits.maxDynamicUniformBuffersPerPipelineLayout
api.GPUSupportedLimits.maxSampledTexturesPerShaderStage
api.GPUSupportedLimits.maxSamplersPerShaderStage
api.GPUSupportedLimits.maxStorageBufferBindingSize
api.GPUSupportedLimits.maxStorageBuffersPerShaderStage
api.GPUSupportedLimits.maxStorageTexturesPerShaderStage
api.GPUSupportedLimits.maxTextureArrayLayers
api.GPUSupportedLimits.maxTextureDimension1D
api.GPUSupportedLimits.maxTextureDimension2D
api.GPUSupportedLimits.maxTextureDimension3D
api.GPUSupportedLimits.maxUniformBufferBindingSize
api.GPUSupportedLimits.maxUniformBuffersPerShaderStage
api.GPUSupportedLimits.maxVertexAttributes
api.GPUSupportedLimits.maxVertexBufferArrayStride
api.GPUSupportedLimits.maxVertexBuffers

Removed tests:

api.GPUAdapterLimits
api.GPUAdapterLimits.maxBindGroups
api.GPUAdapterLimits.maxDynamicStorageBuffersPerPipelineLayout
api.GPUAdapterLimits.maxDynamicUniformBuffersPerPipelineLayout
api.GPUAdapterLimits.maxSampledTexturesPerShaderStage
api.GPUAdapterLimits.maxSamplersPerShaderStage
api.GPUAdapterLimits.maxStorageBufferBindingSize
api.GPUAdapterLimits.maxStorageBuffersPerShaderStage
api.GPUAdapterLimits.maxStorageTexturesPerShaderStage
api.GPUAdapterLimits.maxTextureArrayLayers
api.GPUAdapterLimits.maxTextureDimension1D
api.GPUAdapterLimits.maxTextureDimension2D
api.GPUAdapterLimits.maxTextureDimension3D
api.GPUAdapterLimits.maxUniformBufferBindingSize
api.GPUAdapterLimits.maxUniformBuffersPerShaderStage
api.GPUAdapterLimits.maxVertexAttributes
api.GPUAdapterLimits.maxVertexBufferArrayStride
api.GPUAdapterLimits.maxVertexBuffers
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants