mirror of
https://github.com/dart-lang/sdk
synced 2024-10-04 16:04:53 +00:00
Revert "[dart2js] Migrate dart2js_tools to null safety"
This reverts commit 4b1de45034
.
Reason for revert: Null safety error appears when test fail:
error: Null check operator used on a null value
Original change's description:
> [dart2js] Migrate dart2js_tools to null safety
>
> Change-Id: If93f66f4da2dafb50f58434eb112b1f8af6648c7
> Issue: https://github.com/dart-lang/sdk/issues/46617
> Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/208363
> Commit-Queue: Nicholas Shahan <nshahan@google.com>
> Reviewed-by: Stephen Adams <sra@google.com>
# Not skipping CQ checks because original CL landed > 1 day ago.
Issue: https://github.com/dart-lang/sdk/issues/46617
Change-Id: I7ef7d95a6715d667698e0dfcb38375fd79b4cc73
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/210540
Reviewed-by: Stephen Adams <sra@google.com>
Commit-Queue: Nicholas Shahan <nshahan@google.com>
This commit is contained in:
parent
37ea0f8cb4
commit
9ae8713216
|
@ -220,7 +220,7 @@
|
|||
"name": "dart2js_tools",
|
||||
"rootUri": "../pkg/dart2js_tools",
|
||||
"packageUri": "lib/",
|
||||
"languageVersion": "2.12"
|
||||
"languageVersion": "2.3"
|
||||
},
|
||||
{
|
||||
"name": "dart2native",
|
||||
|
@ -812,4 +812,4 @@
|
|||
"languageVersion": "2.12"
|
||||
}
|
||||
]
|
||||
}
|
||||
}
|
||||
|
|
|
@ -47,24 +47,24 @@ main(List<String> args) {
|
|||
var sb = new StringBuffer();
|
||||
try {
|
||||
String obfuscatedTrace = new File(args[0]).readAsStringSync();
|
||||
String? error = extractErrorMessage(obfuscatedTrace);
|
||||
String error = extractErrorMessage(obfuscatedTrace);
|
||||
var provider = new CachingFileProvider(logger: Logger());
|
||||
StackDeobfuscationResult result =
|
||||
deobfuscateStack(obfuscatedTrace, provider);
|
||||
Frame firstFrame = result.original.frames.first;
|
||||
String? translatedError =
|
||||
String translatedError =
|
||||
translate(error, provider.mappingFor(firstFrame.uri));
|
||||
if (translatedError == null) translatedError = '<no error message found>';
|
||||
printPadded(translatedError, error, sb);
|
||||
int longest =
|
||||
result.deobfuscated.frames.fold(0, (m, f) => max(f.member!.length, m));
|
||||
result.deobfuscated.frames.fold(0, (m, f) => max(f.member.length, m));
|
||||
for (var originalFrame in result.original.frames) {
|
||||
var deobfuscatedFrames = result.frameMap[originalFrame];
|
||||
if (deobfuscatedFrames == null) {
|
||||
printPadded('no mapping', '${originalFrame.location}', sb);
|
||||
} else {
|
||||
for (var frame in deobfuscatedFrames) {
|
||||
printPadded('${frame.member!.padRight(longest)} ${frame.location}',
|
||||
printPadded('${frame.member.padRight(longest)} ${frame.location}',
|
||||
'${originalFrame.location}', sb);
|
||||
}
|
||||
}
|
||||
|
@ -77,7 +77,7 @@ main(List<String> args) {
|
|||
final green = stdout.hasTerminal ? '\x1b[32m' : '';
|
||||
final none = stdout.hasTerminal ? '\x1b[0m' : '';
|
||||
|
||||
printPadded(String mapping, String? original, sb) {
|
||||
printPadded(String mapping, String original, sb) {
|
||||
var len = mapping.length;
|
||||
var s = mapping.indexOf('\n');
|
||||
if (s >= 0) len -= s + 1;
|
||||
|
|
|
@ -2,7 +2,6 @@ import 'dart:io';
|
|||
import 'dart:convert';
|
||||
import 'package:source_maps/source_maps.dart';
|
||||
import 'package:dart2js_tools/src/dart2js_mapping.dart';
|
||||
import 'package:dart2js_tools/src/util.dart';
|
||||
|
||||
main(List<String> args) {
|
||||
if (args.length < 2) {
|
||||
|
@ -17,7 +16,7 @@ main(List<String> args) {
|
|||
exit(1);
|
||||
}
|
||||
var json = jsonDecode(sourcemapFile.readAsStringSync());
|
||||
Dart2jsMapping mapping = Dart2jsMapping(parseSingleMapping(json), json);
|
||||
Dart2jsMapping mapping = Dart2jsMapping(parseJson(json), json);
|
||||
var global = mapping.globalNames[name];
|
||||
if (global != null) print('$name => $global (a global name)');
|
||||
var instance = mapping.instanceNames[name];
|
||||
|
|
|
@ -26,7 +26,7 @@ main(List<String> args) {
|
|||
var offsets = frames.keys.toList()..sort();
|
||||
var sb = new StringBuffer();
|
||||
int depth = 0;
|
||||
int? lastFunctionStart = null;
|
||||
int lastFunctionStart = null;
|
||||
for (var offset in offsets) {
|
||||
int functionStart = nextFunctionStart(starts, offset, lastFunctionStart);
|
||||
if (lastFunctionStart == null || functionStart > lastFunctionStart) {
|
||||
|
@ -43,7 +43,7 @@ main(List<String> args) {
|
|||
var pad = ' ' * offsetPrefix.length;
|
||||
sb.write(offsetPrefix);
|
||||
bool first = true;
|
||||
for (var frame in frames[offset]!) {
|
||||
for (var frame in frames[offset]) {
|
||||
if (!first) sb.write('$pad');
|
||||
sb.write(' $frame\n');
|
||||
first = false;
|
||||
|
@ -75,7 +75,7 @@ List<int> functionStarts(String sources) {
|
|||
return result;
|
||||
}
|
||||
|
||||
int nextFunctionStart(List<int> starts, int offset, int? last) {
|
||||
int nextFunctionStart(List<int> starts, int offset, int last) {
|
||||
int j = last ?? 0;
|
||||
for (; j < starts.length && starts[j] <= offset; j++);
|
||||
return j - 1;
|
||||
|
|
|
@ -27,7 +27,7 @@ import 'package:dart2js_tools/src/trace_decoder.dart';
|
|||
/// `//# sourceMappingURL=` line at the end, and load the corresponding
|
||||
/// source-map file.
|
||||
String deobfuscateStackTrace(String obfuscatedTrace) {
|
||||
String? error = extractErrorMessage(obfuscatedTrace);
|
||||
String error = extractErrorMessage(obfuscatedTrace);
|
||||
var provider = CachingFileProvider();
|
||||
StackDeobfuscationResult result = deobfuscateStack(obfuscatedTrace, provider);
|
||||
Frame firstFrame = result.original.frames.first;
|
||||
|
@ -38,13 +38,13 @@ String deobfuscateStackTrace(String obfuscatedTrace) {
|
|||
|
||||
var sb = StringBuffer();
|
||||
sb.writeln(translatedError);
|
||||
maxMemberLengthHelper(int m, Frame f) => max(f.member!.length, m);
|
||||
maxMemberLengthHelper(int m, Frame f) => max(f.member.length, m);
|
||||
int longest = result.deobfuscated.frames.fold(0, maxMemberLengthHelper);
|
||||
longest = result.original.frames.fold(longest, maxMemberLengthHelper);
|
||||
for (var originalFrame in result.original.frames) {
|
||||
var deobfuscatedFrames = result.frameMap[originalFrame];
|
||||
if (deobfuscatedFrames == null) {
|
||||
var name = originalFrame.member!;
|
||||
var name = originalFrame.member;
|
||||
sb.writeln(' at ${name.padRight(longest)} ${originalFrame.location}');
|
||||
} else {
|
||||
for (var frame in deobfuscatedFrames) {
|
||||
|
@ -53,7 +53,7 @@ String deobfuscateStackTrace(String obfuscatedTrace) {
|
|||
// client, we can start encoding the function name and remove this
|
||||
// workaround.
|
||||
if (name == '<unknown>') name = originalFrame.member;
|
||||
sb.writeln(' at ${name!.padRight(longest)} ${frame.location}');
|
||||
sb.writeln(' at ${name.padRight(longest)} ${frame.location}');
|
||||
}
|
||||
}
|
||||
}
|
||||
|
|
|
@ -23,9 +23,15 @@ class Dart2jsMapping {
|
|||
final Map<String, String> globalNames = {};
|
||||
final Map<String, String> instanceNames = {};
|
||||
final Map<int, List<FrameEntry>> frames = {};
|
||||
late final List<int>? frameIndex = frames.keys.toList()..sort();
|
||||
List<int> _frameIndex;
|
||||
List<int> get frameIndex {
|
||||
if (_frameIndex == null) {
|
||||
_frameIndex = frames.keys.toList()..sort();
|
||||
}
|
||||
return _frameIndex;
|
||||
}
|
||||
|
||||
Dart2jsMapping(this.sourceMap, Map json, {Logger? logger}) {
|
||||
Dart2jsMapping(this.sourceMap, Map json, {Logger logger}) {
|
||||
var extensions = json['x_org_dartlang_dart2js'];
|
||||
if (extensions == null) return;
|
||||
var minifiedNames = extensions['minified_names'];
|
||||
|
@ -35,20 +41,20 @@ class Dart2jsMapping {
|
|||
_extractMinifedNames(
|
||||
minifiedNames['instance'], sourceMap, instanceNames, logger);
|
||||
}
|
||||
String? jsonFrames = extensions['frames'];
|
||||
String jsonFrames = extensions['frames'];
|
||||
if (jsonFrames != null) {
|
||||
new _FrameDecoder(jsonFrames).parseFrames(frames, sourceMap);
|
||||
}
|
||||
}
|
||||
|
||||
Dart2jsMapping.json(Map json) : this(parseSingleMapping(json), json);
|
||||
Dart2jsMapping.json(Map json) : this(parseJson(json), json);
|
||||
}
|
||||
|
||||
class FrameEntry {
|
||||
final String? callUri;
|
||||
final int? callLine;
|
||||
final int? callColumn;
|
||||
final String? inlinedMethodName;
|
||||
final String callUri;
|
||||
final int callLine;
|
||||
final int callColumn;
|
||||
final String inlinedMethodName;
|
||||
final bool isEmpty;
|
||||
FrameEntry.push(
|
||||
this.callUri, this.callLine, this.callColumn, this.inlinedMethodName)
|
||||
|
@ -70,7 +76,7 @@ class FrameEntry {
|
|||
}
|
||||
|
||||
const _marker = "\n//# sourceMappingURL=";
|
||||
Dart2jsMapping? parseMappingFor(Uri uri, {Logger? logger}) {
|
||||
Dart2jsMapping parseMappingFor(Uri uri, {Logger logger}) {
|
||||
var file = new File.fromUri(uri);
|
||||
if (!file.existsSync()) {
|
||||
logger?.log('Error: no such file: $uri');
|
||||
|
@ -94,7 +100,7 @@ Dart2jsMapping? parseMappingFor(Uri uri, {Logger? logger}) {
|
|||
return null;
|
||||
}
|
||||
var json = jsonDecode(sourcemapFile.readAsStringSync());
|
||||
return new Dart2jsMapping(parseSingleMapping(json), json, logger: logger);
|
||||
return new Dart2jsMapping(parseJson(json), json, logger: logger);
|
||||
}
|
||||
|
||||
class _FrameDecoder implements Iterator<String> {
|
||||
|
@ -106,9 +112,8 @@ class _FrameDecoder implements Iterator<String> {
|
|||
// Iterator API is used by decodeVlq to consume VLQ entries.
|
||||
bool moveNext() => ++index < _length;
|
||||
|
||||
String get current => (index >= 0 && index < _length)
|
||||
? _internal[index]
|
||||
: throw StateError('No current value available.');
|
||||
String get current =>
|
||||
(index >= 0 && index < _length) ? _internal[index] : null;
|
||||
|
||||
bool get hasTokens => index < _length - 1 && _length > 0;
|
||||
|
||||
|
@ -146,7 +151,7 @@ class _FrameDecoder implements Iterator<String> {
|
|||
}
|
||||
|
||||
_extractMinifedNames(String encodedInput, SingleMapping sourceMap,
|
||||
Map<String, String> minifiedNames, Logger? logger) {
|
||||
Map<String, String> minifiedNames, Logger logger) {
|
||||
if (encodedInput.isEmpty) return;
|
||||
List<String> input = encodedInput.split(',');
|
||||
if (input.length % 2 != 0) {
|
||||
|
@ -154,7 +159,7 @@ _extractMinifedNames(String encodedInput, SingleMapping sourceMap,
|
|||
}
|
||||
for (int i = 0; i < input.length; i += 2) {
|
||||
String minifiedName = input[i];
|
||||
int id = int.tryParse(input[i + 1])!;
|
||||
int id = int.tryParse(input[i + 1]);
|
||||
minifiedNames[minifiedName] = sourceMap.names[id];
|
||||
}
|
||||
}
|
||||
|
|
|
@ -9,8 +9,8 @@ import 'package:source_maps/source_maps.dart';
|
|||
import 'dart2js_mapping.dart';
|
||||
import 'trace.dart';
|
||||
|
||||
String? translate(String? error, Dart2jsMapping mapping,
|
||||
[StackTraceLine? line, TargetEntry? entry]) {
|
||||
String translate(String error, Dart2jsMapping mapping,
|
||||
[StackTraceLine line, TargetEntry entry]) {
|
||||
for (var decoder in _errorMapDecoders) {
|
||||
var result = decoder.decode(error, mapping, line, entry);
|
||||
// More than one decoder might be applied on a single error message. This
|
||||
|
@ -30,10 +30,10 @@ abstract class ErrorMapDecoder {
|
|||
/// Decode [error] that was reported in [line] and has a corresponding [entry]
|
||||
/// in the source-map file. The provided [mapping] includes additional
|
||||
/// minification data that may be used to decode the error message.
|
||||
String? decode(String? error, Dart2jsMapping mapping, StackTraceLine? line,
|
||||
TargetEntry? entry) {
|
||||
String decode(String error, Dart2jsMapping mapping, StackTraceLine line,
|
||||
TargetEntry entry) {
|
||||
if (error == null) return null;
|
||||
Match? lastMatch = null;
|
||||
Match lastMatch = null;
|
||||
var result = new StringBuffer();
|
||||
for (var match in _matcher.allMatches(error)) {
|
||||
var decodedMatch = _decodeInternal(match, mapping, line, entry);
|
||||
|
@ -49,8 +49,8 @@ abstract class ErrorMapDecoder {
|
|||
return '$result';
|
||||
}
|
||||
|
||||
String? _decodeInternal(Match match, Dart2jsMapping mapping,
|
||||
StackTraceLine? line, TargetEntry? entry);
|
||||
String _decodeInternal(Match match, Dart2jsMapping mapping,
|
||||
StackTraceLine line, TargetEntry entry);
|
||||
}
|
||||
|
||||
typedef String ErrorDecoder(Match match, Dart2jsMapping mapping,
|
||||
|
@ -59,28 +59,28 @@ typedef String ErrorDecoder(Match match, Dart2jsMapping mapping,
|
|||
class MinifiedNameDecoder extends ErrorMapDecoder {
|
||||
final RegExp _matcher = new RegExp("minified:([a-zA-Z0-9_\$]*)");
|
||||
|
||||
String? _decodeInternal(Match match, Dart2jsMapping mapping,
|
||||
StackTraceLine? line, TargetEntry? entry) {
|
||||
String _decodeInternal(Match match, Dart2jsMapping mapping,
|
||||
StackTraceLine line, TargetEntry entry) {
|
||||
var minifiedName = match.group(1);
|
||||
return mapping.globalNames[minifiedName!];
|
||||
return mapping.globalNames[minifiedName];
|
||||
}
|
||||
}
|
||||
|
||||
class CannotReadPropertyDecoder extends ErrorMapDecoder {
|
||||
final RegExp _matcher = new RegExp("Cannot read property '([^']*)' of");
|
||||
|
||||
String? _decodeInternal(Match match, Dart2jsMapping mapping,
|
||||
StackTraceLine? line, TargetEntry? entry) {
|
||||
String _decodeInternal(Match match, Dart2jsMapping mapping,
|
||||
StackTraceLine line, TargetEntry entry) {
|
||||
var minifiedName = match.group(1);
|
||||
var name = mapping.instanceNames[minifiedName!];
|
||||
var name = mapping.instanceNames[minifiedName];
|
||||
if (name == null) return null;
|
||||
return "Cannot read property '$name' of";
|
||||
}
|
||||
}
|
||||
|
||||
abstract class NoSuchMethodDecoderBase extends ErrorMapDecoder {
|
||||
String? _translateMinifiedName(Dart2jsMapping mapping, String? minifiedName) {
|
||||
var name = mapping.instanceNames[minifiedName!];
|
||||
String _translateMinifiedName(Dart2jsMapping mapping, String minifiedName) {
|
||||
var name = mapping.instanceNames[minifiedName];
|
||||
if (name != null) return "'$name'";
|
||||
if (minifiedName.startsWith(new RegExp(r'(call)?\$[0-9]'))) {
|
||||
int first$ = minifiedName.indexOf(r'$');
|
||||
|
@ -89,15 +89,15 @@ abstract class NoSuchMethodDecoderBase extends ErrorMapDecoder {
|
|||
return null;
|
||||
}
|
||||
|
||||
String? _expandCallSignature(String callSignature) {
|
||||
String _expandCallSignature(String callSignature) {
|
||||
// Minified names are one of these forms:
|
||||
// $0 // positional arguments only
|
||||
// $1$2 // type parameters and positional arguments
|
||||
// $3$name // positional and named arguments
|
||||
// $1$3$name // type parameters and positional and named args
|
||||
var signature = callSignature.split(r'$');
|
||||
int? typeArgs = null;
|
||||
int? totalArgs = null;
|
||||
var typeArgs = null;
|
||||
var totalArgs = null;
|
||||
var namedArgs = <String>[];
|
||||
for (var arg in signature) {
|
||||
if (arg == "") continue;
|
||||
|
@ -115,7 +115,6 @@ abstract class NoSuchMethodDecoderBase extends ErrorMapDecoder {
|
|||
namedArgs.add(arg);
|
||||
}
|
||||
}
|
||||
if (totalArgs == null) return null;
|
||||
var sb = new StringBuffer();
|
||||
sb.write("'call'");
|
||||
sb.write(" (with ");
|
||||
|
@ -137,8 +136,8 @@ class NoSuchMethodDecoder1 extends NoSuchMethodDecoderBase {
|
|||
final RegExp _matcher = new RegExp(
|
||||
"NoSuchMethodError: method not found: '([^']*)'( on [^\\(]*)? \\(.*\\)");
|
||||
|
||||
String? _decodeInternal(Match match, Dart2jsMapping mapping,
|
||||
StackTraceLine? line, TargetEntry? entry) {
|
||||
String _decodeInternal(Match match, Dart2jsMapping mapping,
|
||||
StackTraceLine line, TargetEntry entry) {
|
||||
var minifiedName = match.group(1);
|
||||
var suffix = match.group(2) ?? '';
|
||||
var name = _translateMinifiedName(mapping, minifiedName);
|
||||
|
@ -151,8 +150,8 @@ class NoSuchMethodDecoder2 extends NoSuchMethodDecoderBase {
|
|||
final RegExp _matcher =
|
||||
new RegExp("NoSuchMethodError: method not found: '([^']*)'");
|
||||
|
||||
String? _decodeInternal(Match match, Dart2jsMapping mapping,
|
||||
StackTraceLine? line, TargetEntry? entry) {
|
||||
String _decodeInternal(Match match, Dart2jsMapping mapping,
|
||||
StackTraceLine line, TargetEntry entry) {
|
||||
var minifiedName = match.group(1);
|
||||
var name = _translateMinifiedName(mapping, minifiedName);
|
||||
if (name == null) return null;
|
||||
|
@ -163,10 +162,10 @@ class NoSuchMethodDecoder2 extends NoSuchMethodDecoderBase {
|
|||
class UnhandledNotAFunctionError extends ErrorMapDecoder {
|
||||
final RegExp _matcher = new RegExp("Error: ([^']*) is not a function");
|
||||
|
||||
String? _decodeInternal(Match match, Dart2jsMapping mapping,
|
||||
StackTraceLine? line, TargetEntry? entry) {
|
||||
String _decodeInternal(Match match, Dart2jsMapping mapping,
|
||||
StackTraceLine line, TargetEntry entry) {
|
||||
var minifiedName = match.group(1);
|
||||
var name = mapping.instanceNames[minifiedName!];
|
||||
var name = mapping.instanceNames[minifiedName];
|
||||
if (name == null) return null;
|
||||
return "Error: $name is not a function";
|
||||
}
|
||||
|
|
|
@ -10,7 +10,7 @@ import 'util.dart' show FileProvider;
|
|||
|
||||
/// Search backwards in [sources] for a function declaration that includes the
|
||||
/// [start] offset.
|
||||
TargetEntry? findEnclosingFunction(FileProvider provider, Uri uri, int start) {
|
||||
TargetEntry findEnclosingFunction(FileProvider provider, Uri uri, int start) {
|
||||
String sources = provider.sourcesFor(uri);
|
||||
if (sources == null) return null;
|
||||
SourceFile file = provider.fileFor(uri);
|
||||
|
@ -22,7 +22,7 @@ TargetEntry? findEnclosingFunction(FileProvider provider, Uri uri, int start) {
|
|||
var line = file.getLine(index);
|
||||
var lineEntry = findLine(mapping, line);
|
||||
var column = file.getColumn(index);
|
||||
TargetEntry? result = findColumn(line, column, lineEntry);
|
||||
TargetEntry result = findColumn(line, column, lineEntry);
|
||||
// If the name entry doesn't start exactly at the column corresponding to
|
||||
// `index`, we must be in the middle of a string or code that uses the word
|
||||
// "function", but that doesn't have a corresponding mapping. In those
|
||||
|
@ -61,7 +61,7 @@ int nextDeclarationCandidate(String sources, int start) {
|
|||
/// number is lower or equal to [line].
|
||||
///
|
||||
/// Copied from [SingleMapping._findLine].
|
||||
TargetLineEntry? findLine(SingleMapping sourceMap, int line) {
|
||||
TargetLineEntry findLine(SingleMapping sourceMap, int line) {
|
||||
int index = binarySearch(sourceMap.lines, (e) => e.line > line);
|
||||
return (index <= 0) ? null : sourceMap.lines[index - 1];
|
||||
}
|
||||
|
@ -73,7 +73,7 @@ TargetLineEntry? findLine(SingleMapping sourceMap, int line) {
|
|||
/// the very last entry on that line.
|
||||
///
|
||||
/// Copied from [SingleMapping._findColumn].
|
||||
TargetEntry? findColumn(int line, int column, TargetLineEntry? lineEntry) {
|
||||
TargetEntry findColumn(int line, int column, TargetLineEntry lineEntry) {
|
||||
if (lineEntry == null || lineEntry.entries.length == 0) return null;
|
||||
if (lineEntry.line != line) return lineEntry.entries.last;
|
||||
var entries = lineEntry.entries;
|
||||
|
|
|
@ -14,10 +14,10 @@ import 'util.dart';
|
|||
|
||||
/// Represents a stack trace line.
|
||||
class StackTraceLine {
|
||||
final String? methodName;
|
||||
final String fileName;
|
||||
final int? lineNo;
|
||||
final int columnNo;
|
||||
String methodName;
|
||||
String fileName;
|
||||
int lineNo;
|
||||
int columnNo;
|
||||
|
||||
StackTraceLine(this.methodName, this.fileName, this.lineNo, this.columnNo);
|
||||
|
||||
|
@ -31,11 +31,11 @@ class StackTraceLine {
|
|||
/// at <fileName>:<lineNo>
|
||||
/// at <fileName>
|
||||
///
|
||||
factory StackTraceLine.fromText(String text, {Logger? logger}) {
|
||||
factory StackTraceLine.fromText(String text, {Logger logger}) {
|
||||
text = text.trim();
|
||||
assert(text.startsWith('at '));
|
||||
text = text.substring('at '.length);
|
||||
String? methodName;
|
||||
String methodName;
|
||||
int endParen = text.indexOf(')');
|
||||
if (endParen > 0) {
|
||||
int nameEnd = text.indexOf('(');
|
||||
|
@ -46,16 +46,16 @@ class StackTraceLine {
|
|||
logger?.log('Missing left-paren in: $text');
|
||||
}
|
||||
}
|
||||
int? lineNo;
|
||||
int? columnNo;
|
||||
int lineNo;
|
||||
int columnNo;
|
||||
String fileName;
|
||||
int lastColon = text.lastIndexOf(':');
|
||||
if (lastColon != -1) {
|
||||
int? lastValue = int.tryParse(text.substring(lastColon + 1));
|
||||
int lastValue = int.tryParse(text.substring(lastColon + 1));
|
||||
if (lastValue != null) {
|
||||
int secondToLastColon = text.lastIndexOf(':', lastColon - 1);
|
||||
if (secondToLastColon != -1) {
|
||||
int? secondToLastValue =
|
||||
int secondToLastValue =
|
||||
int.tryParse(text.substring(secondToLastColon + 1, lastColon));
|
||||
if (secondToLastValue != null) {
|
||||
lineNo = secondToLastValue;
|
||||
|
@ -84,14 +84,14 @@ class StackTraceLine {
|
|||
if (methodName != null) {
|
||||
sb.write(methodName);
|
||||
sb.write(' (');
|
||||
sb.write(fileName);
|
||||
sb.write(fileName ?? '?');
|
||||
sb.write(':');
|
||||
sb.write(lineNo);
|
||||
sb.write(':');
|
||||
sb.write(columnNo);
|
||||
sb.write(')');
|
||||
} else {
|
||||
sb.write(fileName);
|
||||
sb.write(fileName ?? '?');
|
||||
sb.write(':');
|
||||
sb.write(lineNo);
|
||||
sb.write(':');
|
||||
|
@ -103,27 +103,27 @@ class StackTraceLine {
|
|||
String get inlineString {
|
||||
StringBuffer sb = new StringBuffer();
|
||||
var padding = 20;
|
||||
var lineMethodName = methodName;
|
||||
if (lineMethodName != null) {
|
||||
sb.write(lineMethodName);
|
||||
padding -= (lineMethodName.length);
|
||||
if (methodName != null) {
|
||||
sb.write(methodName);
|
||||
padding -= (methodName.length);
|
||||
if (padding <= 0) {
|
||||
sb.write('\n');
|
||||
padding = 20;
|
||||
}
|
||||
}
|
||||
sb.write(' ' * padding);
|
||||
sb.write(p.url.basename(fileName));
|
||||
sb.write(' ');
|
||||
sb.write(lineNo);
|
||||
sb.write(':');
|
||||
sb.write(columnNo);
|
||||
|
||||
if (fileName != null) {
|
||||
sb.write(p.url.basename(fileName));
|
||||
sb.write(' ');
|
||||
sb.write(lineNo);
|
||||
sb.write(':');
|
||||
sb.write(columnNo);
|
||||
}
|
||||
return sb.toString();
|
||||
}
|
||||
}
|
||||
|
||||
List<StackTraceLine> parseStackTrace(String trace, {Logger? logger}) {
|
||||
List<StackTraceLine> parseStackTrace(String trace, {Logger logger}) {
|
||||
List<String> lines = trace.split(new RegExp(r'(\r|\n|\r\n)'));
|
||||
List<StackTraceLine> jsStackTrace = <StackTraceLine>[];
|
||||
for (String line in lines) {
|
||||
|
@ -138,7 +138,7 @@ List<StackTraceLine> parseStackTrace(String trace, {Logger? logger}) {
|
|||
/// Returns the portion of the output that corresponds to the error message.
|
||||
///
|
||||
/// Note: some errors can span multiple lines.
|
||||
String? extractErrorMessage(String trace) {
|
||||
String extractErrorMessage(String trace) {
|
||||
var firstStackFrame = trace.indexOf(new RegExp('\n +at'));
|
||||
if (firstStackFrame == -1) return null;
|
||||
var errorMarker = trace.indexOf('^') + 1;
|
||||
|
|
|
@ -49,8 +49,8 @@ StackDeobfuscationResult deobfuscateStack(
|
|||
|
||||
// Subtract 1 because stack traces use 1-indexed lines and columns and
|
||||
// source maps uses 0-indexed.
|
||||
SourceSpan? span = mapping.sourceMap
|
||||
.spanFor(frame.line! - 1, column - 1, uri: frame.uri.toString());
|
||||
SourceSpan span = mapping.sourceMap
|
||||
.spanFor(frame.line - 1, column - 1, uri: frame.uri?.toString());
|
||||
|
||||
// If we can't find a source span, ignore the frame. It's probably something
|
||||
// internal that the user doesn't care about.
|
||||
|
@ -59,11 +59,11 @@ StackDeobfuscationResult deobfuscateStack(
|
|||
List<Frame> mappedFrames = frameMap[frame] = [];
|
||||
|
||||
SourceFile jsFile = provider.fileFor(frame.uri);
|
||||
int offset = jsFile.getOffset(frame.line! - 1, column - 1);
|
||||
int offset = jsFile.getOffset(frame.line - 1, column - 1);
|
||||
String nameOf(id) =>
|
||||
_normalizeName(id >= 0 ? mapping.sourceMap.names[id] : null);
|
||||
|
||||
Uri? fileName = span.sourceUrl;
|
||||
Uri fileName = span.sourceUrl;
|
||||
int targetLine = span.start.line + 1;
|
||||
int targetColumn = span.start.column + 1;
|
||||
|
||||
|
@ -73,18 +73,18 @@ StackDeobfuscationResult deobfuscateStack(
|
|||
// of the caller frame until we reach the actual function that dart2js
|
||||
// inlined all the code into.
|
||||
Map<int, List<FrameEntry>> frames = mapping.frames;
|
||||
List<int> index = mapping.frameIndex!;
|
||||
List<int> index = mapping.frameIndex;
|
||||
int key = binarySearch(index, (i) => i > offset) - 1;
|
||||
int depth = 0;
|
||||
outer:
|
||||
while (key >= 0) {
|
||||
for (var frame in frames[index[key]]!.reversed) {
|
||||
for (var frame in frames[index[key]].reversed) {
|
||||
if (frame.isEmpty) break outer;
|
||||
if (frame.isPush) {
|
||||
if (depth <= 0) {
|
||||
mappedFrames.add(new Frame(fileName!, targetLine, targetColumn,
|
||||
mappedFrames.add(new Frame(fileName, targetLine, targetColumn,
|
||||
_normalizeName(frame.inlinedMethodName) + "(inlined)"));
|
||||
fileName = Uri.parse(frame.callUri!);
|
||||
fileName = Uri.parse(frame.callUri);
|
||||
targetLine = (frame.callLine ?? 0) + 1;
|
||||
targetColumn = (frame.callColumn ?? 0) + 1;
|
||||
} else {
|
||||
|
@ -100,8 +100,7 @@ StackDeobfuscationResult deobfuscateStack(
|
|||
|
||||
var functionEntry = findEnclosingFunction(provider, frame.uri, offset);
|
||||
String methodName = nameOf(functionEntry?.sourceNameId ?? -1);
|
||||
mappedFrames
|
||||
.add(new Frame(fileName!, targetLine, targetColumn, methodName));
|
||||
mappedFrames.add(new Frame(fileName, targetLine, targetColumn, methodName));
|
||||
deobfuscatedFrames.addAll(mappedFrames);
|
||||
}
|
||||
return new StackDeobfuscationResult(
|
||||
|
@ -110,6 +109,6 @@ StackDeobfuscationResult deobfuscateStack(
|
|||
|
||||
/// Ensure we don't use spaces in method names. At this time, they are only
|
||||
/// introduced by `<anonymous function>`.
|
||||
_normalizeName(String? methodName) =>
|
||||
_normalizeName(String methodName) =>
|
||||
methodName?.replaceAll("<anonymous function>", "<anonymous>") ??
|
||||
'<unknown>';
|
||||
|
|
|
@ -1,5 +1,4 @@
|
|||
import 'dart:io';
|
||||
import 'package:source_maps/parser.dart';
|
||||
import 'package:source_span/source_span.dart';
|
||||
import 'dart2js_mapping.dart';
|
||||
|
||||
|
@ -13,7 +12,7 @@ class CachingFileProvider implements FileProvider {
|
|||
final Map<Uri, String> _sources = {};
|
||||
final Map<Uri, SourceFile> _files = {};
|
||||
final Map<Uri, Dart2jsMapping> _mappings = {};
|
||||
final Logger? logger;
|
||||
final Logger logger;
|
||||
|
||||
CachingFileProvider({this.logger});
|
||||
|
||||
|
@ -24,7 +23,7 @@ class CachingFileProvider implements FileProvider {
|
|||
_files[uri] ??= new SourceFile.fromString(sourcesFor(uri));
|
||||
|
||||
Dart2jsMapping mappingFor(Uri uri) =>
|
||||
_mappings[uri] ??= parseMappingFor(uri, logger: logger)!;
|
||||
_mappings[uri] ??= parseMappingFor(uri, logger: logger);
|
||||
}
|
||||
|
||||
/// A provider that converts `http:` URLs to a `file:` URI assuming that all
|
||||
|
@ -58,5 +57,3 @@ class Logger {
|
|||
}
|
||||
|
||||
var logger = Logger();
|
||||
|
||||
SingleMapping parseSingleMapping(Map json) => parseJson(json) as SingleMapping;
|
||||
|
|
|
@ -10,4 +10,4 @@ dependencies:
|
|||
source_span: any
|
||||
stack_trace: ^1.9.3
|
||||
environment:
|
||||
sdk: '2.12.0 <3.0.0'
|
||||
sdk: '>=2.3.0 <3.0.0'
|
||||
|
|
Loading…
Reference in a new issue