[firebase_crashlytic] Return Future<bool> as result of method calls to send reports to crashlytic

[suesitran]

Currently all Crashlytic methods to send report to Crashlytic return Future<void>, and developers do not know when it fails to send report. So I'm suggesting to return a Future<bool> value, true means success, and false means error, and if possible, give details of the error as well.

Methods affected:

• recordError
• recordFlutterError
• recordFlutterFatalError
• sendUnsentReports

[radvansky-tomas]

that does not seem consistent. It should still be future but it should throw respective firebase error type.

Did u actually try to capture error there ?

[suesitran]

There's error captured for full app using runZonedGuarded, but I do not see any error here. Do I have to do try/catch on each call?

[suesitran]

and yes, it should still be future, instead of Future<void>, can it be Future<bool>?