Remove more classes and members by making them private.

Remove more classes and members by making them private.

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

[Andrei Mouravski, 2013-01-15 22:50:46]

Here are the remaining classes that need to be removed for now.

Some members are still there for some reason. See FrameSetElement and
MarqueeElement.

[blois, 2013-01-15 23:05:55]

https://codereview.chromium.org/11934004/diff/2001/sdk/lib/html/dart2js/html_...
File sdk/lib/html/dart2js/html_dart2js.dart (right):

https://codereview.chromium.org/11934004/diff/2001/sdk/lib/html/dart2js/html_...
sdk/lib/html/dart2js/html_dart2js.dart:23760: static const
EventStreamProvider<Event> beforeUnloadEvent = const
EventStreamProvider<Event>('beforeunload');
Can you open a bug on this? We just need to have a way to suppress events for
these types.

Or it should be fairly straightforward to do, if you're up to it now.

https://codereview.chromium.org/11934004/diff/2001/sdk/lib/html/dart2js/html_...
sdk/lib/html/dart2js/html_dart2js.dart:24032: static bool get supported =>
Element.isTagSupported('marquee')&& (new Element.tag('marquee') is
MarqueeElement);
Should remove this- it's in systemhtml.py, in the js_support_checks table.

https://codereview.chromium.org/11934004/diff/2001/tools/dom/scripts/htmlrena...
File tools/dom/scripts/htmlrenamer.py (right):

https://codereview.chromium.org/11934004/diff/2001/tools/dom/scripts/htmlrena...
tools/dom/scripts/htmlrenamer.py:24: 'HTMLFontElement' : '_FontElement',
Alphabetize

https://codereview.chromium.org/11934004/diff/2001/tools/dom/scripts/htmlrena...
tools/dom/scripts/htmlrenamer.py:25: 'HTMLMarqueeElement' : '_MarqueeElement',
Should remove the marquee test from element_types_test.dart.

[Andrei Mouravski, 2013-01-15 23:57:59]

PTAL.

https://codereview.chromium.org/11934004/diff/2001/sdk/lib/html/dart2js/html_...
File sdk/lib/html/dart2js/html_dart2js.dart (right):

https://codereview.chromium.org/11934004/diff/2001/sdk/lib/html/dart2js/html_...
sdk/lib/html/dart2js/html_dart2js.dart:23760: static const
EventStreamProvider<Event> beforeUnloadEvent = const
EventStreamProvider<Event>('beforeunload');
On 2013/01/15 23:05:55, blois wrote:
> Can you open a bug on this? We just need to have a way to suppress events for
> these types.
> 
> Or it should be fairly straightforward to do, if you're up to it now.

http://code.google.com/p/dart/issues/detail?id=7921

https://codereview.chromium.org/11934004/diff/2001/sdk/lib/html/dart2js/html_...
sdk/lib/html/dart2js/html_dart2js.dart:24032: static bool get supported =>
Element.isTagSupported('marquee')&& (new Element.tag('marquee') is
MarqueeElement);
On 2013/01/15 23:05:55, blois wrote:
> Should remove this- it's in systemhtml.py, in the js_support_checks table.

Done.

https://codereview.chromium.org/11934004/diff/2001/tools/dom/scripts/htmlrena...
File tools/dom/scripts/htmlrenamer.py (right):

https://codereview.chromium.org/11934004/diff/2001/tools/dom/scripts/htmlrena...
tools/dom/scripts/htmlrenamer.py:24: 'HTMLFontElement' : '_FontElement',
On 2013/01/15 23:05:55, blois wrote:
> Alphabetize

Done.

https://codereview.chromium.org/11934004/diff/2001/tools/dom/scripts/htmlrena...
tools/dom/scripts/htmlrenamer.py:25: 'HTMLMarqueeElement' : '_MarqueeElement',
On 2013/01/15 23:05:55, blois wrote:
> Should remove the marquee test from element_types_test.dart.

Done.

[blois, 2013-01-16 22:51:33]

lgtm