Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1538)

Unified Diff: tools/dom/scripts/htmlrenamer.py

Issue 11934004: Remove more classes and members by making them private. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Removing some stuff. Created 7 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: tools/dom/scripts/htmlrenamer.py
diff --git a/tools/dom/scripts/htmlrenamer.py b/tools/dom/scripts/htmlrenamer.py
index 6f029b26a3319b010885e251287aeaceac0f9269..fac00e8f0838bd56003ddeeb8c4e96e79d1a8725 100644
--- a/tools/dom/scripts/htmlrenamer.py
+++ b/tools/dom/scripts/htmlrenamer.py
@@ -16,7 +16,13 @@ html_interface_renames = monitored.Dict('htmlrenamer.html_interface_renames', {
'DOMURL': 'Url',
'DOMWindow': 'Window',
'HTMLDocument' : 'HtmlDocument',
+ 'HTMLAppletElement' : '_AppletElement',
+ 'HTMLBaseFontElement' : '_BaseFontElement',
'HTMLDirectoryElement' : '_DirectoryElement',
+ 'HTMLFrameElement' : '_FrameElement',
+ 'HTMLFrameSetElement' : '_FrameSetElement',
+ 'HTMLFontElement' : '_FontElement',
blois 2013/01/15 23:05:55 Alphabetize
Andrei Mouravski 2013/01/15 23:57:59 Done.
+ 'HTMLMarqueeElement' : '_MarqueeElement',
blois 2013/01/15 23:05:55 Should remove the marquee test from element_types_
Andrei Mouravski 2013/01/15 23:57:59 Done.
'IDBAny': '_Any', # Suppressed, but needs to exist for Dartium.
'IDBFactory': 'IdbFactory', # Manual to avoid name conflicts.
'SVGDocument': 'SvgDocument', # Manual to avoid name conflicts.
@@ -214,7 +220,6 @@ _removed_html_members = monitored.Set('htmlrenamer._removed_html_members', [
'Document.get:anchors',
'Document.get:characterSet',
'Document.get:compatMode',
- 'Document.get:compatMode',
'Document.get:defaultCharset',
'Document.get:doctype',
'Document.get:documentURI',
@@ -264,19 +269,20 @@ _removed_html_members = monitored.Set('htmlrenamer._removed_html_members', [
'HTMLAnchorElement.rev',
'HTMLAnchorElement.shape',
'HTMLAnchorElement.text',
+ 'HTMLAppletElement.*',
'HTMLAreaElement.noHref',
'HTMLBRElement.clear',
+ 'HTMLBaseFontElement.*',
'HTMLBodyElement.aLink',
'HTMLBodyElement.background',
'HTMLBodyElement.bgColor',
- 'HTMLBodyElement.bgColor',
'HTMLBodyElement.link',
'HTMLBodyElement.text',
- 'HTMLBodyElement.text',
'HTMLBodyElement.vLink',
'HTMLDListElement.compact',
'HTMLDirectoryElement.*',
'HTMLDivElement.align',
+ 'HTMLFontElement.*',
'HTMLFormElement.get:elements',
'HTMLFrameElement.*',
'HTMLFrameSetElement.*',
@@ -288,7 +294,6 @@ _removed_html_members = monitored.Set('htmlrenamer._removed_html_members', [
'HTMLHeadingElement.align',
'HTMLHtmlElement.manifest',
'HTMLHtmlElement.version',
- 'HTMLHtmlElement.version',
'HTMLIFrameElement.align',
'HTMLIFrameElement.frameBorder',
'HTMLIFrameElement.longDesc',
@@ -305,6 +310,7 @@ _removed_html_members = monitored.Set('htmlrenamer._removed_html_members', [
'HTMLLinkElement.charset',
'HTMLLinkElement.rev',
'HTMLLinkElement.target',
+ 'HTMLMarqueeElement.*',
'HTMLMenuElement.compact',
'HTMLMetaElement.scheme',
'HTMLOListElement.compact',
« sdk/lib/html/dart2js/html_dart2js.dart ('K') | « sdk/lib/html/dartium/html_dartium.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698