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

Side by Side Diff: pkg/browser/lib/dart.js

Issue 99943006: Remove <script></script> comment in dart.js, which would allow people to inline (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 6 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | pkg/browser/pubspec.yaml » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 (function() { 5 (function() {
6 // Bootstrap support for Dart scripts on the page as this script. 6 // Bootstrap support for Dart scripts on the page as this script.
7 if (navigator.userAgent.indexOf('(Dart)') === -1) { 7 if (navigator.userAgent.indexOf('(Dart)') === -1) {
8 // TODO: 8 // TODO:
9 // - Support in-browser compilation. 9 // - Support in-browser compilation.
10 // - Handle inline Dart scripts. 10 // - Handle inline Dart scripts.
11 11
12 // Fall back to compiled JS. Run through all the scripts and 12 // Fall back to compiled JS. Run through all the scripts and
13 // replace them if they have a type that indicate that they source 13 // replace them if they have a type that indicate that they source
14 // in Dart code. 14 // in Dart code (type="application/dart").
15 //
16 // <script type="application/dart" src="..."></script>
17 //
18 var scripts = document.getElementsByTagName("script"); 15 var scripts = document.getElementsByTagName("script");
19 var length = scripts.length; 16 var length = scripts.length;
20 for (var i = 0; i < length; ++i) { 17 for (var i = 0; i < length; ++i) {
21 if (scripts[i].type == "application/dart") { 18 if (scripts[i].type == "application/dart") {
22 // Remap foo.dart to foo.dart.js. 19 // Remap foo.dart to foo.dart.js.
23 if (scripts[i].src && scripts[i].src != '') { 20 if (scripts[i].src && scripts[i].src != '') {
24 var script = document.createElement('script'); 21 var script = document.createElement('script');
25 script.src = scripts[i].src.replace(/\.dart(?=\?|$)/, '.dart.js'); 22 script.src = scripts[i].src.replace(/\.dart(?=\?|$)/, '.dart.js');
26 var parent = scripts[i].parentNode; 23 var parent = scripts[i].parentNode;
27 // TODO(vsm): Find a solution for issue 8455 that works with more 24 // TODO(vsm): Find a solution for issue 8455 that works with more
28 // than one script. 25 // than one script.
29 document.currentScript = script; 26 document.currentScript = script;
30 parent.replaceChild(script, scripts[i]); 27 parent.replaceChild(script, scripts[i]);
31 } 28 }
32 } 29 }
33 } 30 }
34 } 31 }
35 })(); 32 })();
OLDNEW
« no previous file with comments | « no previous file | pkg/browser/pubspec.yaml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698