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

Side by Side Diff: third_party/WebKit/Source/platform/fonts/shaping/HarfBuzzShaper.cpp

Issue 2776203002: Migrate WTF::Vector::remove() to ::erase() (Closed)
Patch Set: rebase, repatch VectorTest 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) 2012 Google Inc. All rights reserved. 2 * Copyright (c) 2012 Google Inc. All rights reserved.
3 * Copyright (C) 2013 BlackBerry Limited. All rights reserved. 3 * Copyright (C) 2013 BlackBerry Limited. All rights reserved.
4 * 4 *
5 * Redistribution and use in source and binary forms, with or without 5 * Redistribution and use in source and binary forms, with or without
6 * modification, are permitted provided that the following conditions are 6 * modification, are permitted provided that the following conditions are
7 * met: 7 * met:
8 * 8 *
9 * * Redistributions of source code must retain the above copyright 9 * * Redistributions of source code must retain the above copyright
10 * notice, this list of conditions and the following disclaimer. 10 * notice, this list of conditions and the following disclaimer.
(...skipping 529 matching lines...) Expand 10 before | Expand all | Expand 10 after
540 } 540 }
541 } 541 }
542 542
543 void CapsFeatureSettingsScopedOverlay::prependCounting( 543 void CapsFeatureSettingsScopedOverlay::prependCounting(
544 const hb_feature_t& feature) { 544 const hb_feature_t& feature) {
545 m_features->push_front(feature); 545 m_features->push_front(feature);
546 m_countFeatures++; 546 m_countFeatures++;
547 } 547 }
548 548
549 CapsFeatureSettingsScopedOverlay::~CapsFeatureSettingsScopedOverlay() { 549 CapsFeatureSettingsScopedOverlay::~CapsFeatureSettingsScopedOverlay() {
550 m_features->remove(0, m_countFeatures); 550 m_features->erase(0, m_countFeatures);
551 } 551 }
552 552
553 } // namespace 553 } // namespace
554 554
555 void HarfBuzzShaper::shapeSegment(RangeData* rangeData, 555 void HarfBuzzShaper::shapeSegment(RangeData* rangeData,
556 RunSegmenter::RunSegmenterRange segment, 556 RunSegmenter::RunSegmenterRange segment,
557 ShapeResult* result) const { 557 ShapeResult* result) const {
558 DCHECK(result); 558 DCHECK(result);
559 DCHECK(rangeData->buffer); 559 DCHECK(rangeData->buffer);
560 560
(...skipping 306 matching lines...) Expand 10 before | Expand all | Expand 10 after
867 if (lastSafe > firstSafe) 867 if (lastSafe > firstSafe)
868 result->copyRange(firstSafe, lastSafe, lineResult.get()); 868 result->copyRange(firstSafe, lastSafe, lineResult.get());
869 if (lineEndResult) 869 if (lineEndResult)
870 lineEndResult->copyRange(lastSafe, maxLength, lineResult.get()); 870 lineEndResult->copyRange(lastSafe, maxLength, lineResult.get());
871 871
872 *breakOffset = breakOpportunity; 872 *breakOffset = breakOpportunity;
873 return lineResult.release(); 873 return lineResult.release();
874 } 874 }
875 875
876 } // namespace blink 876 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698