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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 2783653002: Remove ie10 and opera sections from status files (Closed)

Created:
3 years, 8 months ago by Bill Hesse
Modified:
3 years, 8 months ago
Reviewers:
sortie, kustermann
CC:
reviews_dartlang.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+26 lines, -2116 lines) Patch
M pkg/pkg.status View 1 chunk +0 lines, -7 lines 1 comment Download
M samples-dev/samples-dev.status View 1 chunk +0 lines, -4 lines 0 comments Download
M tests/co19/co19-dart2js.status View 1 chunk +0 lines, -2009 lines 0 comments Download
M tests/compiler/dart2js/dart2js.status View 1 chunk +1 line, -1 line 0 comments Download
M tests/corelib/corelib.status View 1 chunk +0 lines, -5 lines 0 comments Download
M tests/html/html.status View 3 chunks +25 lines, -80 lines 0 comments Download
M tests/lib/lib.status View 1 chunk +0 lines, -7 lines 0 comments Download
M utils/tests/peg/peg.status View 1 chunk +0 lines, -3 lines 0 comments Download

Messages

Total messages: 5 (2 generated)
Bill Hesse
https://codereview.chromium.org/2783653002/diff/1/pkg/pkg.status File pkg/pkg.status (left): https://codereview.chromium.org/2783653002/diff/1/pkg/pkg.status#oldcode158 pkg/pkg.status:158: [ $runtime == ie11 ] We don't run kernel ...
3 years, 8 months ago (2017-03-28 12:56:07 UTC) #2
kustermann
lgtm
3 years, 8 months ago (2017-03-28 13:57:40 UTC) #3
Bill Hesse
3 years, 8 months ago (2017-03-28 14:39:11 UTC) #5
Message was sent while issue was closed. 
Committed patchset #1 (id:1) manually as
011a5a6a66726bac1bb5042f9b796e44970c7996 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698