Make it harder to forget itemCount for FixedHeightScrollable

sky/sdk/example/stocks/lib/stock_list.dart

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 import 'package:sky/widgets/fixed_height_scrollable.dart';
 import 'package:sky/widgets/basic.dart';
 
 import 'stock_data.dart';
 import 'stock_row.dart';
 
 class Stocklist extends FixedHeightScrollable {
 
   Stocklist({
     String key,
-    this.stocks,
+    List<Stock> stocks,
     this.query

[Hixie, 2015/07/07 15:25:13]

do we still need to remember the query?

[abarth-chromium, 2015/07/07 15:26:21]

Nope!

-  }) : super(itemHeight: StockRow.kHeight, key: key);
+  }) : super(itemHeight: StockRow.kHeight, key: key) {
+    if (query == null) {
+      filteredStocks = stocks;
+    } else {
+      RegExp regexp = new RegExp(query, caseSensitive: false);
+      filteredStocks = stocks
+        .where((stock) => stock.symbol.contains(regexp))
+        .toList(growable: false);
+    }
+  }
 
   String query;
-  List<Stock> stocks;
+  List<Stock> filteredStocks;
+
+  int get itemCount => filteredStocks.length;
 
   void syncFields(Stocklist source) {
     query = source.query;
-    stocks = source.stocks;
+    filteredStocks = source.filteredStocks;
     super.syncFields(source);
   }
 
   List<Widget> buildItems(int start, int count) {
-    var filteredStocks = stocks.where((stock) {
-      return query == null ||
-             stock.symbol.contains(new RegExp(query, caseSensitive: false));
-    });
-    itemCount = filteredStocks.length;
     return filteredStocks
       .skip(start)
       .take(count)
       .map((stock) => new StockRow(stock: stock))
       .toList(growable: false);
   }
 }