Make DOMImplementation::hasFeature() behave according to specification

Make DOMImplementation::hasFeature() behave according to specification

Make DOMImplementation::hasFeature() behave according to the latest specification:
http://dom.spec.whatwg.org/#dom-domimplementation-hasfeature

Specifically, the function now returns true unconditionally for any non-SVG feature.
This behavior is consistent with Firefox.

hasFeature() originally would report whether the user agent claimed to support a
given DOM feature, but experience proved it was not nearly as reliable or granular
as simply checking whether the desired interfaces, attributes, or methods existed.
As such, it should no longer be used, but continues to exist (and simply returns
true) so that old pages don't stop working.

BUG=264683
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=155050

[Stephen Chennney, 2013-07-26 14:37:38]

I'm fine with the SVG portions.

[abarth-chromium, 2013-07-26 17:24:25]

Code change LGTM.  I'm not sure about modifying the SVG conformance test...

https://codereview.chromium.org/20738002/diff/1/LayoutTests/svg/W3C-SVG-1.1-S...
File LayoutTests/svg/W3C-SVG-1.1-SE/types-dom-06-f.svg (left):

https://codereview.chromium.org/20738002/diff/1/LayoutTests/svg/W3C-SVG-1.1-S...
LayoutTests/svg/W3C-SVG-1.1-SE/types-dom-06-f.svg:64: if(i1 !=
"this.is.a.bogus.feature.string")
There are SVG conformance tests.  Should we be changing the tests themselves or
just our expected behavior?

[do-not-use, 2013-07-26 17:49:29]

https://codereview.chromium.org/20738002/diff/1/LayoutTests/svg/W3C-SVG-1.1-S...
File LayoutTests/svg/W3C-SVG-1.1-SE/types-dom-06-f.svg (left):

https://codereview.chromium.org/20738002/diff/1/LayoutTests/svg/W3C-SVG-1.1-S...
LayoutTests/svg/W3C-SVG-1.1-SE/types-dom-06-f.svg:64: if(i1 !=
"this.is.a.bogus.feature.string")
On 2013/07/26 17:24:25, abarth wrote:
> There are SVG conformance tests.  Should we be changing the tests themselves
or
> just our expected behavior?

Sorry, I updated many SVG tests and I missed that this one was a conformance
test. I rebaselined it instead in the new patch set.

[commit-bot: I haz the power, 2013-07-27 09:21:35]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ch.dumez@sisa.samsung.com/20738002/18001

[commit-bot: I haz the power, 2013-07-27 12:23:31]

Change committed as 155050

[arv (Not doing code reviews), 2013-07-30 20:08:25]

LGTM