Cleanup NodeSelectors.

Cleanup NodeSelectors.

Currently, query and queryAll methods generation in dart:html is somehow affected by a combination of the following factors:
1) Document, DocumentFragment, and Element are declared to inherit from NodeSelector interface in dart.idl
2) NodeSelector methods for some of these interfaces are present in_private_html_members, but not all, resulting in generation of $dom_* variants
3) querySelector is renamed to query via _html_library_renames for all interfaces except for Document
4) Some of these methods and their private variants have custom implementations in impl_*.darttemplate's

This patch aims to simplify that logic.

R=jacobr@google.com,sra@google.com

Committed: https://code.google.com/p/dart/source/detail?r=8240

[Anton Muhin, 2012-05-31 18:20:44]

STV!

http://codereview.chromium.org/10447091/diff/1/lib/dom/idl/dart/dart.idl
File lib/dom/idl/dart/dart.idl (left):

http://codereview.chromium.org/10447091/diff/1/lib/dom/idl/dart/dart.idl#oldc...
lib/dom/idl/dart/dart.idl:37: // Force NodeSelector. WebKit defines these
operations directly.
why remove it from here?

http://codereview.chromium.org/10447091/diff/1/lib/dom/idl/dart/dart.idl#oldc...
lib/dom/idl/dart/dart.idl:44: Element implements NodeSelector;
I can see that Element explicitly says it inherits from NodeSelector.  Where
Document and DocumentFragment are marked as inheriting from NodeSelector?

http://codereview.chromium.org/10447091/diff/1/lib/dom/templates/html/interfa...
File lib/dom/templates/html/interface/interface_Element.darttemplate (right):

http://codereview.chromium.org/10447091/diff/1/lib/dom/templates/html/interfa...
lib/dom/templates/html/interface/interface_Element.darttemplate:40: interface
NodeSelector {
I'd rather keep this interface in dart.idl.  Ditto for other interfaces in this
file.

[podivilov, 2012-06-04 09:34:36]

http://codereview.chromium.org/10447091/diff/1/lib/dom/idl/dart/dart.idl
File lib/dom/idl/dart/dart.idl (left):

http://codereview.chromium.org/10447091/diff/1/lib/dom/idl/dart/dart.idl#oldc...
lib/dom/idl/dart/dart.idl:37: // Force NodeSelector. WebKit defines these
operations directly.
On 2012/05/31 18:20:44, antonmuhin wrote:
> why remove it from here?

Reverted.

http://codereview.chromium.org/10447091/diff/1/lib/dom/idl/dart/dart.idl#oldc...
lib/dom/idl/dart/dart.idl:44: Element implements NodeSelector;
On 2012/05/31 18:20:44, antonmuhin wrote:
> I can see that Element explicitly says it inherits from NodeSelector.  Where
> Document and DocumentFragment are marked as inheriting from NodeSelector?

Document and DocumentFragment interfaces extend NodeSelector. But I think it's
better to revert this change as well.

http://codereview.chromium.org/10447091/diff/1/lib/dom/templates/html/interfa...
File lib/dom/templates/html/interface/interface_Element.darttemplate (right):

http://codereview.chromium.org/10447091/diff/1/lib/dom/templates/html/interfa...
lib/dom/templates/html/interface/interface_Element.darttemplate:40: interface
NodeSelector {
On 2012/05/31 18:20:44, antonmuhin wrote:
> I'd rather keep this interface in dart.idl.  Ditto for other interfaces in
this
> file.

Done.