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

Side by Side Diff: third_party/WebKit/Source/core/editing/markers/DocumentMarkerController.cpp

Issue 2836563002: [DMC #1.913] Rename DocumentMarkerController::RemoveMarkers() to RemoveMarkersOfTypes() (Closed)
Patch Set: Rebase Created 3 years, 8 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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org) 2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
3 * (C) 1999 Antti Koivisto (koivisto@kde.org) 3 * (C) 1999 Antti Koivisto (koivisto@kde.org)
4 * (C) 2001 Dirk Mueller (mueller@kde.org) 4 * (C) 2001 Dirk Mueller (mueller@kde.org)
5 * (C) 2006 Alexey Proskuryakov (ap@webkit.org) 5 * (C) 2006 Alexey Proskuryakov (ap@webkit.org)
6 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010 Apple Inc. All rights 6 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010 Apple Inc. All rights
7 * reserved. 7 * reserved.
8 * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. 8 * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved.
9 * (http://www.torchmobile.com/) 9 * (http://www.torchmobile.com/)
10 * Copyright (C) Research In Motion Limited 2010. All rights reserved. 10 * Copyright (C) Research In Motion Limited 2010. All rights reserved.
(...skipping 498 matching lines...) Expand 10 before | Expand all | Expand 10 after
509 list->erase(j - 1); 509 list->erase(j - 1);
510 removed_markers = true; 510 removed_markers = true;
511 } 511 }
512 } 512 }
513 if (removed_markers && type == DocumentMarker::kTextMatch) 513 if (removed_markers && type == DocumentMarker::kTextMatch)
514 InvalidatePaintForTickmarks(node); 514 InvalidatePaintForTickmarks(node);
515 } 515 }
516 } 516 }
517 } 517 }
518 518
519 void DocumentMarkerController::RemoveMarkers( 519 void DocumentMarkerController::RemoveMarkersOfTypes(
520 DocumentMarker::MarkerTypes marker_types) { 520 DocumentMarker::MarkerTypes marker_types) {
521 if (!PossiblyHasMarkers(marker_types)) 521 if (!PossiblyHasMarkers(marker_types))
522 return; 522 return;
523 DCHECK(!markers_.IsEmpty()); 523 DCHECK(!markers_.IsEmpty());
524 524
525 HeapVector<Member<const Node>> nodes_with_markers; 525 HeapVector<Member<const Node>> nodes_with_markers;
526 CopyKeysToVector(markers_, nodes_with_markers); 526 CopyKeysToVector(markers_, nodes_with_markers);
527 unsigned size = nodes_with_markers.size(); 527 unsigned size = nodes_with_markers.size();
528 for (unsigned i = 0; i < size; ++i) { 528 for (unsigned i = 0; i < size; ++i) {
529 MarkerMap::iterator iterator = markers_.Find(nodes_with_markers[i]); 529 MarkerMap::iterator iterator = markers_.Find(nodes_with_markers[i]);
(...skipping 208 matching lines...) Expand 10 before | Expand all | Expand 10 after
738 } 738 }
739 739
740 } // namespace blink 740 } // namespace blink
741 741
742 #ifndef NDEBUG 742 #ifndef NDEBUG
743 void showDocumentMarkers(const blink::DocumentMarkerController* controller) { 743 void showDocumentMarkers(const blink::DocumentMarkerController* controller) {
744 if (controller) 744 if (controller)
745 controller->ShowMarkers(); 745 controller->ShowMarkers();
746 } 746 }
747 #endif 747 #endif
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698