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

Unified Diff: site/try/src/interaction_manager.dart

Issue 542623003: Fix Try Dart handling of Backspace that joins lines (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Added applescript to validate Firefox line-merging fix Created 6 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | tests/try/firefox-linemerges.applescript » ('j') | tests/try/firefox-linemerges.applescript » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: site/try/src/interaction_manager.dart
diff --git a/site/try/src/interaction_manager.dart b/site/try/src/interaction_manager.dart
index 3776dc23014dc3cf52fde88b4c2757f84d77b426..2dc2aa8e820b380689a8109ca53540b60ef6afea 100644
--- a/site/try/src/interaction_manager.dart
+++ b/site/try/src/interaction_manager.dart
@@ -1234,7 +1234,7 @@ void normalizeMutationRecord(MutationRecord record,
var first = record.removedNodes.first;
var line = findLine(record.target);
- if (first is Text && first.data=="\n" && line.nextNode != null) {
+ if (first is Text && line.nextNode != null) {
normalizedNodes.add(line.nextNode);
}
normalizedNodes.add(line);
« no previous file with comments | « no previous file | tests/try/firefox-linemerges.applescript » ('j') | tests/try/firefox-linemerges.applescript » ('J')

Powered by Google App Engine
This is Rietveld 408576698