Move native and js interop properties from the element model to the JS backend

Move native and js interop properties from the element model to the JS backend

Main changes:
  Moved native/jsinterop members from
    elements/elements.dart, (interface)
    elements/modelx.dart, (implementation)
    native/native.dart (implementation)
  to
    common/backend_api.dart (interface)
    js_backend/backend.dart (implementation)

  Removed LibraryElement.canUseNative and added BackendApi.canUseNative to
    common/backend_api.dart
  to encapsulate the maybeEnableNative function in
    native/native.dart

  Added ScannerOptions to
    parser/element_listener.dart
  as a required argument of ElementListener
  and added Parsing.getScannerOptionsFor to
    common/resolution.dart (interface)
    compiler.dart (implementation)
  to allow the `native` pseudo keyword in
    native/scanner.dart

The remaining changes are side-effects of the changes above.

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

Committed: https://github.com/dart-lang/sdk/commit/e38285c424fd04d8f408e534eee6ce2cea27da7f

[Johnni Winther, 2015-10-16 10:15:58]

johnniwinther@google.com changed reviewers:
+ jacobr@google.com, sra@google.com

[Jacob, 2015-10-16 17:02:19]

thanks for doing this refactor.
lgtm on the isJsInterop portion.
Fyi: the tests for the isJsInterop part are all in tests/html

https://codereview.chromium.org/1408043002/diff/20001/pkg/compiler/lib/src/js...
File pkg/compiler/lib/src/js_emitter/runtime_type_generator.dart (right):

https://codereview.chromium.org/1408043002/diff/20001/pkg/compiler/lib/src/js...
pkg/compiler/lib/src/js_emitter/runtime_type_generator.dart:71: if
(backend.isJsInterop(classElement) ||
nt your fault but isJsInterop imples isNative so remove the check for
isJsInterop.

[Johnni Winther, 2015-10-21 07:56:10]

johnniwinther@google.com changed reviewers:
+ sigurdm@google.com

[Johnni Winther, 2015-10-21 07:56:11]

@sigurdm: Can you take a look at the LibraryElement.canUseNative and
ScannerOptions part?

[sigurdm, 2015-10-21 10:28:28]

LGTM

https://codereview.chromium.org/1408043002/diff/20001/pkg/compiler/lib/src/js...
File pkg/compiler/lib/src/js_backend/backend.dart (right):

https://codereview.chromium.org/1408043002/diff/20001/pkg/compiler/lib/src/js...
pkg/compiler/lib/src/js_backend/backend.dart:808: /// Tag infor for native
JavaScript classes names. See
Remove "infor"?

https://codereview.chromium.org/1408043002/diff/20001/pkg/compiler/lib/src/js...
pkg/compiler/lib/src/js_backend/backend.dart:812: /// Returns `true` if
[element] is explicitly mark as part of JsInterop.
mark -> marked

https://codereview.chromium.org/1408043002/diff/20001/pkg/compiler/lib/src/js...
pkg/compiler/lib/src/js_backend/backend.dart:876: /// is either given through a
native annotation or a
or a...?? the suspense is killing me!

https://codereview.chromium.org/1408043002/diff/20001/pkg/compiler/lib/src/pa...
File pkg/compiler/lib/src/parser/element_listener.dart (right):

https://codereview.chromium.org/1408043002/diff/20001/pkg/compiler/lib/src/pa...
pkg/compiler/lib/src/parser/element_listener.dart:57: class ScannerOptions {
Could use a doc-string

[Johnni Winther, 2015-10-22 09:06:03]

https://codereview.chromium.org/1408043002/diff/20001/pkg/compiler/lib/src/js...
File pkg/compiler/lib/src/js_backend/backend.dart (right):

https://codereview.chromium.org/1408043002/diff/20001/pkg/compiler/lib/src/js...
pkg/compiler/lib/src/js_backend/backend.dart:808: /// Tag infor for native
JavaScript classes names. See
On 2015/10/21 10:28:28, sigurdm wrote:
> Remove "infor"?

Done.

https://codereview.chromium.org/1408043002/diff/20001/pkg/compiler/lib/src/js...
pkg/compiler/lib/src/js_backend/backend.dart:812: /// Returns `true` if
[element] is explicitly mark as part of JsInterop.
On 2015/10/21 10:28:28, sigurdm wrote:
> mark -> marked

Done.

https://codereview.chromium.org/1408043002/diff/20001/pkg/compiler/lib/src/js...
pkg/compiler/lib/src/js_backend/backend.dart:876: /// is either given through a
native annotation or a
On 2015/10/21 10:28:28, sigurdm wrote:
> or a...?? the suspense is killing me!

Cliffhanger... ;)

https://codereview.chromium.org/1408043002/diff/20001/pkg/compiler/lib/src/js...
File pkg/compiler/lib/src/js_emitter/runtime_type_generator.dart (right):

https://codereview.chromium.org/1408043002/diff/20001/pkg/compiler/lib/src/js...
pkg/compiler/lib/src/js_emitter/runtime_type_generator.dart:71: if
(backend.isJsInterop(classElement) ||
On 2015/10/16 17:02:18, Jacob wrote:
> nt your fault but isJsInterop imples isNative so remove the check for
> isJsInterop.

Done.

https://codereview.chromium.org/1408043002/diff/20001/pkg/compiler/lib/src/pa...
File pkg/compiler/lib/src/parser/element_listener.dart (right):

https://codereview.chromium.org/1408043002/diff/20001/pkg/compiler/lib/src/pa...
pkg/compiler/lib/src/parser/element_listener.dart:57: class ScannerOptions {
On 2015/10/21 10:28:28, sigurdm wrote:
> Could use a doc-string

Done.

[Johnni Winther, 2015-10-22 09:42:16]

Committed patchset #3 (id:40001) manually as
e38285c424fd04d8f408e534eee6ce2cea27da7f (presubmit successful).