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

Issue 1148893004: Make sanitizer tests handle IE11, where tagName can't be clobbered (Closed)

Created:
5 years, 7 months ago by Alan Knight
Modified:
5 years, 7 months ago
Reviewers:
sra1
CC:
reviews_dartlang.org, ricow1
Base URL:
https://github.com/dart-lang/sdk.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Make sanitizer tests handle IE11, where tagName can't be clobbered BUG= R=sra@google.com Committed: https://github.com/dart-lang/sdk/commit/ee0a5db6a5a07cc84db95532357b11ad5552bcd1

Patch Set 1 #

Total comments: 2

Patch Set 2 : Removing paranoia around tagName, which wasn't likely to work anyway #

Unified diffs Side-by-side diffs Delta from patch set Stats (+30 lines, -12 lines) Patch
M sdk/lib/html/dart2js/html_dart2js.dart View 1 1 chunk +1 line, -1 line 0 comments Download
M sdk/lib/html/dartium/html_dartium.dart View 1 1 chunk +1 line, -1 line 0 comments Download
M tests/html/html.status View 1 1 chunk +0 lines, -1 line 0 comments Download
M tests/html/node_validator_important_if_you_suppress_make_the_bug_critical_test.dart View 1 chunk +27 lines, -8 lines 0 comments Download
M tools/dom/src/Validators.dart View 1 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 5 (1 generated)
Alan Knight
Mostly just changing the tests, but I felt like tagName was just getting caught by ...
5 years, 7 months ago (2015-05-26 23:20:53 UTC) #2
sra1
lgtm https://codereview.chromium.org/1148893004/diff/1/tools/dom/src/Validators.dart File tools/dom/src/Validators.dart (right): https://codereview.chromium.org/1148893004/diff/1/tools/dom/src/Validators.dart#newcode201 tools/dom/src/Validators.dart:201: if (elementTagName != elementTagName.toString() { 1. Parse error ...
5 years, 7 months ago (2015-05-27 00:45:19 UTC) #3
Alan Knight
https://codereview.chromium.org/1148893004/diff/1/tools/dom/src/Validators.dart File tools/dom/src/Validators.dart (right): https://codereview.chromium.org/1148893004/diff/1/tools/dom/src/Validators.dart#newcode201 tools/dom/src/Validators.dart:201: if (elementTagName != elementTagName.toString() { On 2015/05/27 00:45:18, sra1 ...
5 years, 7 months ago (2015-05-27 19:54:47 UTC) #4
Alan Knight
5 years, 7 months ago (2015-05-27 19:56:15 UTC) #5
Message was sent while issue was closed.
Committed patchset #2 (id:20001) manually as
ee0a5db6a5a07cc84db95532357b11ad5552bcd1 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698