Add a quick fix to wrap a flutter widget with streamBuilder and futureBuilder

[jwren]

From @shakil807g on August 24, 2018 7:21

Thanks for the feedback! If your issue is related to the Flutter framework itself,
please open an issue at
github.com/flutter/flutter.

Steps to Reproduce

Please tell us what you were doing and what went wrong

Version info

Please paste the output of running flutter doctor -v here (available from the command
line or from Tools > Flutter > Flutter Doctor). It will provide the version of the
Flutter framework as well as of the IntelliJ plugin.

Copied from original issue: flutter/flutter-intellij#2557

[jwren]

From @devoncarew on August 27, 2018 4:39

There must be a intentions for streamBuilder and futureBuilder

@shakil807g, could you clarify this request a bit? Thanks!

[jwren]

From @shakil807g on August 31, 2018 13:48

Can i add wrap with widget like shortcut for streambuilder or futurebuilder . which wrap widget with streamBuilder.

[jwren]

This support would come from the Dart Analysis Server, moving the request to their bug tracker.

[SPodjasek]

I see that StreamBuilder has been already implemented:

sdk/pkg/analysis_server/lib/src/services/correction/assist.dart

Lines 154 to 155 in 010052c

	static const FLUTTER_WRAP_STREAM_BUILDER = const AssistKind(
	'dart.assist.flutter.wrap.streamBuilder', 30, "Wrap with StreamBuilder");

What about FutureBuilder?

[pq]

See also: flutter/flutter-intellij#4540

[FMorschel]

About a discussion over at https://dart-review.googlesource.com/c/sdk/+/408341, @bwilkerson @bkonyi and @kenzieschmoll here is what I meant:

Widget build(BuildContext context) {
  final valueNotifier = ValueNotifier(0);
  return Scaffold(
    appBar: AppBar(
      title: ValueListenableBuilder(
        valueListenable: valueNotifier,
        builder: (context, value, child) {
          return const Text('My test');
        },
      ),
    ),
  );
}

And of course, if I change the declaration of valueNotifier to ValueNotifier< double > or ValueNotifier< num > the value changes too.

But this is not that relevant for this CL, as @bwilkerson said when we make the builders use the same code, we can think of this change to be more consistent and change all of them together. This is being done over at https://dart-review.googlesource.com/c/sdk/+/408860 and #60075. I'm almost done, I only need this CL to merge and we can continue this discussion over there.