Commit 1eaefe18 authored by Alexandre Ardhuin's avatar Alexandre Ardhuin Committed by GitHub

more const for immutables (#10563)

parent dfc7788c
...@@ -152,7 +152,7 @@ class ListModel<E> { ...@@ -152,7 +152,7 @@ class ListModel<E> {
/// This widget's height is based on the animation parameter, it varies /// This widget's height is based on the animation parameter, it varies
/// from 0 to 128 as the animation varies from 0.0 to 1.0. /// from 0 to 128 as the animation varies from 0.0 to 1.0.
class CardItem extends StatelessWidget { class CardItem extends StatelessWidget {
CardItem({ const CardItem({
Key key, Key key,
@required this.animation, @required this.animation,
this.onTap, this.onTap,
......
...@@ -68,7 +68,7 @@ final List<Entry> data = <Entry>[ ...@@ -68,7 +68,7 @@ final List<Entry> data = <Entry>[
// Displays one Entry. If the entry has children then it's displayed // Displays one Entry. If the entry has children then it's displayed
// with an ExpansionTile. // with an ExpansionTile.
class EntryItem extends StatelessWidget { class EntryItem extends StatelessWidget {
EntryItem(this.entry); const EntryItem(this.entry);
final Entry entry; final Entry entry;
......
...@@ -75,7 +75,7 @@ class _RenderStatusBarPaddingSliver extends RenderSliver { ...@@ -75,7 +75,7 @@ class _RenderStatusBarPaddingSliver extends RenderSliver {
} }
class _StatusBarPaddingSliver extends SingleChildRenderObjectWidget { class _StatusBarPaddingSliver extends SingleChildRenderObjectWidget {
_StatusBarPaddingSliver({ const _StatusBarPaddingSliver({
Key key, Key key,
@required this.maxHeight, @required this.maxHeight,
this.scrollFactor: 5.0, this.scrollFactor: 5.0,
...@@ -369,7 +369,7 @@ class _AllSectionsView extends AnimatedWidget { ...@@ -369,7 +369,7 @@ class _AllSectionsView extends AnimatedWidget {
// app bar's height is _kAppBarMidHeight and only one section heading is // app bar's height is _kAppBarMidHeight and only one section heading is
// visible. // visible.
class _SnappingScrollPhysics extends ClampingScrollPhysics { class _SnappingScrollPhysics extends ClampingScrollPhysics {
_SnappingScrollPhysics({ const _SnappingScrollPhysics({
ScrollPhysics parent, ScrollPhysics parent,
@required this.midScrollOffset, @required this.midScrollOffset,
}) : assert(midScrollOffset != null), }) : assert(midScrollOffset != null),
......
...@@ -11,7 +11,7 @@ const double kSectionIndicatorWidth = 32.0; ...@@ -11,7 +11,7 @@ const double kSectionIndicatorWidth = 32.0;
// The card for a single section. Displays the section's gradient and background image. // The card for a single section. Displays the section's gradient and background image.
class SectionCard extends StatelessWidget { class SectionCard extends StatelessWidget {
SectionCard({ Key key, @required this.section }) const SectionCard({ Key key, @required this.section })
: assert(section != null), : assert(section != null),
super(key: key); super(key: key);
...@@ -60,7 +60,7 @@ class SectionTitle extends StatelessWidget { ...@@ -60,7 +60,7 @@ class SectionTitle extends StatelessWidget {
color: const Color(0x19000000), color: const Color(0x19000000),
); );
SectionTitle({ const SectionTitle({
Key key, Key key,
@required this.section, @required this.section,
@required this.scale, @required this.scale,
......
...@@ -42,7 +42,7 @@ final List<Palette> allPalettes = <Palette>[ ...@@ -42,7 +42,7 @@ final List<Palette> allPalettes = <Palette>[
class ColorItem extends StatelessWidget { class ColorItem extends StatelessWidget {
ColorItem({ const ColorItem({
Key key, Key key,
@required this.index, @required this.index,
@required this.color, @required this.color,
......
...@@ -116,7 +116,7 @@ class _ShrineGridDelegate extends SliverGridDelegate { ...@@ -116,7 +116,7 @@ class _ShrineGridDelegate extends SliverGridDelegate {
// Displays the Vendor's name and avatar. // Displays the Vendor's name and avatar.
class _VendorItem extends StatelessWidget { class _VendorItem extends StatelessWidget {
_VendorItem({ Key key, @required this.vendor }) const _VendorItem({ Key key, @required this.vendor })
: assert(vendor != null), : assert(vendor != null),
super(key: key); super(key: key);
...@@ -294,7 +294,7 @@ class _Heading extends StatelessWidget { ...@@ -294,7 +294,7 @@ class _Heading extends StatelessWidget {
// A card that displays a product's image, price, and vendor. The _ProductItem // A card that displays a product's image, price, and vendor. The _ProductItem
// cards appear in a grid below the heading. // cards appear in a grid below the heading.
class _ProductItem extends StatelessWidget { class _ProductItem extends StatelessWidget {
_ProductItem({ Key key, @required this.product, this.onPressed }) const _ProductItem({ Key key, @required this.product, this.onPressed })
: assert(product != null), : assert(product != null),
super(key: key); super(key: key);
......
...@@ -12,7 +12,7 @@ import 'shrine_types.dart'; ...@@ -12,7 +12,7 @@ import 'shrine_types.dart';
// Displays the product title's, description, and order quantity dropdown. // Displays the product title's, description, and order quantity dropdown.
class _ProductItem extends StatelessWidget { class _ProductItem extends StatelessWidget {
_ProductItem({ const _ProductItem({
Key key, Key key,
@required this.product, @required this.product,
@required this.quantity, @required this.quantity,
...@@ -69,7 +69,7 @@ class _ProductItem extends StatelessWidget { ...@@ -69,7 +69,7 @@ class _ProductItem extends StatelessWidget {
// Vendor name and description // Vendor name and description
class _VendorItem extends StatelessWidget { class _VendorItem extends StatelessWidget {
_VendorItem({ Key key, @required this.vendor }) const _VendorItem({ Key key, @required this.vendor })
: assert(vendor != null), : assert(vendor != null),
super(key: key); super(key: key);
...@@ -140,7 +140,7 @@ class _HeadingLayout extends MultiChildLayoutDelegate { ...@@ -140,7 +140,7 @@ class _HeadingLayout extends MultiChildLayoutDelegate {
// Describes a product and vendor in detail, supports specifying // Describes a product and vendor in detail, supports specifying
// a order quantity (0-5). Appears at the top of the OrderPage. // a order quantity (0-5). Appears at the top of the OrderPage.
class _Heading extends StatelessWidget { class _Heading extends StatelessWidget {
_Heading({ const _Heading({
Key key, Key key,
@required this.product, @required this.product,
@required this.quantity, @required this.quantity,
......
...@@ -15,7 +15,7 @@ enum ShrineAction { ...@@ -15,7 +15,7 @@ enum ShrineAction {
} }
class ShrinePage extends StatefulWidget { class ShrinePage extends StatefulWidget {
ShrinePage({ const ShrinePage({
Key key, Key key,
@required this.scaffoldKey, @required this.scaffoldKey,
@required this.body, @required this.body,
......
...@@ -6,7 +6,7 @@ import 'package:flutter/foundation.dart'; ...@@ -6,7 +6,7 @@ import 'package:flutter/foundation.dart';
import 'package:flutter/material.dart'; import 'package:flutter/material.dart';
class TextStyleItem extends StatelessWidget { class TextStyleItem extends StatelessWidget {
TextStyleItem({ const TextStyleItem({
Key key, Key key,
@required this.name, @required this.name,
@required this.style, @required this.style,
......
...@@ -88,7 +88,7 @@ class _GalleryDrawerHeaderState extends State<GalleryDrawerHeader> { ...@@ -88,7 +88,7 @@ class _GalleryDrawerHeaderState extends State<GalleryDrawerHeader> {
} }
class GalleryDrawer extends StatelessWidget { class GalleryDrawer extends StatelessWidget {
GalleryDrawer({ const GalleryDrawer({
Key key, Key key,
this.useLightTheme, this.useLightTheme,
@required this.onThemeChanged, @required this.onThemeChanged,
......
...@@ -66,7 +66,7 @@ class _AppBarBackground extends StatelessWidget { ...@@ -66,7 +66,7 @@ class _AppBarBackground extends StatelessWidget {
} }
class GalleryHome extends StatefulWidget { class GalleryHome extends StatefulWidget {
GalleryHome({ const GalleryHome({
Key key, Key key,
this.useLightTheme, this.useLightTheme,
@required this.onThemeChanged, @required this.onThemeChanged,
......
...@@ -12,7 +12,7 @@ import '../demo/all.dart'; ...@@ -12,7 +12,7 @@ import '../demo/all.dart';
typedef Widget GalleryDemoBuilder(); typedef Widget GalleryDemoBuilder();
class GalleryItem extends StatelessWidget { class GalleryItem extends StatelessWidget {
GalleryItem({ const GalleryItem({
@required this.title, @required this.title,
this.subtitle, this.subtitle,
@required this.category, @required this.category,
......
...@@ -12,7 +12,7 @@ import 'package:url_launcher/url_launcher.dart'; ...@@ -12,7 +12,7 @@ import 'package:url_launcher/url_launcher.dart';
typedef Future<String> UpdateUrlFetcher(); typedef Future<String> UpdateUrlFetcher();
class Updater extends StatefulWidget { class Updater extends StatefulWidget {
Updater({ @required this.updateUrlFetcher, this.child, Key key }) const Updater({ @required this.updateUrlFetcher, this.child, Key key })
: assert(updateUrlFetcher != null), : assert(updateUrlFetcher != null),
super(key: key); super(key: key);
......
...@@ -19,13 +19,13 @@ class PlatformView extends StatelessWidget { ...@@ -19,13 +19,13 @@ class PlatformView extends StatelessWidget {
theme: new ThemeData( theme: new ThemeData(
primarySwatch: Colors.grey, primarySwatch: Colors.grey,
), ),
home: new MyHomePage(title: 'Platform View'), home: const MyHomePage(title: 'Platform View'),
); );
} }
} }
class MyHomePage extends StatefulWidget { class MyHomePage extends StatefulWidget {
MyHomePage({Key key, this.title}) : super(key: key); const MyHomePage({Key key, this.title}) : super(key: key);
final String title; final String title;
......
...@@ -35,7 +35,7 @@ abstract class MessageCodec<T> { ...@@ -35,7 +35,7 @@ abstract class MessageCodec<T> {
class MethodCall { class MethodCall {
/// Creates a [MethodCall] representing the invocation of [method] with the /// Creates a [MethodCall] representing the invocation of [method] with the
/// specified [arguments]. /// specified [arguments].
MethodCall(this.method, [this.arguments]) const MethodCall(this.method, [this.arguments])
: assert(method != null); : assert(method != null);
/// The name of the method to be called. /// The name of the method to be called.
......
...@@ -46,7 +46,7 @@ class _ActiveItem implements Comparable<_ActiveItem> { ...@@ -46,7 +46,7 @@ class _ActiveItem implements Comparable<_ActiveItem> {
/// This widget is similar to one created by [ListView.builder]. /// This widget is similar to one created by [ListView.builder].
class AnimatedList extends StatefulWidget { class AnimatedList extends StatefulWidget {
/// Creates a scrolling container that animates items when they are inserted or removed. /// Creates a scrolling container that animates items when they are inserted or removed.
AnimatedList({ const AnimatedList({
Key key, Key key,
@required this.itemBuilder, @required this.itemBuilder,
this.initialItemCount: 0, this.initialItemCount: 0,
......
...@@ -195,7 +195,7 @@ class AsyncSnapshot<T> { ...@@ -195,7 +195,7 @@ class AsyncSnapshot<T> {
const AsyncSnapshot.nothing() : this._(ConnectionState.none, null, null); const AsyncSnapshot.nothing() : this._(ConnectionState.none, null, null);
/// Creates an [AsyncSnapshot] in the specified [state] and with the specified [data]. /// Creates an [AsyncSnapshot] in the specified [state] and with the specified [data].
AsyncSnapshot.withData(ConnectionState state, T data) : this._(state, data, null); // not const because https://github.com/dart-lang/sdk/issues/29432 const AsyncSnapshot.withData(ConnectionState state, T data) : this._(state, data, null);
/// Creates an [AsyncSnapshot] in the specified [state] and with the specified [error]. /// Creates an [AsyncSnapshot] in the specified [state] and with the specified [error].
const AsyncSnapshot.withError(ConnectionState state, Object error) : this._(state, null, error); const AsyncSnapshot.withError(ConnectionState state, Object error) : this._(state, null, error);
......
...@@ -313,10 +313,10 @@ class LabeledGlobalKey<T extends State<StatefulWidget>> extends GlobalKey<T> { ...@@ -313,10 +313,10 @@ class LabeledGlobalKey<T extends State<StatefulWidget>> extends GlobalKey<T> {
/// Creates a global key with a debugging label. /// Creates a global key with a debugging label.
/// ///
/// The label does not affect the key's identity. /// The label does not affect the key's identity.
const LabeledGlobalKey(this._debugLabel) : super.constructor(); LabeledGlobalKey(this._debugLabel) : super.constructor();
// Used for forwarding the constructor from GlobalKey. // Used for forwarding the constructor from GlobalKey.
const LabeledGlobalKey._({ String debugLabel }) : _debugLabel = debugLabel, super.constructor(); LabeledGlobalKey._({ String debugLabel }) : _debugLabel = debugLabel, super.constructor();
final String _debugLabel; final String _debugLabel;
......
...@@ -36,7 +36,7 @@ typedef List<Widget> NestedScrollViewHeaderSliversBuilder(BuildContext context, ...@@ -36,7 +36,7 @@ typedef List<Widget> NestedScrollViewHeaderSliversBuilder(BuildContext context,
const double _kInitialScrollOffset = 0.0; const double _kInitialScrollOffset = 0.0;
class NestedScrollView extends StatefulWidget { class NestedScrollView extends StatefulWidget {
NestedScrollView({ const NestedScrollView({
Key key, Key key,
this.scrollDirection: Axis.vertical, this.scrollDirection: Axis.vertical,
this.reverse: false, this.reverse: false,
......
...@@ -15,6 +15,6 @@ void main() { ...@@ -15,6 +15,6 @@ void main() {
await HapticFeedback.vibrate(); await HapticFeedback.vibrate();
expect(log, equals(<MethodCall>[new MethodCall('HapticFeedback.vibrate')])); expect(log, equals(<MethodCall>[const MethodCall('HapticFeedback.vibrate')]));
}); });
} }
...@@ -91,7 +91,7 @@ void main() { ...@@ -91,7 +91,7 @@ void main() {
}); });
test('can handle method call with no registered plugin', () async { test('can handle method call with no registered plugin', () async {
channel.setMethodCallHandler(null); channel.setMethodCallHandler(null);
final ByteData call = jsonMethod.encodeMethodCall(new MethodCall('sayHello', 'hello')); final ByteData call = jsonMethod.encodeMethodCall(const MethodCall('sayHello', 'hello'));
ByteData envelope; ByteData envelope;
await BinaryMessages.handlePlatformMessage('ch7', call, (ByteData result) { await BinaryMessages.handlePlatformMessage('ch7', call, (ByteData result) {
envelope = result; envelope = result;
...@@ -102,7 +102,7 @@ void main() { ...@@ -102,7 +102,7 @@ void main() {
channel.setMethodCallHandler((MethodCall call) async { channel.setMethodCallHandler((MethodCall call) async {
throw new MissingPluginException(); throw new MissingPluginException();
}); });
final ByteData call = jsonMethod.encodeMethodCall(new MethodCall('sayHello', 'hello')); final ByteData call = jsonMethod.encodeMethodCall(const MethodCall('sayHello', 'hello'));
ByteData envelope; ByteData envelope;
await BinaryMessages.handlePlatformMessage('ch7', call, (ByteData result) { await BinaryMessages.handlePlatformMessage('ch7', call, (ByteData result) {
envelope = result; envelope = result;
...@@ -111,7 +111,7 @@ void main() { ...@@ -111,7 +111,7 @@ void main() {
}); });
test('can handle method call with successful result', () async { test('can handle method call with successful result', () async {
channel.setMethodCallHandler((MethodCall call) async => '${call.arguments}, world'); channel.setMethodCallHandler((MethodCall call) async => '${call.arguments}, world');
final ByteData call = jsonMethod.encodeMethodCall(new MethodCall('sayHello', 'hello')); final ByteData call = jsonMethod.encodeMethodCall(const MethodCall('sayHello', 'hello'));
ByteData envelope; ByteData envelope;
await BinaryMessages.handlePlatformMessage('ch7', call, (ByteData result) { await BinaryMessages.handlePlatformMessage('ch7', call, (ByteData result) {
envelope = result; envelope = result;
...@@ -122,7 +122,7 @@ void main() { ...@@ -122,7 +122,7 @@ void main() {
channel.setMethodCallHandler((MethodCall call) async { channel.setMethodCallHandler((MethodCall call) async {
throw new PlatformException(code: 'bad', message: 'sayHello failed', details: null); throw new PlatformException(code: 'bad', message: 'sayHello failed', details: null);
}); });
final ByteData call = jsonMethod.encodeMethodCall(new MethodCall('sayHello', 'hello')); final ByteData call = jsonMethod.encodeMethodCall(const MethodCall('sayHello', 'hello'));
ByteData envelope; ByteData envelope;
await BinaryMessages.handlePlatformMessage('ch7', call, (ByteData result) { await BinaryMessages.handlePlatformMessage('ch7', call, (ByteData result) {
envelope = result; envelope = result;
...@@ -141,7 +141,7 @@ void main() { ...@@ -141,7 +141,7 @@ void main() {
channel.setMethodCallHandler((MethodCall call) async { channel.setMethodCallHandler((MethodCall call) async {
throw new ArgumentError('bad'); throw new ArgumentError('bad');
}); });
final ByteData call = jsonMethod.encodeMethodCall(new MethodCall('sayHello', 'hello')); final ByteData call = jsonMethod.encodeMethodCall(const MethodCall('sayHello', 'hello'));
ByteData envelope; ByteData envelope;
await BinaryMessages.handlePlatformMessage('ch7', call, (ByteData result) { await BinaryMessages.handlePlatformMessage('ch7', call, (ByteData result) {
envelope = result; envelope = result;
......
...@@ -15,6 +15,6 @@ void main() { ...@@ -15,6 +15,6 @@ void main() {
await SystemNavigator.pop(); await SystemNavigator.pop();
expect(log, equals(<MethodCall>[new MethodCall('SystemNavigator.pop')])); expect(log, equals(<MethodCall>[const MethodCall('SystemNavigator.pop')]));
}); });
} }
...@@ -8,13 +8,13 @@ import 'package:test/test.dart'; ...@@ -8,13 +8,13 @@ import 'package:test/test.dart';
void main() { void main() {
test('System sound control test', () async { test('System sound control test', () async {
final List<MethodCall> log = <MethodCall>[]; final List<MethodCall> log = <MethodCall>[];
SystemChannels.platform.setMockMethodCallHandler((MethodCall methodCall) async { SystemChannels.platform.setMockMethodCallHandler((MethodCall methodCall) async {
log.add(methodCall); log.add(methodCall);
}); });
await SystemSound.play(SystemSoundType.click); await SystemSound.play(SystemSoundType.click);
expect(log, equals(<MethodCall>[new MethodCall('SystemSound.play', "SystemSoundType.click")])); expect(log, equals(<MethodCall>[const MethodCall('SystemSound.play', "SystemSoundType.click")]));
}); });
} }
...@@ -14,7 +14,7 @@ void main() { ...@@ -14,7 +14,7 @@ void main() {
group('AsyncSnapshot', () { group('AsyncSnapshot', () {
test('requiring data succeeds if data is present', () { test('requiring data succeeds if data is present', () {
expect( expect(
new AsyncSnapshot<String>.withData(ConnectionState.done, 'hello').requireData, const AsyncSnapshot<String>.withData(ConnectionState.done, 'hello').requireData,
'hello', 'hello',
); );
}); });
......
...@@ -9,7 +9,7 @@ import 'package:flutter/src/widgets/layout_builder.dart'; ...@@ -9,7 +9,7 @@ import 'package:flutter/src/widgets/layout_builder.dart';
import 'package:flutter_test/flutter_test.dart' hide TypeMatcher; import 'package:flutter_test/flutter_test.dart' hide TypeMatcher;
class Wrapper extends StatelessWidget { class Wrapper extends StatelessWidget {
Wrapper({ const Wrapper({
Key key, Key key,
@required this.child @required this.child
}) : assert(child != null), }) : assert(child != null),
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment