firka/flutter
firka/flutter
5
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

[flutter_tools] fix debug stack traces (#50872)

Browse Source
This commit is contained in:
Jonah Williams 2020-02-18 11:28:02 -08:00 committed by GitHub
parent 57acc68740
commit 67362ce5e0
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 20 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
dev/integration_tests/web/lib/stack_trace.dart
View File

@ -33,9 +33,9 @@ const List<StackFrame> expectedDebugStackFrames = <StackFrame>[
number: -1,
packageScheme: 'package',
package: 'web_integration',
packagePath: 'stack_trace.dart.lib.js',
line: 138,
column: 15,
packagePath: 'stack_trace.dart',
line: 121,
column: 3,
className: '<unknown>',
method: 'baz',
source: '',
@ -44,9 +44,9 @@ const List<StackFrame> expectedDebugStackFrames = <StackFrame>[
number: -1,
packageScheme: 'package',
package: 'web_integration',
packagePath: 'stack_trace.dart.lib.js',
line: 135,
column: 17,
packagePath: 'stack_trace.dart',
line: 116,
column: 3,
className: '<unknown>',
method: 'bar',
source: '',
@ -55,9 +55,9 @@ const List<StackFrame> expectedDebugStackFrames = <StackFrame>[
number: -1,
packageScheme: 'package',
package: 'web_integration',
packagePath: 'stack_trace.dart.lib.js',
line: 132,
column: 17,
packagePath: 'stack_trace.dart',
line: 111,
column: 3,
className: '<unknown>',
method: 'foo',
source: '',

12
packages/flutter_tools/lib/src/web/bootstrap.dart
View File

@ -194,7 +194,17 @@ define("main_module.bootstrap", ["$entrypoint", "dart_sdk"], function(app, dart_
function(url) {
url = url.replace(baseUrl, '/');
var module = window.\$dartLoader.urlToModuleId.get(url);
if (!module) return null;
if (!module) {
if (url.endsWith('dart_sdk.js')) {
module = 'dart_sdk';
} else {
try {
module = '/packages' + url.split('packages')[1].split('.')[0] + '.dart';
} catch (e) {
return null;
}
}
}
return dart.getSourceMap(module);
});
}