Peter Van der Beken [:peterv] Assignee
	Description • 11 years ago

I removed it from a bunch of interfaces that have it now:

AnalyserNode
ArchiveReader
AudioBuffer
AudioBufferSourceNode
AudioContext
AudioDestinationNode
AudioListener
AudioParam
AudioStreamTrack
battery::BatteryManager
BiquadFilterNode
CallsList
CameraControl
CameraManager
CanvasGradient
CanvasPattern
CanvasRenderingContext2D
TextMetrics
ChannelMergerNode
ChannelSplitterNode
ConvolverNode
Coordinates
DataChannel
DOMError
DOMRequest
DelayNode
DesktopNotification
DesktopNotificationCenter
DeviceStorage
DynamicsCompressorNode
FileHandle
FileReader
GainNode
Gamepad
Geolocation
GetUserMediaRequest
HTMLCanvasPrintState
MozInterAppConnection
MozInterAppConnectionRequest
MozInterAppMessagePort
MediaElementAudioSourceNode
MediaError
MediaRecorder
MediaSource
MediaStreamAudioDestinationNode
MediaStreamAudioSourceNode
MediaStream
MediaStreamList
MessageChannel
MessagePort
MessagePortList
PowerManager
time::TimeManager
Notification
OscillatorNode
PannerNode
PeriodicWave
PhoneNumberService
Position
PositionError
Promise
PushManager
mozRTCIceCandidate
mozRTCPeerConnection
mozRTCSessionDescription
ScriptProcessorNode
SettingsLock
SettingsManager
SourceBuffer
SourceBufferList
SpeechGrammar
SpeechGrammarList
SpeechRecognitionAlternative
SpeechRecognition
SpeechRecognitionResult
SpeechRecognitionResultList
SpeechSynthesis
SpeechSynthesisUtterance
SpeechSynthesisVoice
Telephony
TelephonyCall
TelephonyCallGroup
TextTrack
TextTrackCueList
TextTrackList
TextTrackRegion
TextTrackRegionList
TimeRanges
TextTrackCue
VideoPlaybackQuality
VideoStreamTrack
WaveShaperNode
WebGLQuery
WebGLActiveInfo
WebGLBuffer
WebGLExtensionCompressedTextureATC
WebGLExtensionCompressedTexturePVRTC
WebGLExtensionCompressedTextureS3TC
WebGLExtensionDebugRendererInfo
WebGLExtensionDepthTexture
WebGLExtensionDrawBuffers
WebGLExtensionElementIndexUint
WebGLExtensionInstancedArrays
WebGLExtensionLoseContext
WebGLExtensionStandardDerivatives
WebGLExtensionTextureFilterAnisotropic
WebGLExtensionTextureFloat
WebGLExtensionTextureFloatLinear
WebGLExtensionVertexArray
WebGLFramebuffer
WebGLProgram
WebGLRenderbuffer
WebGLContext
WebGLShader
WebGLShaderPrecisionFormat
WebGLTexture
WebGLUniformLocation
WebGLVertexArray

I did it a bit randomly, so let me know if there's any that you think we shouldn't remove it from.

	Boris Zbarsky [:bzbarsky]
	Comment 1 • 11 years ago

Comment on attachment 809385 [details] [diff] [review]
v1

r=awesomesauce

	Masatoshi Kimura [:emk]
	Comment 2 • 11 years ago

Maybe we can remove QI from TextDecoder and TextEncoder. They have never implemented XPCOM interfaces.

	Kyle Huey (Exited; not receiving bugmail, old account, do not use)
	Comment 3 • 11 years ago

TextDecoder/Encoder have not had QI since I removed the nsISupports inheritance in bug 903772.  Things that don't inherit from nsISupports don't get QI.

	Peter Van der Beken [:peterv] Assignee
	Comment 4 • 11 years ago

So it turns out that I had a bunch of interfaces that wouldn't have a QI anyway because they don't inherit from nsISupports. This removes them and makes the WantsQI static assert that the interface inherits from nsISupports.

	Boris Zbarsky [:bzbarsky]
	Comment 5 • 11 years ago

Comment on attachment 809995 [details] [diff] [review]
Remove some more interfaces v1

Sweet!

	Peter Van der Beken [:peterv] Assignee
	Comment 6 • 11 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/6ea361ff5333

	Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)
	Comment 7 • 11 years ago

https://hg.mozilla.org/mozilla-central/rev/6ea361ff5333