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

[web][upstream] Fix debugPrintStack for web platform (#37638)

Browse Source 
* [web][upstream] Fix debugPrintStack for web platform

Fixes: https://github.com/flutter/flutter/issues/37488
This commit is contained in:
Ferhat 2019-08-06 13:22:40 -07:00 committed by GitHub
parent 626ca086d0
commit 920961abc8
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
packages/flutter/lib/src/foundation/assertions.dart
View File

@ -750,6 +750,12 @@ void debugPrintStack({ String label, int maxFrames }) {
if (label != null)
debugPrint(label);
Iterable<String> lines = StackTrace.current.toString().trimRight().split('\n');
if (kIsWeb) {
// Remove extra call to StackTrace.current for web platform.
// TODO(ferhat): remove when https://github.com/flutter/flutter/issues/37635
// is addressed.
lines = lines.skip(1);
}
if (maxFrames != null)
lines = lines.take(maxFrames);
debugPrint(FlutterError.defaultStackFilter(lines).join('\n'));

4
packages/flutter/test/foundation/assertions_test.dart
View File

@ -14,7 +14,7 @@ void main() {
});
expect(log[0], contains('Example label'));
expect(log[1], contains('debugPrintStack'));
}, skip: isBrowser);
});
test('debugPrintStack', () {
final List<String> log = captureOutput(() {
@ -39,7 +39,7 @@ void main() {
expect(joined, contains('captureOutput'));
expect(joined, contains('\nExample information\n'));
}, skip: isBrowser);
});
test('FlutterErrorDetails.toString', () {
expect(