Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(42)

Issue 954513003: Wire in more-real stock data for stocks example. (Closed)

Created:
5 years, 10 months ago by eseidel
Modified:
5 years, 10 months ago
Reviewers:
abarth-chromium
CC:
abarth-chromium, esprehn, mojo-reviews_chromium.org, ojan, qsr+mojo_chromium.org
Base URL:
git@github.com:domokit/mojo.git@master
Target Ref:
refs/heads/master
Project:
mojo
Visibility:
Public.

Description

Patch Set 1 #

Total comments: 3
Unified diffs Side-by-side diffs Delta from patch set Stats (+3025 lines, -23 lines) Patch
A sky/examples/stocks/companylist.sky View 1 chunk +3012 lines, -0 lines 3 comments Download
M sky/examples/stocks/stocks.sky View 3 chunks +13 lines, -23 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
eseidel
5 years, 10 months ago (2015-02-23 19:01:07 UTC) #1
abarth-chromium
lgtm https://codereview.chromium.org/954513003/diff/1/sky/examples/stocks/companylist.sky File sky/examples/stocks/companylist.sky (right): https://codereview.chromium.org/954513003/diff/1/sky/examples/stocks/companylist.sky#newcode5 sky/examples/stocks/companylist.sky:5: final List<List<String>> companyList = [ s/companyList/_kCompanyList/ The _ ...
5 years, 10 months ago (2015-02-23 19:08:16 UTC) #2
eseidel
5 years, 10 months ago (2015-02-23 20:52:21 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
d32aa663a4ff488e1455a443513b7bf162b74530 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698