Remove dom_deprecated from everywhere but lib/dom and lib/html

Remove dom_deprecated from everywhere but lib/dom and lib/html.
Committed: https://code.google.com/p/dart/source/detail?r=11908

[Emily Fortuna, 2012-09-05 18:43:42]

This removes dom_deprecated from utils/apidoc, which then allows us to remove it
from everywhere else in the repo (with the exception of lib/dom and lib/html). 

The changes to lib/compiler are identical to the ones in this CL:
https://chromiumcodereview.appspot.com/10910013/ that were reverted because of
the apidoc dependency. eub, johnniwinther, you mainly need to look at
utils/apidoc/apidoc.dart, utils/apidoc/html_diff.dart, and
utils/apidoc/html_diff_dump.dart.

Phil, I am including a small editor change in
editor/tools/plugins/com.google.dart.tools.core/src/com/google/dart/tools/core/internal/model/DartLibraryImpl.java.
Let me know if this shouldn't be changed.

[eub, 2012-09-05 21:28:11]

I'm still not an apidoc expert, but it looks good to me, if it works when you
run it.

https://chromiumcodereview.appspot.com/10918063/diff/11005/utils/apidoc/apido...
File utils/apidoc/apidoc.dart (right):

https://chromiumcodereview.appspot.com/10918063/diff/11005/utils/apidoc/apido...
utils/apidoc/apidoc.dart:466: final mdnType = mdn[typeString];
I may be paranoid but I have a worry that mdn[''] could find us some garbage. 
Should I stop worrying?

https://chromiumcodereview.appspot.com/10918063/diff/11005/utils/apidoc/apido...
utils/apidoc/apidoc.dart:483: // If it's an HTML type, try to map it to a DOM
name type so we can find
"DOM type name"?

https://chromiumcodereview.appspot.com/10918063/diff/11005/utils/apidoc/apido...
utils/apidoc/apidoc.dart:504: if (mdnType == null) return null;
Can this happen from split()?

https://chromiumcodereview.appspot.com/10918063/diff/11005/utils/apidoc/html_...
File utils/apidoc/html_diff.dart (right):

https://chromiumcodereview.appspot.com/10918063/diff/11005/utils/apidoc/html_...
utils/apidoc/html_diff.dart:51: final Map<String, Set<MemberMirror>> domToHtml;
Can you note why we use MemberMirrors just in this one place?

https://chromiumcodereview.appspot.com/10918063/diff/11005/utils/apidoc/html_...
utils/apidoc/html_diff.dart:51: final Map<String, Set<MemberMirror>> domToHtml;
BTW, hm, is it true that domToHtml is consumed only by html_diff_dump (whatever
that's for :)?

[Emily Fortuna, 2012-09-05 21:57:26]

ptal. 
Added suggestions.

The output of linking MDN to docs seems to be the same with api.dartlang.org as
in the new stuff. And it runs! Therefore it must be right! Or something. :-]

https://chromiumcodereview.appspot.com/10918063/diff/11005/utils/apidoc/apido...
File utils/apidoc/apidoc.dart (right):

https://chromiumcodereview.appspot.com/10918063/diff/11005/utils/apidoc/apido...
utils/apidoc/apidoc.dart:466: final mdnType = mdn[typeString];
On 2012/09/05 21:28:11, eub wrote:
> I may be paranoid but I have a worry that mdn[''] could find us some garbage. 
> Should I stop worrying?

Good point. I've added an else branch above so that typestring = '' can't reach
here.

https://chromiumcodereview.appspot.com/10918063/diff/11005/utils/apidoc/apido...
utils/apidoc/apidoc.dart:483: // If it's an HTML type, try to map it to a DOM
name type so we can find
On 2012/09/05 21:28:11, eub wrote:
> "DOM type name"?

Done.

https://chromiumcodereview.appspot.com/10918063/diff/11005/utils/apidoc/apido...
utils/apidoc/apidoc.dart:504: if (mdnType == null) return null;
On 2012/09/05 21:28:11, eub wrote:
> Can this happen from split()?

Yes, if we try to look up in the mdn dictionary a name that doesn't actually
exist. Unlikely, but possible.

https://chromiumcodereview.appspot.com/10918063/diff/11005/utils/apidoc/html_...
File utils/apidoc/html_diff.dart (right):

https://chromiumcodereview.appspot.com/10918063/diff/11005/utils/apidoc/html_...
utils/apidoc/html_diff.dart:51: final Map<String, Set<MemberMirror>> domToHtml;
On 2012/09/05 21:28:11, eub wrote:
> BTW, hm, is it true that domToHtml is consumed only by html_diff_dump
(whatever
> that's for :)?

Good catch. This was some ... vestigial code from when we wanted to map between
html -> dom and also back from dom -> html. Getting rid of it.

[Emily Fortuna, 2012-09-05 22:07:40]

+ Keerthi for looking at change in
editor/tools/plugins/com.google.dart.tools.core/src/com/google/dart/tools/core/internal/model/DartLibraryImpl.java

[keertip, 2012-09-05 22:10:22]

lgtm - editor changes look fine

[pquitslund, 2012-09-05 22:20:53]

LGTM.  From my end, looks awesome.

[Johnni Winther, 2012-09-06 07:59:36]

lgtm