<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN"
"http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head><meta http-equiv="content-type" content="text/html; charset=utf-8" />
<title>[190613] trunk/Source/WebCore</title>
</head>
<body>
<style type="text/css"><!--
#msg dl.meta { border: 1px #006 solid; background: #369; padding: 6px; color: #fff; }
#msg dl.meta dt { float: left; width: 6em; font-weight: bold; }
#msg dt:after { content:':';}
#msg dl, #msg dt, #msg ul, #msg li, #header, #footer, #logmsg { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; }
#msg dl a { font-weight: bold}
#msg dl a:link { color:#fc3; }
#msg dl a:active { color:#ff0; }
#msg dl a:visited { color:#cc6; }
h3 { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; font-weight: bold; }
#msg pre { overflow: auto; background: #ffc; border: 1px #fa0 solid; padding: 6px; }
#logmsg { background: #ffc; border: 1px #fa0 solid; padding: 1em 1em 0 1em; }
#logmsg p, #logmsg pre, #logmsg blockquote { margin: 0 0 1em 0; }
#logmsg p, #logmsg li, #logmsg dt, #logmsg dd { line-height: 14pt; }
#logmsg h1, #logmsg h2, #logmsg h3, #logmsg h4, #logmsg h5, #logmsg h6 { margin: .5em 0; }
#logmsg h1:first-child, #logmsg h2:first-child, #logmsg h3:first-child, #logmsg h4:first-child, #logmsg h5:first-child, #logmsg h6:first-child { margin-top: 0; }
#logmsg ul, #logmsg ol { padding: 0; list-style-position: inside; margin: 0 0 0 1em; }
#logmsg ul { text-indent: -1em; padding-left: 1em; }#logmsg ol { text-indent: -1.5em; padding-left: 1.5em; }
#logmsg > ul, #logmsg > ol { margin: 0 0 1em 0; }
#logmsg pre { background: #eee; padding: 1em; }
#logmsg blockquote { border: 1px solid #fa0; border-left-width: 10px; padding: 1em 1em 0 1em; background: white;}
#logmsg dl { margin: 0; }
#logmsg dt { font-weight: bold; }
#logmsg dd { margin: 0; padding: 0 0 0.5em 0; }
#logmsg dd:before { content:'\00bb';}
#logmsg table { border-spacing: 0px; border-collapse: collapse; border-top: 4px solid #fa0; border-bottom: 1px solid #fa0; background: #fff; }
#logmsg table th { text-align: left; font-weight: normal; padding: 0.2em 0.5em; border-top: 1px dotted #fa0; }
#logmsg table td { text-align: right; border-top: 1px dotted #fa0; padding: 0.2em 0.5em; }
#logmsg table thead th { text-align: center; border-bottom: 1px solid #fa0; }
#logmsg table th.Corner { text-align: left; }
#logmsg hr { border: none 0; border-top: 2px dashed #fa0; height: 1px; }
#header, #footer { color: #fff; background: #636; border: 1px #300 solid; padding: 6px; }
#patch { width: 100%; }
#patch h4 {font-family: verdana,arial,helvetica,sans-serif;font-size:10pt;padding:8px;background:#369;color:#fff;margin:0;}
#patch .propset h4, #patch .binary h4 {margin:0;}
#patch pre {padding:0;line-height:1.2em;margin:0;}
#patch .diff {width:100%;background:#eee;padding: 0 0 10px 0;overflow:auto;}
#patch .propset .diff, #patch .binary .diff {padding:10px 0;}
#patch span {display:block;padding:0 10px;}
#patch .modfile, #patch .addfile, #patch .delfile, #patch .propset, #patch .binary, #patch .copfile {border:1px solid #ccc;margin:10px 0;}
#patch ins {background:#dfd;text-decoration:none;display:block;padding:0 10px;}
#patch del {background:#fdd;text-decoration:none;display:block;padding:0 10px;}
#patch .lines, .info {color:#888;background:#fff;}
--></style>
<div id="msg">
<dl class="meta">
<dt>Revision</dt> <dd><a href="http://trac.webkit.org/projects/webkit/changeset/190613">190613</a></dd>
<dt>Author</dt> <dd>commit-queue@webkit.org</dd>
<dt>Date</dt> <dd>2015-10-06 06:31:14 -0700 (Tue, 06 Oct 2015)</dd>
</dl>
<h3>Log Message</h3>
<pre>Use modern for-loops in WebCore/html.
https://bugs.webkit.org/show_bug.cgi?id=149662
Patch by Hunseop Jeong <hs85.jeong@samsung.com> on 2015-10-06
Reviewed by Darin Adler.
No new tests because there is no behavior change.
* html/DOMFormData.cpp:
(WebCore::DOMFormData::DOMFormData):
* html/EmailInputType.cpp:
(WebCore::EmailInputType::typeMismatchFor):
* html/FileInputType.cpp:
(WebCore::FileInputType::receiveDroppedFiles):
* html/FormController.cpp:
(WebCore::FormControlState::serializeTo):
(WebCore::FormControlState::deserialize):
(WebCore::SavedFormState::serializeTo):
(WebCore::SavedFormState::getReferencedFilePaths):
(WebCore::FormController::createSavedFormStateMap):
(WebCore::FormController::formElementsState):
(WebCore::FormController::restoreControlStateIn):
(WebCore::FormController::getReferencedFilePaths):
* html/HTMLAnchorElement.cpp:
(WebCore::hasNonEmptyBox):
* html/HTMLCanvasElement.cpp:
(WebCore::HTMLCanvasElement::~HTMLCanvasElement):
(WebCore::HTMLCanvasElement::notifyObserversCanvasChanged):
(WebCore::HTMLCanvasElement::reset):
(WebCore::HTMLCanvasElement::paintsIntoCanvasBuffer):
* html/HTMLFieldSetElement.cpp:
(WebCore::HTMLFieldSetElement::length):
* html/HTMLFormControlsCollection.cpp:
(WebCore::firstNamedItem):
* html/HTMLFormElement.cpp:
(WebCore::HTMLFormElement::~HTMLFormElement):
(WebCore::HTMLFormElement::formWouldHaveSecureSubmission):
(WebCore::HTMLFormElement::removedFrom):
(WebCore::HTMLFormElement::length):
(WebCore::HTMLFormElement::submitImplicitly):
(WebCore::HTMLFormElement::validateInteractively):
(WebCore::HTMLFormElement::getTextFieldValues):
(WebCore::HTMLFormElement::submit):
(WebCore::HTMLFormElement::reset):
(WebCore::HTMLFormElement::defaultButton):
(WebCore::HTMLFormElement::checkInvalidControlsAndCollectUnhandled):
(WebCore::HTMLFormElement::removeFromPastNamesMap):
(WebCore::HTMLFormElement::documentDidResumeFromPageCache):
* html/HTMLInputElement.cpp:
(WebCore::parseAcceptAttribute):
* html/HTMLKeygenElement.cpp:
(WebCore::HTMLKeygenElement::HTMLKeygenElement):
* html/HTMLMediaElement.cpp:
(WebCore::HTMLMediaElement::setMediaGroup):
* html/HTMLOptionElement.cpp:
(WebCore::HTMLOptionElement::index):
* html/HTMLSelectElement.cpp:
(WebCore::HTMLSelectElement::saveLastSelection):
(WebCore::HTMLSelectElement::setActiveSelectionAnchorIndex):
(WebCore::HTMLSelectElement::setActiveSelectionEndIndex):
(WebCore::HTMLSelectElement::selectedIndex):
(WebCore::HTMLSelectElement::deselectItemsWithoutValidation):
(WebCore::HTMLSelectElement::saveFormControlState):
(WebCore::HTMLSelectElement::restoreFormControlState):
(WebCore::HTMLSelectElement::appendFormData):
(WebCore::HTMLSelectElement::reset):
* html/HTMLTextAreaElement.cpp:
(WebCore::HTMLTextAreaElement::setDefaultValue):
* html/InputType.cpp:
(WebCore::populateInputTypeFactoryMap):
* html/MediaController.cpp:
(MediaController::duration):
(MediaController::setCurrentTime):
(MediaController::play):
(MediaController::setPlaybackRate):
(MediaController::setVolume):
(MediaController::setMuted):
(playbackStateWaiting):
(MediaController::updateMediaElements):
(MediaController::bringElementUpToSpeed):
(MediaController::isBlocked):
(MediaController::hasEnded):
(MediaController::asyncEventTimerFired):
(MediaController::clearPositionTimerFired):
(MediaController::hasAudio):
(MediaController::hasVideo):
(MediaController::hasClosedCaptions):
(MediaController::setClosedCaptionsVisible):
(MediaController::supportsScanning):
(MediaController::beginScrubbing):
(MediaController::endScrubbing):
(MediaController::canPlay):
(MediaController::isLiveStream):
(MediaController::hasCurrentSrc):
(MediaController::returnToRealtime):
* html/MediaFragmentURIParser.cpp:
(WebCore::MediaFragmentURIParser::parseTimeFragment):
* html/PublicURLManager.cpp:
(WebCore::PublicURLManager::revoke):
(WebCore::PublicURLManager::stop):
* html/canvas/WebGLBuffer.cpp:
(WebCore::WebGLBuffer::getCachedMaxIndex):
(WebCore::WebGLBuffer::setCachedMaxIndex):
(WebCore::WebGLBuffer::setTarget):
* html/canvas/WebGLContextGroup.cpp:
(WebCore::WebGLContextGroup::loseContextGroup):
* html/canvas/WebGLDrawBuffers.cpp:
(WebCore::WebGLDrawBuffers::satisfiesWebGLRequirements):
* html/canvas/WebGLFramebuffer.cpp:
(WebCore::WebGLFramebuffer::removeAttachmentFromBoundFramebuffer):
(WebCore::WebGLFramebuffer::checkStatus):
(WebCore::WebGLFramebuffer::deleteObjectImpl):
(WebCore::WebGLFramebuffer::initializeAttachments):
(WebCore::WebGLFramebuffer::drawBuffers):
* html/canvas/WebGLRenderingContextBase.cpp:
(WebCore::WebGLRenderingContextBase::~WebGLRenderingContextBase):
(WebCore::WebGLRenderingContextBase::deleteTexture):
* html/canvas/WebGLVertexArrayObject.cpp:
(WebCore::WebGLVertexArrayObject::deleteObjectImpl):
* html/canvas/WebGLVertexArrayObjectOES.cpp:
(WebCore::WebGLVertexArrayObjectOES::deleteObjectImpl):
* html/parser/AtomicHTMLToken.h:
(WebCore::AtomicHTMLToken::initializeAttributes):
* html/parser/HTMLConstructionSite.cpp:
(WebCore::HTMLConstructionSite::mergeAttributesFromTokenIntoElement):
* html/parser/HTMLFormattingElementList.cpp:
(WebCore::HTMLFormattingElementList::ensureNoahsArkCondition):
* html/parser/HTMLPreloadScanner.cpp:
(WebCore::TokenPreloadScanner::StartTagScanner::processAttributes):
* html/parser/XSSAuditor.cpp:
(WebCore::semicolonSeparatedValueContainsJavaScriptURL):
* html/shadow/ContentDistributor.cpp:
(WebCore::ContentDistributor::distribute):
(WebCore::ContentDistributor::invalidate):
* html/shadow/MediaControlElements.cpp:
(WebCore::MediaControlClosedCaptionsTrackListElement::updateDisplay):
(WebCore::MediaControlClosedCaptionsTrackListElement::rebuildTrackListMenu):
(WebCore::MediaControlTextTrackContainerElement::updateActiveCuesFontSize):
* html/track/AudioTrackList.cpp:
(AudioTrackList::getTrackById):
* html/track/LoadableTextTrack.cpp:
(WebCore::LoadableTextTrack::newCuesAvailable):
(WebCore::LoadableTextTrack::newRegionsAvailable):
* html/track/TextTrackCueList.cpp:
(WebCore::TextTrackCueList::getCueById):
(WebCore::TextTrackCueList::activeCues):
* html/track/TextTrackList.cpp:
(TextTrackList::getTrackIndexRelativeToRenderedTracks):
(TextTrackList::invalidateTrackIndexesAfterTrack):
* html/track/TrackListBase.cpp:
(TrackListBase::isAnyTrackEnabled):
* html/track/VideoTrackList.cpp:
(VideoTrackList::getTrackById):</pre>
<h3>Modified Paths</h3>
<ul>
<li><a href="#trunkSourceWebCoreChangeLog">trunk/Source/WebCore/ChangeLog</a></li>
<li><a href="#trunkSourceWebCorehtmlDOMFormDatacpp">trunk/Source/WebCore/html/DOMFormData.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmlEmailInputTypecpp">trunk/Source/WebCore/html/EmailInputType.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmlFileInputTypecpp">trunk/Source/WebCore/html/FileInputType.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmlFormControllercpp">trunk/Source/WebCore/html/FormController.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmlHTMLAnchorElementcpp">trunk/Source/WebCore/html/HTMLAnchorElement.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmlHTMLCanvasElementcpp">trunk/Source/WebCore/html/HTMLCanvasElement.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmlHTMLFieldSetElementcpp">trunk/Source/WebCore/html/HTMLFieldSetElement.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmlHTMLFormControlsCollectioncpp">trunk/Source/WebCore/html/HTMLFormControlsCollection.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmlHTMLFormElementcpp">trunk/Source/WebCore/html/HTMLFormElement.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmlHTMLInputElementcpp">trunk/Source/WebCore/html/HTMLInputElement.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmlHTMLKeygenElementcpp">trunk/Source/WebCore/html/HTMLKeygenElement.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmlHTMLMediaElementcpp">trunk/Source/WebCore/html/HTMLMediaElement.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmlHTMLOptionElementcpp">trunk/Source/WebCore/html/HTMLOptionElement.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmlHTMLSelectElementcpp">trunk/Source/WebCore/html/HTMLSelectElement.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmlHTMLTextAreaElementcpp">trunk/Source/WebCore/html/HTMLTextAreaElement.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmlInputTypecpp">trunk/Source/WebCore/html/InputType.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmlMediaControllercpp">trunk/Source/WebCore/html/MediaController.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmlMediaFragmentURIParsercpp">trunk/Source/WebCore/html/MediaFragmentURIParser.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmlPublicURLManagercpp">trunk/Source/WebCore/html/PublicURLManager.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmlcanvasWebGLBuffercpp">trunk/Source/WebCore/html/canvas/WebGLBuffer.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmlcanvasWebGLContextGroupcpp">trunk/Source/WebCore/html/canvas/WebGLContextGroup.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmlcanvasWebGLDrawBufferscpp">trunk/Source/WebCore/html/canvas/WebGLDrawBuffers.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmlcanvasWebGLFramebuffercpp">trunk/Source/WebCore/html/canvas/WebGLFramebuffer.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmlcanvasWebGLRenderingContextBasecpp">trunk/Source/WebCore/html/canvas/WebGLRenderingContextBase.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmlcanvasWebGLVertexArrayObjectcpp">trunk/Source/WebCore/html/canvas/WebGLVertexArrayObject.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmlcanvasWebGLVertexArrayObjectOEScpp">trunk/Source/WebCore/html/canvas/WebGLVertexArrayObjectOES.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmlparserAtomicHTMLTokenh">trunk/Source/WebCore/html/parser/AtomicHTMLToken.h</a></li>
<li><a href="#trunkSourceWebCorehtmlparserHTMLConstructionSitecpp">trunk/Source/WebCore/html/parser/HTMLConstructionSite.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmlparserHTMLFormattingElementListcpp">trunk/Source/WebCore/html/parser/HTMLFormattingElementList.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmlparserHTMLPreloadScannercpp">trunk/Source/WebCore/html/parser/HTMLPreloadScanner.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmlparserXSSAuditorcpp">trunk/Source/WebCore/html/parser/XSSAuditor.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmlshadowContentDistributorcpp">trunk/Source/WebCore/html/shadow/ContentDistributor.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmlshadowMediaControlElementscpp">trunk/Source/WebCore/html/shadow/MediaControlElements.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmltrackAudioTrackListcpp">trunk/Source/WebCore/html/track/AudioTrackList.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmltrackLoadableTextTrackcpp">trunk/Source/WebCore/html/track/LoadableTextTrack.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmltrackTextTrackCueListcpp">trunk/Source/WebCore/html/track/TextTrackCueList.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmltrackTextTrackListcpp">trunk/Source/WebCore/html/track/TextTrackList.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmltrackTrackListBasecpp">trunk/Source/WebCore/html/track/TrackListBase.cpp</a></li>
<li><a href="#trunkSourceWebCorehtmltrackVideoTrackListcpp">trunk/Source/WebCore/html/track/VideoTrackList.cpp</a></li>
</ul>
</div>
<div id="patch">
<h3>Diff</h3>
<a id="trunkSourceWebCoreChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/ChangeLog (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/ChangeLog 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/ChangeLog 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -1,3 +1,158 @@
</span><ins>+2015-10-06 Hunseop Jeong <hs85.jeong@samsung.com>
+
+ Use modern for-loops in WebCore/html.
+ https://bugs.webkit.org/show_bug.cgi?id=149662
+
+ Reviewed by Darin Adler.
+
+ No new tests because there is no behavior change.
+
+ * html/DOMFormData.cpp:
+ (WebCore::DOMFormData::DOMFormData):
+ * html/EmailInputType.cpp:
+ (WebCore::EmailInputType::typeMismatchFor):
+ * html/FileInputType.cpp:
+ (WebCore::FileInputType::receiveDroppedFiles):
+ * html/FormController.cpp:
+ (WebCore::FormControlState::serializeTo):
+ (WebCore::FormControlState::deserialize):
+ (WebCore::SavedFormState::serializeTo):
+ (WebCore::SavedFormState::getReferencedFilePaths):
+ (WebCore::FormController::createSavedFormStateMap):
+ (WebCore::FormController::formElementsState):
+ (WebCore::FormController::restoreControlStateIn):
+ (WebCore::FormController::getReferencedFilePaths):
+ * html/HTMLAnchorElement.cpp:
+ (WebCore::hasNonEmptyBox):
+ * html/HTMLCanvasElement.cpp:
+ (WebCore::HTMLCanvasElement::~HTMLCanvasElement):
+ (WebCore::HTMLCanvasElement::notifyObserversCanvasChanged):
+ (WebCore::HTMLCanvasElement::reset):
+ (WebCore::HTMLCanvasElement::paintsIntoCanvasBuffer):
+ * html/HTMLFieldSetElement.cpp:
+ (WebCore::HTMLFieldSetElement::length):
+ * html/HTMLFormControlsCollection.cpp:
+ (WebCore::firstNamedItem):
+ * html/HTMLFormElement.cpp:
+ (WebCore::HTMLFormElement::~HTMLFormElement):
+ (WebCore::HTMLFormElement::formWouldHaveSecureSubmission):
+ (WebCore::HTMLFormElement::removedFrom):
+ (WebCore::HTMLFormElement::length):
+ (WebCore::HTMLFormElement::submitImplicitly):
+ (WebCore::HTMLFormElement::validateInteractively):
+ (WebCore::HTMLFormElement::getTextFieldValues):
+ (WebCore::HTMLFormElement::submit):
+ (WebCore::HTMLFormElement::reset):
+ (WebCore::HTMLFormElement::defaultButton):
+ (WebCore::HTMLFormElement::checkInvalidControlsAndCollectUnhandled):
+ (WebCore::HTMLFormElement::removeFromPastNamesMap):
+ (WebCore::HTMLFormElement::documentDidResumeFromPageCache):
+ * html/HTMLInputElement.cpp:
+ (WebCore::parseAcceptAttribute):
+ * html/HTMLKeygenElement.cpp:
+ (WebCore::HTMLKeygenElement::HTMLKeygenElement):
+ * html/HTMLMediaElement.cpp:
+ (WebCore::HTMLMediaElement::setMediaGroup):
+ * html/HTMLOptionElement.cpp:
+ (WebCore::HTMLOptionElement::index):
+ * html/HTMLSelectElement.cpp:
+ (WebCore::HTMLSelectElement::saveLastSelection):
+ (WebCore::HTMLSelectElement::setActiveSelectionAnchorIndex):
+ (WebCore::HTMLSelectElement::setActiveSelectionEndIndex):
+ (WebCore::HTMLSelectElement::selectedIndex):
+ (WebCore::HTMLSelectElement::deselectItemsWithoutValidation):
+ (WebCore::HTMLSelectElement::saveFormControlState):
+ (WebCore::HTMLSelectElement::restoreFormControlState):
+ (WebCore::HTMLSelectElement::appendFormData):
+ (WebCore::HTMLSelectElement::reset):
+ * html/HTMLTextAreaElement.cpp:
+ (WebCore::HTMLTextAreaElement::setDefaultValue):
+ * html/InputType.cpp:
+ (WebCore::populateInputTypeFactoryMap):
+ * html/MediaController.cpp:
+ (MediaController::duration):
+ (MediaController::setCurrentTime):
+ (MediaController::play):
+ (MediaController::setPlaybackRate):
+ (MediaController::setVolume):
+ (MediaController::setMuted):
+ (playbackStateWaiting):
+ (MediaController::updateMediaElements):
+ (MediaController::bringElementUpToSpeed):
+ (MediaController::isBlocked):
+ (MediaController::hasEnded):
+ (MediaController::asyncEventTimerFired):
+ (MediaController::clearPositionTimerFired):
+ (MediaController::hasAudio):
+ (MediaController::hasVideo):
+ (MediaController::hasClosedCaptions):
+ (MediaController::setClosedCaptionsVisible):
+ (MediaController::supportsScanning):
+ (MediaController::beginScrubbing):
+ (MediaController::endScrubbing):
+ (MediaController::canPlay):
+ (MediaController::isLiveStream):
+ (MediaController::hasCurrentSrc):
+ (MediaController::returnToRealtime):
+ * html/MediaFragmentURIParser.cpp:
+ (WebCore::MediaFragmentURIParser::parseTimeFragment):
+ * html/PublicURLManager.cpp:
+ (WebCore::PublicURLManager::revoke):
+ (WebCore::PublicURLManager::stop):
+ * html/canvas/WebGLBuffer.cpp:
+ (WebCore::WebGLBuffer::getCachedMaxIndex):
+ (WebCore::WebGLBuffer::setCachedMaxIndex):
+ (WebCore::WebGLBuffer::setTarget):
+ * html/canvas/WebGLContextGroup.cpp:
+ (WebCore::WebGLContextGroup::loseContextGroup):
+ * html/canvas/WebGLDrawBuffers.cpp:
+ (WebCore::WebGLDrawBuffers::satisfiesWebGLRequirements):
+ * html/canvas/WebGLFramebuffer.cpp:
+ (WebCore::WebGLFramebuffer::removeAttachmentFromBoundFramebuffer):
+ (WebCore::WebGLFramebuffer::checkStatus):
+ (WebCore::WebGLFramebuffer::deleteObjectImpl):
+ (WebCore::WebGLFramebuffer::initializeAttachments):
+ (WebCore::WebGLFramebuffer::drawBuffers):
+ * html/canvas/WebGLRenderingContextBase.cpp:
+ (WebCore::WebGLRenderingContextBase::~WebGLRenderingContextBase):
+ (WebCore::WebGLRenderingContextBase::deleteTexture):
+ * html/canvas/WebGLVertexArrayObject.cpp:
+ (WebCore::WebGLVertexArrayObject::deleteObjectImpl):
+ * html/canvas/WebGLVertexArrayObjectOES.cpp:
+ (WebCore::WebGLVertexArrayObjectOES::deleteObjectImpl):
+ * html/parser/AtomicHTMLToken.h:
+ (WebCore::AtomicHTMLToken::initializeAttributes):
+ * html/parser/HTMLConstructionSite.cpp:
+ (WebCore::HTMLConstructionSite::mergeAttributesFromTokenIntoElement):
+ * html/parser/HTMLFormattingElementList.cpp:
+ (WebCore::HTMLFormattingElementList::ensureNoahsArkCondition):
+ * html/parser/HTMLPreloadScanner.cpp:
+ (WebCore::TokenPreloadScanner::StartTagScanner::processAttributes):
+ * html/parser/XSSAuditor.cpp:
+ (WebCore::semicolonSeparatedValueContainsJavaScriptURL):
+ * html/shadow/ContentDistributor.cpp:
+ (WebCore::ContentDistributor::distribute):
+ (WebCore::ContentDistributor::invalidate):
+ * html/shadow/MediaControlElements.cpp:
+ (WebCore::MediaControlClosedCaptionsTrackListElement::updateDisplay):
+ (WebCore::MediaControlClosedCaptionsTrackListElement::rebuildTrackListMenu):
+ (WebCore::MediaControlTextTrackContainerElement::updateActiveCuesFontSize):
+ * html/track/AudioTrackList.cpp:
+ (AudioTrackList::getTrackById):
+ * html/track/LoadableTextTrack.cpp:
+ (WebCore::LoadableTextTrack::newCuesAvailable):
+ (WebCore::LoadableTextTrack::newRegionsAvailable):
+ * html/track/TextTrackCueList.cpp:
+ (WebCore::TextTrackCueList::getCueById):
+ (WebCore::TextTrackCueList::activeCues):
+ * html/track/TextTrackList.cpp:
+ (TextTrackList::getTrackIndexRelativeToRenderedTracks):
+ (TextTrackList::invalidateTrackIndexesAfterTrack):
+ * html/track/TrackListBase.cpp:
+ (TrackListBase::isAnyTrackEnabled):
+ * html/track/VideoTrackList.cpp:
+ (VideoTrackList::getTrackById):
+
</ins><span class="cx"> 2015-10-06 Alex Christensen <achristensen@webkit.org>
</span><span class="cx">
</span><span class="cx"> Report error when main resource is blocked by content blocker
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlDOMFormDatacpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/DOMFormData.cpp (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/DOMFormData.cpp 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/html/DOMFormData.cpp 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -47,10 +47,9 @@
</span><span class="cx"> if (!form)
</span><span class="cx"> return;
</span><span class="cx">
</span><del>- for (unsigned i = 0; i < form->associatedElements().size(); ++i) {
- FormAssociatedElement& element = *form->associatedElements()[i];
- if (!element.asHTMLElement().isDisabledFormControl())
- element.appendFormData(*this, true);
</del><ins>+ for (auto& element : form->associatedElements()) {
+ if (!element->asHTMLElement().isDisabledFormControl())
+ element->appendFormData(*this, true);
</ins><span class="cx"> }
</span><span class="cx"> }
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlEmailInputTypecpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/EmailInputType.cpp (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/EmailInputType.cpp 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/html/EmailInputType.cpp 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -65,8 +65,8 @@
</span><span class="cx"> return !isValidEmailAddress(value);
</span><span class="cx"> Vector<String> addresses;
</span><span class="cx"> value.split(',', true, addresses);
</span><del>- for (unsigned i = 0; i < addresses.size(); ++i) {
- if (!isValidEmailAddress(stripLeadingAndTrailingHTMLSpaces(addresses[i])))
</del><ins>+ for (auto& address : addresses) {
+ if (!isValidEmailAddress(stripLeadingAndTrailingHTMLSpaces(address)))
</ins><span class="cx"> return true;
</span><span class="cx"> }
</span><span class="cx"> return false;
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlFileInputTypecpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/FileInputType.cpp (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/FileInputType.cpp 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/html/FileInputType.cpp 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -399,8 +399,8 @@
</span><span class="cx"> HTMLInputElement* input = &element();
</span><span class="cx">
</span><span class="cx"> Vector<FileChooserFileInfo> files;
</span><del>- for (unsigned i = 0; i < paths.size(); ++i)
- files.append(FileChooserFileInfo(paths[i]));
</del><ins>+ for (auto& path : paths)
+ files.append(FileChooserFileInfo(path));
</ins><span class="cx">
</span><span class="cx"> if (input->fastHasAttribute(multipleAttr))
</span><span class="cx"> filesChosen(files);
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlFormControllercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/FormController.cpp (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/FormController.cpp 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/html/FormController.cpp 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -55,8 +55,8 @@
</span><span class="cx"> {
</span><span class="cx"> ASSERT(!isFailure());
</span><span class="cx"> stateVector.append(String::number(m_values.size()));
</span><del>- for (size_t i = 0; i < m_values.size(); ++i)
- stateVector.append(m_values[i].isNull() ? emptyString() : m_values[i]);
</del><ins>+ for (auto& value : m_values)
+ stateVector.append(value.isNull() ? emptyString() : value);
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> FormControlState FormControlState::deserialize(const Vector<String>& stateVector, size_t& index)
</span><span class="lines">@@ -222,13 +222,12 @@
</span><span class="cx"> void SavedFormState::serializeTo(Vector<String>& stateVector) const
</span><span class="cx"> {
</span><span class="cx"> stateVector.append(String::number(m_controlStateCount));
</span><del>- for (FormElementStateMap::const_iterator it = m_stateForNewFormElements.begin(); it != m_stateForNewFormElements.end(); ++it) {
- const FormElementKey& key = it->key;
- const Deque<FormControlState>& queue = it->value;
- for (Deque<FormControlState>::const_iterator queIterator = queue.begin(); queIterator != queue.end(); ++queIterator) {
</del><ins>+ for (auto& element : m_stateForNewFormElements) {
+ const FormElementKey& key = element.key;
+ for (auto& state : element.value) {
</ins><span class="cx"> stateVector.append(key.name());
</span><span class="cx"> stateVector.append(key.type());
</span><del>- queIterator->serializeTo(stateVector);
</del><ins>+ state.serializeTo(stateVector);
</ins><span class="cx"> }
</span><span class="cx"> }
</span><span class="cx"> }
</span><span class="lines">@@ -265,15 +264,12 @@
</span><span class="cx"> Vector<String> SavedFormState::getReferencedFilePaths() const
</span><span class="cx"> {
</span><span class="cx"> Vector<String> toReturn;
</span><del>- for (FormElementStateMap::const_iterator it = m_stateForNewFormElements.begin(); it != m_stateForNewFormElements.end(); ++it) {
- const FormElementKey& key = it->key;
- if (!equal(key.type(), "file", 4))
</del><ins>+ for (auto& element : m_stateForNewFormElements) {
+ if (!equal(element.key.type(), "file", 4))
</ins><span class="cx"> continue;
</span><del>- const Deque<FormControlState>& queue = it->value;
- for (Deque<FormControlState>::const_iterator queIterator = queue.begin(); queIterator != queue.end(); ++queIterator) {
- const Vector<FileChooserFileInfo>& selectedFiles = HTMLInputElement::filesFromFileInputFormControlState(*queIterator);
- for (size_t i = 0; i < selectedFiles.size(); ++i)
- toReturn.append(selectedFiles[i].path);
</del><ins>+ for (auto& state : element.value) {
+ for (auto& file : HTMLInputElement::filesFromFileInputFormControlState(state))
+ toReturn.append(file.path);
</ins><span class="cx"> }
</span><span class="cx"> }
</span><span class="cx"> return toReturn;
</span><span class="lines">@@ -398,8 +394,7 @@
</span><span class="cx"> {
</span><span class="cx"> FormKeyGenerator keyGenerator;
</span><span class="cx"> auto stateMap = std::make_unique<SavedFormStateMap>();
</span><del>- for (FormElementListHashSet::const_iterator it = controlList.begin(); it != controlList.end(); ++it) {
- HTMLFormControlElementWithState* control = it->get();
</del><ins>+ for (auto& control : controlList) {
</ins><span class="cx"> if (!control->shouldSaveAndRestoreFormControlState())
</span><span class="cx"> continue;
</span><span class="cx"> auto& formState = stateMap->add(keyGenerator.formKey(*control).impl(), nullptr).iterator->value;
</span><span class="lines">@@ -416,9 +411,9 @@
</span><span class="cx"> Vector<String> stateVector;
</span><span class="cx"> stateVector.reserveInitialCapacity(m_formElementsWithState.size() * 4);
</span><span class="cx"> stateVector.append(formStateSignature());
</span><del>- for (SavedFormStateMap::const_iterator it = stateMap->begin(); it != stateMap->end(); ++it) {
- stateVector.append(it->key.get());
- it->value->serializeTo(stateVector);
</del><ins>+ for (auto& state : *stateMap) {
+ stateVector.append(state.key.get());
+ state.value->serializeTo(stateVector);
</ins><span class="cx"> }
</span><span class="cx"> bool hasOnlySignature = stateVector.size() == 1;
</span><span class="cx"> if (hasOnlySignature)
</span><span class="lines">@@ -489,11 +484,10 @@
</span><span class="cx">
</span><span class="cx"> void FormController::restoreControlStateIn(HTMLFormElement& form)
</span><span class="cx"> {
</span><del>- const Vector<FormAssociatedElement*>& elements = form.associatedElements();
- for (size_t i = 0; i < elements.size(); ++i) {
- if (!elements[i]->isFormControlElementWithState())
</del><ins>+ for (auto& element : form.associatedElements()) {
+ if (!element->isFormControlElementWithState())
</ins><span class="cx"> continue;
</span><del>- HTMLFormControlElementWithState* control = static_cast<HTMLFormControlElementWithState*>(elements[i]);
</del><ins>+ HTMLFormControlElementWithState* control = static_cast<HTMLFormControlElementWithState*>(element);
</ins><span class="cx"> if (!control->shouldSaveAndRestoreFormControlState())
</span><span class="cx"> continue;
</span><span class="cx"> if (ownerFormForState(*control) != &form)
</span><span class="lines">@@ -509,8 +503,8 @@
</span><span class="cx"> Vector<String> toReturn;
</span><span class="cx"> SavedFormStateMap map;
</span><span class="cx"> formStatesFromStateVector(stateVector, map);
</span><del>- for (SavedFormStateMap::const_iterator it = map.begin(), end = map.end(); it != end; ++it)
- toReturn.appendVector(it->value->getReferencedFilePaths());
</del><ins>+ for (auto& state : map.values())
+ toReturn.appendVector(state->getReferencedFilePaths());
</ins><span class="cx"> return toReturn;
</span><span class="cx"> }
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlHTMLAnchorElementcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/HTMLAnchorElement.cpp (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/HTMLAnchorElement.cpp 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/html/HTMLAnchorElement.cpp 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -121,9 +121,8 @@
</span><span class="cx"> // pass in 0,0 for the layout point instead of calling localToAbsolute?
</span><span class="cx"> Vector<IntRect> rects;
</span><span class="cx"> renderer->absoluteRects(rects, flooredLayoutPoint(renderer->localToAbsolute()));
</span><del>- size_t size = rects.size();
- for (size_t i = 0; i < size; ++i) {
- if (!rects[i].isEmpty())
</del><ins>+ for (auto& rect : rects) {
+ if (!rect.isEmpty())
</ins><span class="cx"> return true;
</span><span class="cx"> }
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlHTMLCanvasElementcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/HTMLCanvasElement.cpp (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/HTMLCanvasElement.cpp 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/html/HTMLCanvasElement.cpp 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -99,8 +99,8 @@
</span><span class="cx">
</span><span class="cx"> HTMLCanvasElement::~HTMLCanvasElement()
</span><span class="cx"> {
</span><del>- for (auto it = m_observers.begin(), end = m_observers.end(); it != end; ++it)
- (*it)->canvasDestroyed(*this);
</del><ins>+ for (auto& observer : m_observers)
+ observer->canvasDestroyed(*this);
</ins><span class="cx">
</span><span class="cx"> m_context = nullptr; // Ensure this goes away before the ImageBuffer.
</span><span class="cx"> }
</span><span class="lines">@@ -284,8 +284,8 @@
</span><span class="cx">
</span><span class="cx"> void HTMLCanvasElement::notifyObserversCanvasChanged(const FloatRect& rect)
</span><span class="cx"> {
</span><del>- for (auto it = m_observers.begin(), end = m_observers.end(); it != end; ++it)
- (*it)->canvasChanged(*this, rect);
</del><ins>+ for (auto& observer : m_observers)
+ observer->canvasChanged(*this, rect);
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void HTMLCanvasElement::reset()
</span><span class="lines">@@ -344,8 +344,8 @@
</span><span class="cx"> }
</span><span class="cx"> }
</span><span class="cx">
</span><del>- for (auto it = m_observers.begin(), end = m_observers.end(); it != end; ++it)
- (*it)->canvasResized(*this);
</del><ins>+ for (auto& observer : m_observers)
+ observer->canvasResized(*this);
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> bool HTMLCanvasElement::paintsIntoCanvasBuffer() const
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlHTMLFieldSetElementcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/HTMLFieldSetElement.cpp (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/HTMLFieldSetElement.cpp 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/html/HTMLFieldSetElement.cpp 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -193,8 +193,8 @@
</span><span class="cx"> {
</span><span class="cx"> refreshElementsIfNeeded();
</span><span class="cx"> unsigned length = 0;
</span><del>- for (unsigned i = 0; i < m_associatedElements.size(); ++i) {
- if (m_associatedElements[i]->isEnumeratable())
</del><ins>+ for (auto& element : m_associatedElements) {
+ if (element->isEnumeratable())
</ins><span class="cx"> ++length;
</span><span class="cx"> }
</span><span class="cx"> return length;
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlHTMLFormControlsCollectioncpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/HTMLFormControlsCollection.cpp (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/HTMLFormControlsCollection.cpp 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/html/HTMLFormControlsCollection.cpp 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -103,19 +103,18 @@
</span><span class="cx"> {
</span><span class="cx"> ASSERT(attrName == idAttr || attrName == nameAttr);
</span><span class="cx">
</span><del>- for (unsigned i = 0; i < elementsArray.size(); ++i) {
- HTMLElement& element = elementsArray[i]->asHTMLElement();
- if (elementsArray[i]->isEnumeratable() && element.fastGetAttribute(attrName) == name)
</del><ins>+ for (auto& entry : elementsArray) {
+ HTMLElement& element = entry->asHTMLElement();
+ if (entry->isEnumeratable() && element.fastGetAttribute(attrName) == name)
</ins><span class="cx"> return &element;
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> if (!imageElementsArray)
</span><span class="cx"> return 0;
</span><span class="cx">
</span><del>- for (unsigned i = 0; i < imageElementsArray->size(); ++i) {
- HTMLImageElement& element = *(*imageElementsArray)[i];
- if (element.fastGetAttribute(attrName) == name)
- return &element;
</del><ins>+ for (auto& element : *imageElementsArray) {
+ if (element->fastGetAttribute(attrName) == name)
+ return element;
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> return nullptr;
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlHTMLFormElementcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/HTMLFormElement.cpp (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/HTMLFormElement.cpp 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/html/HTMLFormElement.cpp 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -86,10 +86,10 @@
</span><span class="cx"> if (!shouldAutocomplete())
</span><span class="cx"> document().unregisterForPageCacheSuspensionCallbacks(this);
</span><span class="cx">
</span><del>- for (unsigned i = 0; i < m_associatedElements.size(); ++i)
- m_associatedElements[i]->formWillBeDestroyed();
- for (unsigned i = 0; i < m_imageElements.size(); ++i)
- m_imageElements[i]->m_form = 0;
</del><ins>+ for (auto& associatedElement : m_associatedElements)
+ associatedElement->formWillBeDestroyed();
+ for (auto& imageElement : m_imageElements)
+ imageElement->m_form = nullptr;
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> bool HTMLFormElement::formWouldHaveSecureSubmission(const String& url)
</span><span class="lines">@@ -147,8 +147,8 @@
</span><span class="cx"> {
</span><span class="cx"> Node* root = findRoot(this);
</span><span class="cx"> Vector<FormAssociatedElement*> associatedElements(m_associatedElements);
</span><del>- for (unsigned i = 0; i < associatedElements.size(); ++i)
- associatedElements[i]->formRemovedFromTree(root);
</del><ins>+ for (auto& associatedElement : associatedElements)
+ associatedElement->formRemovedFromTree(root);
</ins><span class="cx"> HTMLElement::removedFrom(insertionPoint);
</span><span class="cx"> }
</span><span class="cx">
</span><span class="lines">@@ -165,9 +165,10 @@
</span><span class="cx"> unsigned HTMLFormElement::length() const
</span><span class="cx"> {
</span><span class="cx"> unsigned len = 0;
</span><del>- for (unsigned i = 0; i < m_associatedElements.size(); ++i)
- if (m_associatedElements[i]->isEnumeratable())
</del><ins>+ for (auto& associatedElement : m_associatedElements) {
+ if (associatedElement->isEnumeratable())
</ins><span class="cx"> ++len;
</span><ins>+ }
</ins><span class="cx"> return len;
</span><span class="cx"> }
</span><span class="cx">
</span><span class="lines">@@ -179,8 +180,7 @@
</span><span class="cx"> void HTMLFormElement::submitImplicitly(Event* event, bool fromImplicitSubmissionTrigger)
</span><span class="cx"> {
</span><span class="cx"> unsigned submissionTriggerCount = 0;
</span><del>- for (unsigned i = 0; i < m_associatedElements.size(); ++i) {
- FormAssociatedElement* formAssociatedElement = m_associatedElements[i];
</del><ins>+ for (auto& formAssociatedElement : m_associatedElements) {
</ins><span class="cx"> if (!is<HTMLFormControlElement>(*formAssociatedElement))
</span><span class="cx"> continue;
</span><span class="cx"> HTMLFormControlElement& formElement = downcast<HTMLFormControlElement>(*formAssociatedElement);
</span><span class="lines">@@ -221,9 +221,9 @@
</span><span class="cx"> if (submitElement && submitElement->formNoValidate())
</span><span class="cx"> return true;
</span><span class="cx">
</span><del>- for (unsigned i = 0; i < m_associatedElements.size(); ++i) {
- if (is<HTMLFormControlElement>(*m_associatedElements[i]))
- downcast<HTMLFormControlElement>(*m_associatedElements[i]).hideVisibleValidationMessage();
</del><ins>+ for (auto& associatedElement : m_associatedElements) {
+ if (is<HTMLFormControlElement>(*associatedElement))
+ downcast<HTMLFormControlElement>(*associatedElement).hideVisibleValidationMessage();
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> Vector<RefPtr<FormAssociatedElement>> unhandledInvalidControls;
</span><span class="lines">@@ -239,8 +239,8 @@
</span><span class="cx"> Ref<HTMLFormElement> protect(*this);
</span><span class="cx">
</span><span class="cx"> // Focus on the first focusable control and show a validation message.
</span><del>- for (unsigned i = 0; i < unhandledInvalidControls.size(); ++i) {
- HTMLElement& element = unhandledInvalidControls[i]->asHTMLElement();
</del><ins>+ for (auto& control : unhandledInvalidControls) {
+ HTMLElement& element = control->asHTMLElement();
</ins><span class="cx"> if (element.inDocument() && element.isFocusable()) {
</span><span class="cx"> element.scrollIntoViewIfNeeded(false);
</span><span class="cx"> element.focus();
</span><span class="lines">@@ -252,13 +252,12 @@
</span><span class="cx">
</span><span class="cx"> // Warn about all of unfocusable controls.
</span><span class="cx"> if (document().frame()) {
</span><del>- for (unsigned i = 0; i < unhandledInvalidControls.size(); ++i) {
- FormAssociatedElement& control = *unhandledInvalidControls[i];
- HTMLElement& element = control.asHTMLElement();
</del><ins>+ for (auto& control : unhandledInvalidControls) {
+ HTMLElement& element = control->asHTMLElement();
</ins><span class="cx"> if (element.inDocument() && element.isFocusable())
</span><span class="cx"> continue;
</span><span class="cx"> String message("An invalid form control with name='%name' is not focusable.");
</span><del>- message.replace("%name", control.name());
</del><ins>+ message.replace("%name", control->name());
</ins><span class="cx"> document().addConsoleMessage(MessageSource::Rendering, MessageLevel::Error, message);
</span><span class="cx"> }
</span><span class="cx"> }
</span><span class="lines">@@ -312,9 +311,8 @@
</span><span class="cx"> ASSERT_ARG(fieldNamesAndValues, fieldNamesAndValues.isEmpty());
</span><span class="cx">
</span><span class="cx"> fieldNamesAndValues.reserveCapacity(m_associatedElements.size());
</span><del>- for (unsigned i = 0; i < m_associatedElements.size(); ++i) {
- FormAssociatedElement& control = *m_associatedElements[i];
- HTMLElement& element = control.asHTMLElement();
</del><ins>+ for (auto& associatedElement : m_associatedElements) {
+ HTMLElement& element = associatedElement->asHTMLElement();
</ins><span class="cx"> if (!is<HTMLInputElement>(element))
</span><span class="cx"> continue;
</span><span class="cx"> HTMLInputElement& input = downcast<HTMLInputElement>(element);
</span><span class="lines">@@ -342,8 +340,7 @@
</span><span class="cx"> RefPtr<HTMLFormControlElement> firstSuccessfulSubmitButton;
</span><span class="cx"> bool needButtonActivation = activateSubmitButton; // do we need to activate a submit button?
</span><span class="cx">
</span><del>- for (unsigned i = 0; i < m_associatedElements.size(); ++i) {
- FormAssociatedElement* associatedElement = m_associatedElements[i];
</del><ins>+ for (auto& associatedElement : m_associatedElements) {
</ins><span class="cx"> if (!is<HTMLFormControlElement>(*associatedElement))
</span><span class="cx"> continue;
</span><span class="cx"> if (needButtonActivation) {
</span><span class="lines">@@ -382,9 +379,9 @@
</span><span class="cx"> return;
</span><span class="cx"> }
</span><span class="cx">
</span><del>- for (unsigned i = 0; i < m_associatedElements.size(); ++i) {
- if (is<HTMLFormControlElement>(*m_associatedElements[i]))
- downcast<HTMLFormControlElement>(*m_associatedElements[i]).reset();
</del><ins>+ for (auto& associatedElement : m_associatedElements) {
+ if (is<HTMLFormControlElement>(*associatedElement))
+ downcast<HTMLFormControlElement>(*associatedElement).reset();
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> m_isInResetFunction = false;
</span><span class="lines">@@ -692,10 +689,10 @@
</span><span class="cx">
</span><span class="cx"> HTMLFormControlElement* HTMLFormElement::defaultButton() const
</span><span class="cx"> {
</span><del>- for (unsigned i = 0; i < m_associatedElements.size(); ++i) {
- if (!is<HTMLFormControlElement>(*m_associatedElements[i]))
</del><ins>+ for (auto& associatedElement : m_associatedElements) {
+ if (!is<HTMLFormControlElement>(*associatedElement))
</ins><span class="cx"> continue;
</span><del>- HTMLFormControlElement& control = downcast<HTMLFormControlElement>(*m_associatedElements[i]);
</del><ins>+ HTMLFormControlElement& control = downcast<HTMLFormControlElement>(*associatedElement);
</ins><span class="cx"> if (control.isSuccessfulSubmitButton())
</span><span class="cx"> return &control;
</span><span class="cx"> }
</span><span class="lines">@@ -716,12 +713,12 @@
</span><span class="cx"> // HTMLFormControlElement::checkValidity() might change m_associatedElements.
</span><span class="cx"> Vector<RefPtr<FormAssociatedElement>> elements;
</span><span class="cx"> elements.reserveCapacity(m_associatedElements.size());
</span><del>- for (unsigned i = 0; i < m_associatedElements.size(); ++i)
- elements.append(m_associatedElements[i]);
</del><ins>+ for (auto& associatedElement : m_associatedElements)
+ elements.append(associatedElement);
</ins><span class="cx"> bool hasInvalidControls = false;
</span><del>- for (unsigned i = 0; i < elements.size(); ++i) {
- if (elements[i]->form() == this && is<HTMLFormControlElement>(*elements[i])) {
- HTMLFormControlElement& control = downcast<HTMLFormControlElement>(*elements[i]);
</del><ins>+ for (auto& element : elements) {
+ if (element->form() == this && is<HTMLFormControlElement>(*element)) {
+ HTMLFormControlElement& control = downcast<HTMLFormControlElement>(*element);
</ins><span class="cx"> if (!control.checkValidity(&unhandledInvalidControls) && control.form() == this)
</span><span class="cx"> hasInvalidControls = true;
</span><span class="cx"> }
</span><span class="lines">@@ -777,10 +774,9 @@
</span><span class="cx"> if (!m_pastNamesMap)
</span><span class="cx"> return;
</span><span class="cx">
</span><del>- PastNamesMap::iterator end = m_pastNamesMap->end();
- for (PastNamesMap::iterator it = m_pastNamesMap->begin(); it != end; ++it) {
- if (it->value == item)
- it->value = 0; // Keep looping. Single element can have multiple names.
</del><ins>+ for (auto& pastName : m_pastNamesMap->values()) {
+ if (pastName == item)
+ pastName = nullptr; // Keep looping. Single element can have multiple names.
</ins><span class="cx"> }
</span><span class="cx"> }
</span><span class="cx">
</span><span class="lines">@@ -813,9 +809,9 @@
</span><span class="cx"> {
</span><span class="cx"> ASSERT(!shouldAutocomplete());
</span><span class="cx">
</span><del>- for (unsigned i = 0; i < m_associatedElements.size(); ++i) {
- if (is<HTMLFormControlElement>(*m_associatedElements[i]))
- downcast<HTMLFormControlElement>(*m_associatedElements[i]).reset();
</del><ins>+ for (auto& associatedElement : m_associatedElements) {
+ if (is<HTMLFormControlElement>(*associatedElement))
+ downcast<HTMLFormControlElement>(*associatedElement).reset();
</ins><span class="cx"> }
</span><span class="cx"> }
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlHTMLInputElementcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/HTMLInputElement.cpp (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/HTMLInputElement.cpp 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/html/HTMLInputElement.cpp 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -1224,8 +1224,8 @@
</span><span class="cx">
</span><span class="cx"> Vector<String> splitTypes;
</span><span class="cx"> acceptString.split(',', false, splitTypes);
</span><del>- for (size_t i = 0; i < splitTypes.size(); ++i) {
- String trimmedType = stripLeadingAndTrailingHTMLSpaces(splitTypes[i]);
</del><ins>+ for (auto& splitType : splitTypes) {
+ String trimmedType = stripLeadingAndTrailingHTMLSpaces(splitType);
</ins><span class="cx"> if (trimmedType.isEmpty())
</span><span class="cx"> continue;
</span><span class="cx"> if (!predicate(trimmedType))
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlHTMLKeygenElementcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/HTMLKeygenElement.cpp (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/HTMLKeygenElement.cpp 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/html/HTMLKeygenElement.cpp 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -74,10 +74,10 @@
</span><span class="cx"> getSupportedKeySizes(keys);
</span><span class="cx">
</span><span class="cx"> Ref<HTMLSelectElement> select = KeygenSelectElement::create(document);
</span><del>- for (size_t i = 0; i < keys.size(); ++i) {
</del><ins>+ for (auto& key : keys) {
</ins><span class="cx"> Ref<HTMLOptionElement> option = HTMLOptionElement::create(document);
</span><span class="cx"> select->appendChild(option.copyRef(), IGNORE_EXCEPTION);
</span><del>- option->appendChild(Text::create(document, keys[i]), IGNORE_EXCEPTION);
</del><ins>+ option->appendChild(Text::create(document, key), IGNORE_EXCEPTION);
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> ensureUserAgentShadowRoot().appendChild(WTF::move(select), IGNORE_EXCEPTION);
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlHTMLMediaElementcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/HTMLMediaElement.cpp (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/HTMLMediaElement.cpp 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/html/HTMLMediaElement.cpp 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -5724,15 +5724,15 @@
</span><span class="cx"> // 4. If there is another media element whose Document is the same as m's Document (even if one or both
</span><span class="cx"> // of these elements are not actually in the Document),
</span><span class="cx"> HashSet<HTMLMediaElement*> elements = documentToElementSetMap().get(&document());
</span><del>- for (HashSet<HTMLMediaElement*>::iterator i = elements.begin(); i != elements.end(); ++i) {
- if (*i == this)
</del><ins>+ for (auto& element : elements) {
+ if (element == this)
</ins><span class="cx"> continue;
</span><span class="cx">
</span><span class="cx"> // and which also has a mediagroup attribute, and whose mediagroup attribute has the same value as
</span><span class="cx"> // the new value of m's mediagroup attribute,
</span><del>- if ((*i)->mediaGroup() == group) {
</del><ins>+ if (element->mediaGroup() == group) {
</ins><span class="cx"> // then let controller be that media element's current media controller.
</span><del>- setController((*i)->controller());
</del><ins>+ setController(element->controller());
</ins><span class="cx"> return;
</span><span class="cx"> }
</span><span class="cx"> }
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlHTMLOptionElementcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/HTMLOptionElement.cpp (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/HTMLOptionElement.cpp 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/html/HTMLOptionElement.cpp 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -156,12 +156,10 @@
</span><span class="cx">
</span><span class="cx"> int optionIndex = 0;
</span><span class="cx">
</span><del>- const Vector<HTMLElement*>& items = selectElement->listItems();
- size_t length = items.size();
- for (size_t i = 0; i < length; ++i) {
- if (!is<HTMLOptionElement>(*items[i]))
</del><ins>+ for (auto& item : selectElement->listItems()) {
+ if (!is<HTMLOptionElement>(*item))
</ins><span class="cx"> continue;
</span><del>- if (items[i] == this)
</del><ins>+ if (item == this)
</ins><span class="cx"> return optionIndex;
</span><span class="cx"> ++optionIndex;
</span><span class="cx"> }
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlHTMLSelectElementcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/HTMLSelectElement.cpp (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/HTMLSelectElement.cpp 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/html/HTMLSelectElement.cpp 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -608,11 +608,8 @@
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> m_lastOnChangeSelection.clear();
</span><del>- const Vector<HTMLElement*>& items = listItems();
- for (unsigned i = 0; i < items.size(); ++i) {
- HTMLElement* element = items[i];
</del><ins>+ for (auto& element : listItems())
</ins><span class="cx"> m_lastOnChangeSelection.append(is<HTMLOptionElement>(*element) && downcast<HTMLOptionElement>(*element).selected());
</span><del>- }
</del><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void HTMLSelectElement::setActiveSelectionAnchorIndex(int index)
</span><span class="lines">@@ -623,11 +620,8 @@
</span><span class="cx"> // selection pivots around this anchor index.
</span><span class="cx"> m_cachedStateForActiveSelection.clear();
</span><span class="cx">
</span><del>- const Vector<HTMLElement*>& items = listItems();
- for (unsigned i = 0; i < items.size(); ++i) {
- HTMLElement* element = items[i];
</del><ins>+ for (auto& element : listItems())
</ins><span class="cx"> m_cachedStateForActiveSelection.append(is<HTMLOptionElement>(*element) && downcast<HTMLOptionElement>(*element).selected());
</span><del>- }
</del><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void HTMLSelectElement::setActiveSelectionEndIndex(int index)
</span><span class="lines">@@ -843,9 +837,7 @@
</span><span class="cx"> unsigned index = 0;
</span><span class="cx">
</span><span class="cx"> // Return the number of the first option selected.
</span><del>- const Vector<HTMLElement*>& items = listItems();
- for (size_t i = 0; i < items.size(); ++i) {
- HTMLElement* element = items[i];
</del><ins>+ for (auto& element : listItems()) {
</ins><span class="cx"> if (is<HTMLOptionElement>(*element)) {
</span><span class="cx"> if (downcast<HTMLOptionElement>(*element).selected())
</span><span class="cx"> return index;
</span><span class="lines">@@ -971,9 +963,7 @@
</span><span class="cx">
</span><span class="cx"> void HTMLSelectElement::deselectItemsWithoutValidation(HTMLElement* excludeElement)
</span><span class="cx"> {
</span><del>- const Vector<HTMLElement*>& items = listItems();
- for (unsigned i = 0; i < items.size(); ++i) {
- HTMLElement* element = items[i];
</del><ins>+ for (auto& element : listItems()) {
</ins><span class="cx"> if (element != excludeElement && is<HTMLOptionElement>(*element))
</span><span class="cx"> downcast<HTMLOptionElement>(*element).setSelectedState(false);
</span><span class="cx"> }
</span><span class="lines">@@ -981,13 +971,11 @@
</span><span class="cx">
</span><span class="cx"> FormControlState HTMLSelectElement::saveFormControlState() const
</span><span class="cx"> {
</span><del>- const Vector<HTMLElement*>& items = listItems();
- size_t length = items.size();
</del><span class="cx"> FormControlState state;
</span><del>- for (unsigned i = 0; i < length; ++i) {
- if (!is<HTMLOptionElement>(*items[i]))
</del><ins>+ for (auto& element : listItems()) {
+ if (!is<HTMLOptionElement>(*element))
</ins><span class="cx"> continue;
</span><del>- HTMLOptionElement& option = downcast<HTMLOptionElement>(*items[i]);
</del><ins>+ HTMLOptionElement& option = downcast<HTMLOptionElement>(*element);
</ins><span class="cx"> if (!option.selected())
</span><span class="cx"> continue;
</span><span class="cx"> state.append(option.value());
</span><span class="lines">@@ -1019,10 +1007,10 @@
</span><span class="cx"> if (!itemsSize)
</span><span class="cx"> return;
</span><span class="cx">
</span><del>- for (size_t i = 0; i < itemsSize; ++i) {
- if (!is<HTMLOptionElement>(*items[i]))
</del><ins>+ for (auto& element : items) {
+ if (!is<HTMLOptionElement>(*element))
</ins><span class="cx"> continue;
</span><del>- downcast<HTMLOptionElement>(*items[i]).setSelectedState(false);
</del><ins>+ downcast<HTMLOptionElement>(*element).setSelectedState(false);
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> if (!multiple()) {
</span><span class="lines">@@ -1063,10 +1051,7 @@
</span><span class="cx"> return false;
</span><span class="cx">
</span><span class="cx"> bool successful = false;
</span><del>- const Vector<HTMLElement*>& items = listItems();
-
- for (unsigned i = 0; i < items.size(); ++i) {
- HTMLElement* element = items[i];
</del><ins>+ for (auto& element : listItems()) {
</ins><span class="cx"> if (is<HTMLOptionElement>(*element) && downcast<HTMLOptionElement>(*element).selected() && !downcast<HTMLOptionElement>(*element).isDisabledFormControl()) {
</span><span class="cx"> list.appendData(name, downcast<HTMLOptionElement>(*element).value());
</span><span class="cx"> successful = true;
</span><span class="lines">@@ -1084,9 +1069,7 @@
</span><span class="cx"> HTMLOptionElement* firstOption = nullptr;
</span><span class="cx"> HTMLOptionElement* selectedOption = nullptr;
</span><span class="cx">
</span><del>- const Vector<HTMLElement*>& items = listItems();
- for (unsigned i = 0; i < items.size(); ++i) {
- HTMLElement* element = items[i];
</del><ins>+ for (auto& element : listItems()) {
</ins><span class="cx"> if (!is<HTMLOptionElement>(*element))
</span><span class="cx"> continue;
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlHTMLTextAreaElementcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/HTMLTextAreaElement.cpp (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/HTMLTextAreaElement.cpp 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/html/HTMLTextAreaElement.cpp 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -410,9 +410,8 @@
</span><span class="cx"> for (Text* textNode = TextNodeTraversal::firstChild(*this); textNode; textNode = TextNodeTraversal::nextSibling(*textNode))
</span><span class="cx"> textNodes.append(*textNode);
</span><span class="cx">
</span><del>- size_t size = textNodes.size();
- for (size_t i = 0; i < size; ++i)
- removeChild(textNodes[i].get(), IGNORE_EXCEPTION);
</del><ins>+ for (auto& textNode : textNodes)
+ removeChild(textNode.get(), IGNORE_EXCEPTION);
</ins><span class="cx">
</span><span class="cx"> // Normalize line endings.
</span><span class="cx"> String value = defaultValue;
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlInputTypecpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/InputType.cpp (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/InputType.cpp 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/html/InputType.cpp 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -140,10 +140,10 @@
</span><span class="cx"> // No need to register "text" because it is the default type.
</span><span class="cx"> };
</span><span class="cx">
</span><del>- for (unsigned i = 0; i < WTF_ARRAY_LENGTH(inputTypes); ++i) {
- auto conditionalFunction = inputTypes[i].conditionalFunction;
</del><ins>+ for (auto& inputType : inputTypes) {
+ auto conditionalFunction = inputType.conditionalFunction;
</ins><span class="cx"> if (!conditionalFunction || (RuntimeEnabledFeatures::sharedFeatures().*conditionalFunction)())
</span><del>- map.add(inputTypes[i].nameFunction(), inputTypes[i].factoryFunction);
</del><ins>+ map.add(inputType.nameFunction(), inputType.factoryFunction);
</ins><span class="cx"> }
</span><span class="cx"> }
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlMediaControllercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/MediaController.cpp (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/MediaController.cpp 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/html/MediaController.cpp 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -132,8 +132,8 @@
</span><span class="cx"> // FIXME: Investigate caching the maximum duration and only updating the cached value
</span><span class="cx"> // when the slaved media elements' durations change.
</span><span class="cx"> double maxDuration = 0;
</span><del>- for (size_t index = 0; index < m_mediaElements.size(); ++index) {
- double duration = m_mediaElements[index]->duration();
</del><ins>+ for (auto& mediaElement : m_mediaElements) {
+ double duration = mediaElement->duration();
</ins><span class="cx"> if (std::isnan(duration))
</span><span class="cx"> continue;
</span><span class="cx"> maxDuration = std::max(maxDuration, duration);
</span><span class="lines">@@ -170,8 +170,8 @@
</span><span class="cx"> m_clock->setCurrentTime(time);
</span><span class="cx">
</span><span class="cx"> // Seek each slaved media element to the new playback position relative to the media element timeline.
</span><del>- for (size_t index = 0; index < m_mediaElements.size(); ++index)
- m_mediaElements[index]->seek(MediaTime::createWithDouble(time));
</del><ins>+ for (auto& mediaElement : m_mediaElements)
+ mediaElement->seek(MediaTime::createWithDouble(time));
</ins><span class="cx">
</span><span class="cx"> scheduleTimeupdateEvent();
</span><span class="cx"> m_resetCurrentTimeInNextPlay = false;
</span><span class="lines">@@ -194,8 +194,8 @@
</span><span class="cx"> {
</span><span class="cx"> // When the play() method is invoked, the user agent must invoke the play method of each
</span><span class="cx"> // slaved media element in turn,
</span><del>- for (size_t index = 0; index < m_mediaElements.size(); ++index)
- m_mediaElements[index]->play();
</del><ins>+ for (auto& mediaElement : m_mediaElements)
+ mediaElement->play();
</ins><span class="cx">
</span><span class="cx"> // and then invoke the unpause method of the MediaController.
</span><span class="cx"> unpause();
</span><span class="lines">@@ -242,8 +242,8 @@
</span><span class="cx"> // playback rate to the new value,
</span><span class="cx"> m_clock->setPlayRate(rate);
</span><span class="cx">
</span><del>- for (size_t index = 0; index < m_mediaElements.size(); ++index)
- m_mediaElements[index]->updatePlaybackRate();
</del><ins>+ for (auto& mediaElement : m_mediaElements)
+ mediaElement->updatePlaybackRate();
</ins><span class="cx">
</span><span class="cx"> // then queue a task to fire a simple event named ratechange at the MediaController.
</span><span class="cx"> scheduleEvent(eventNames().ratechangeEvent);
</span><span class="lines">@@ -268,8 +268,8 @@
</span><span class="cx"> // and queue a task to fire a simple event named volumechange at the MediaController.
</span><span class="cx"> scheduleEvent(eventNames().volumechangeEvent);
</span><span class="cx">
</span><del>- for (size_t index = 0; index < m_mediaElements.size(); ++index)
- m_mediaElements[index]->updateVolume();
</del><ins>+ for (auto& mediaElement : m_mediaElements)
+ mediaElement->updateVolume();
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void MediaController::setMuted(bool flag)
</span><span class="lines">@@ -284,8 +284,8 @@
</span><span class="cx"> // and queue a task to fire a simple event named volumechange at the MediaController.
</span><span class="cx"> scheduleEvent(eventNames().volumechangeEvent);
</span><span class="cx">
</span><del>- for (size_t index = 0; index < m_mediaElements.size(); ++index)
- m_mediaElements[index]->updateVolume();
</del><ins>+ for (auto& mediaElement : m_mediaElements)
+ mediaElement->updateVolume();
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> static const AtomicString& playbackStateWaiting()
</span><span class="lines">@@ -472,8 +472,8 @@
</span><span class="cx">
</span><span class="cx"> void MediaController::updateMediaElements()
</span><span class="cx"> {
</span><del>- for (size_t index = 0; index < m_mediaElements.size(); ++index)
- m_mediaElements[index]->updatePlayState();
</del><ins>+ for (auto& mediaElement : m_mediaElements)
+ mediaElement->updatePlayState();
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void MediaController::bringElementUpToSpeed(HTMLMediaElement* element)
</span><span class="lines">@@ -498,8 +498,7 @@
</span><span class="cx"> return false;
</span><span class="cx">
</span><span class="cx"> bool allPaused = true;
</span><del>- for (size_t index = 0; index < m_mediaElements.size(); ++index) {
- HTMLMediaElement* element = m_mediaElements[index];
</del><ins>+ for (auto& element : m_mediaElements) {
</ins><span class="cx"> // or if any of its slaved media elements are blocked media elements,
</span><span class="cx"> if (element->isBlocked())
</span><span class="cx"> return true;
</span><span class="lines">@@ -529,8 +528,8 @@
</span><span class="cx"> return false;
</span><span class="cx">
</span><span class="cx"> bool allHaveEnded = true;
</span><del>- for (size_t index = 0; index < m_mediaElements.size(); ++index) {
- if (!m_mediaElements[index]->ended())
</del><ins>+ for (auto& mediaElement : m_mediaElements) {
+ if (!mediaElement->ended())
</ins><span class="cx"> allHaveEnded = false;
</span><span class="cx"> }
</span><span class="cx"> return allHaveEnded;
</span><span class="lines">@@ -548,9 +547,8 @@
</span><span class="cx"> Vector<RefPtr<Event>> pendingEvents;
</span><span class="cx">
</span><span class="cx"> m_pendingEvents.swap(pendingEvents);
</span><del>- size_t count = pendingEvents.size();
- for (size_t index = 0; index < count; ++index)
- dispatchEvent(pendingEvents[index].release(), IGNORE_EXCEPTION);
</del><ins>+ for (auto& pendingEvent : pendingEvents)
+ dispatchEvent(pendingEvent.release(), IGNORE_EXCEPTION);
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void MediaController::clearPositionTimerFired()
</span><span class="lines">@@ -560,8 +558,8 @@
</span><span class="cx">
</span><span class="cx"> bool MediaController::hasAudio() const
</span><span class="cx"> {
</span><del>- for (size_t index = 0; index < m_mediaElements.size(); ++index) {
- if (m_mediaElements[index]->hasAudio())
</del><ins>+ for (auto& mediaElement : m_mediaElements) {
+ if (mediaElement->hasAudio())
</ins><span class="cx"> return true;
</span><span class="cx"> }
</span><span class="cx"> return false;
</span><span class="lines">@@ -569,8 +567,8 @@
</span><span class="cx">
</span><span class="cx"> bool MediaController::hasVideo() const
</span><span class="cx"> {
</span><del>- for (size_t index = 0; index < m_mediaElements.size(); ++index) {
- if (m_mediaElements[index]->hasVideo())
</del><ins>+ for (auto& mediaElement : m_mediaElements) {
+ if (mediaElement->hasVideo())
</ins><span class="cx"> return true;
</span><span class="cx"> }
</span><span class="cx"> return false;
</span><span class="lines">@@ -578,8 +576,8 @@
</span><span class="cx">
</span><span class="cx"> bool MediaController::hasClosedCaptions() const
</span><span class="cx"> {
</span><del>- for (size_t index = 0; index < m_mediaElements.size(); ++index) {
- if (m_mediaElements[index]->hasClosedCaptions())
</del><ins>+ for (auto& mediaElement : m_mediaElements) {
+ if (mediaElement->hasClosedCaptions())
</ins><span class="cx"> return true;
</span><span class="cx"> }
</span><span class="cx"> return false;
</span><span class="lines">@@ -588,14 +586,14 @@
</span><span class="cx"> void MediaController::setClosedCaptionsVisible(bool visible)
</span><span class="cx"> {
</span><span class="cx"> m_closedCaptionsVisible = visible;
</span><del>- for (size_t index = 0; index < m_mediaElements.size(); ++index)
- m_mediaElements[index]->setClosedCaptionsVisible(visible);
</del><ins>+ for (auto& mediaElement : m_mediaElements)
+ mediaElement->setClosedCaptionsVisible(visible);
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> bool MediaController::supportsScanning() const
</span><span class="cx"> {
</span><del>- for (size_t index = 0; index < m_mediaElements.size(); ++index) {
- if (!m_mediaElements[index]->supportsScanning())
</del><ins>+ for (auto& mediaElement : m_mediaElements) {
+ if (!mediaElement->supportsScanning())
</ins><span class="cx"> return false;
</span><span class="cx"> }
</span><span class="cx"> return true;
</span><span class="lines">@@ -603,16 +601,16 @@
</span><span class="cx">
</span><span class="cx"> void MediaController::beginScrubbing()
</span><span class="cx"> {
</span><del>- for (size_t index = 0; index < m_mediaElements.size(); ++index)
- m_mediaElements[index]->beginScrubbing();
</del><ins>+ for (auto& mediaElement : m_mediaElements)
+ mediaElement->beginScrubbing();
</ins><span class="cx"> if (m_playbackState == PLAYING)
</span><span class="cx"> m_clock->stop();
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void MediaController::endScrubbing()
</span><span class="cx"> {
</span><del>- for (size_t index = 0; index < m_mediaElements.size(); ++index)
- m_mediaElements[index]->endScrubbing();
</del><ins>+ for (auto& mediaElement : m_mediaElements)
+ mediaElement->endScrubbing();
</ins><span class="cx"> if (m_playbackState == PLAYING)
</span><span class="cx"> m_clock->start();
</span><span class="cx"> }
</span><span class="lines">@@ -634,8 +632,8 @@
</span><span class="cx"> if (m_paused)
</span><span class="cx"> return true;
</span><span class="cx">
</span><del>- for (size_t index = 0; index < m_mediaElements.size(); ++index) {
- if (!m_mediaElements[index]->canPlay())
</del><ins>+ for (auto& mediaElement : m_mediaElements) {
+ if (!mediaElement->canPlay())
</ins><span class="cx"> return false;
</span><span class="cx"> }
</span><span class="cx"> return true;
</span><span class="lines">@@ -643,8 +641,8 @@
</span><span class="cx">
</span><span class="cx"> bool MediaController::isLiveStream() const
</span><span class="cx"> {
</span><del>- for (size_t index = 0; index < m_mediaElements.size(); ++index) {
- if (!m_mediaElements[index]->isLiveStream())
</del><ins>+ for (auto& mediaElement : m_mediaElements) {
+ if (!mediaElement->isLiveStream())
</ins><span class="cx"> return false;
</span><span class="cx"> }
</span><span class="cx"> return true;
</span><span class="lines">@@ -652,8 +650,8 @@
</span><span class="cx">
</span><span class="cx"> bool MediaController::hasCurrentSrc() const
</span><span class="cx"> {
</span><del>- for (size_t index = 0; index < m_mediaElements.size(); ++index) {
- if (!m_mediaElements[index]->hasCurrentSrc())
</del><ins>+ for (auto& mediaElement : m_mediaElements) {
+ if (!mediaElement->hasCurrentSrc())
</ins><span class="cx"> return false;
</span><span class="cx"> }
</span><span class="cx"> return true;
</span><span class="lines">@@ -661,8 +659,8 @@
</span><span class="cx">
</span><span class="cx"> void MediaController::returnToRealtime()
</span><span class="cx"> {
</span><del>- for (size_t index = 0; index < m_mediaElements.size(); ++index)
- m_mediaElements[index]->returnToRealtime();
</del><ins>+ for (auto& mediaElement : m_mediaElements)
+ mediaElement->returnToRealtime();
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> // The spec says to fire periodic timeupdate events (those sent while playing) every
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlMediaFragmentURIParsercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/MediaFragmentURIParser.cpp (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/MediaFragmentURIParser.cpp 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/html/MediaFragmentURIParser.cpp 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -161,9 +161,7 @@
</span><span class="cx">
</span><span class="cx"> m_timeFormat = Invalid;
</span><span class="cx">
</span><del>- for (unsigned i = 0; i < m_fragments.size(); ++i) {
- std::pair<String, String>& fragment = m_fragments[i];
-
</del><ins>+ for (auto& fragment : m_fragments) {
</ins><span class="cx"> ASSERT(fragment.first.is8Bit());
</span><span class="cx"> ASSERT(fragment.second.is8Bit());
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlPublicURLManagercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/PublicURLManager.cpp (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/PublicURLManager.cpp 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/html/PublicURLManager.cpp 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -57,10 +57,10 @@
</span><span class="cx">
</span><span class="cx"> void PublicURLManager::revoke(const URL& url)
</span><span class="cx"> {
</span><del>- for (RegistryURLMap::iterator i = m_registryToURL.begin(); i != m_registryToURL.end(); ++i) {
- if (i->value.contains(url.string())) {
- i->key->unregisterURL(url);
- i->value.remove(url.string());
</del><ins>+ for (auto& registry : m_registryToURL) {
+ if (registry.value.contains(url.string())) {
+ registry.key->unregisterURL(url);
+ registry.value.remove(url.string());
</ins><span class="cx"> break;
</span><span class="cx"> }
</span><span class="cx"> }
</span><span class="lines">@@ -72,9 +72,9 @@
</span><span class="cx"> return;
</span><span class="cx">
</span><span class="cx"> m_isStopped = true;
</span><del>- for (RegistryURLMap::iterator i = m_registryToURL.begin(); i != m_registryToURL.end(); ++i) {
- for (URLSet::iterator j = i->value.begin(); j != i->value.end(); ++j)
- i->key->unregisterURL(URL(ParsedURLString, *j));
</del><ins>+ for (auto& registry : m_registryToURL) {
+ for (auto& url : registry.value)
+ registry.key->unregisterURL(URL(ParsedURLString, url));
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> m_registryToURL.clear();
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlcanvasWebGLBuffercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/canvas/WebGLBuffer.cpp (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/canvas/WebGLBuffer.cpp 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/html/canvas/WebGLBuffer.cpp 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -166,23 +166,24 @@
</span><span class="cx">
</span><span class="cx"> int WebGLBuffer::getCachedMaxIndex(GC3Denum type)
</span><span class="cx"> {
</span><del>- for (size_t i = 0; i < WTF_ARRAY_LENGTH(m_maxIndexCache); ++i)
- if (m_maxIndexCache[i].type == type)
- return m_maxIndexCache[i].maxIndex;
</del><ins>+ for (auto& cache : m_maxIndexCache) {
+ if (cache.type == type)
+ return cache.maxIndex;
+ }
</ins><span class="cx"> return -1;
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void WebGLBuffer::setCachedMaxIndex(GC3Denum type, int value)
</span><span class="cx"> {
</span><del>- size_t numEntries = WTF_ARRAY_LENGTH(m_maxIndexCache);
- for (size_t i = 0; i < numEntries; ++i)
- if (m_maxIndexCache[i].type == type) {
- m_maxIndexCache[i].maxIndex = value;
</del><ins>+ for (auto& cache : m_maxIndexCache) {
+ if (cache.type == type) {
+ cache.maxIndex = value;
</ins><span class="cx"> return;
</span><span class="cx"> }
</span><ins>+ }
</ins><span class="cx"> m_maxIndexCache[m_nextAvailableCacheEntry].type = type;
</span><span class="cx"> m_maxIndexCache[m_nextAvailableCacheEntry].maxIndex = value;
</span><del>- m_nextAvailableCacheEntry = (m_nextAvailableCacheEntry + 1) % numEntries;
</del><ins>+ m_nextAvailableCacheEntry = (m_nextAvailableCacheEntry + 1) % WTF_ARRAY_LENGTH(m_maxIndexCache);
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void WebGLBuffer::setTarget(GC3Denum target)
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlcanvasWebGLContextGroupcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/canvas/WebGLContextGroup.cpp (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/canvas/WebGLContextGroup.cpp 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/html/canvas/WebGLContextGroup.cpp 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -90,8 +90,8 @@
</span><span class="cx">
</span><span class="cx"> void WebGLContextGroup::loseContextGroup(WebGLRenderingContextBase::LostContextMode mode)
</span><span class="cx"> {
</span><del>- for (HashSet<WebGLRenderingContextBase*>::iterator it = m_contexts.begin(); it != m_contexts.end(); ++it)
- (*it)->loseContextImpl(mode);
</del><ins>+ for (auto& context : m_contexts)
+ context->loseContextImpl(mode);
</ins><span class="cx">
</span><span class="cx"> detachAndRemoveAllObjects();
</span><span class="cx"> }
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlcanvasWebGLDrawBufferscpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/canvas/WebGLDrawBuffers.cpp (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/canvas/WebGLDrawBuffers.cpp 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/html/canvas/WebGLDrawBuffers.cpp 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -173,8 +173,8 @@
</span><span class="cx"> context->deleteTexture(depth);
</span><span class="cx"> if (supportsDepthStencil)
</span><span class="cx"> context->deleteTexture(depthStencil);
</span><del>- for (size_t i = 0; i < colors.size(); ++i)
- context->deleteTexture(colors[i]);
</del><ins>+ for (auto& color : colors)
+ context->deleteTexture(color);
</ins><span class="cx"> return ok;
</span><span class="cx"> }
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlcanvasWebGLFramebuffercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/canvas/WebGLFramebuffer.cpp (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/canvas/WebGLFramebuffer.cpp 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/html/canvas/WebGLFramebuffer.cpp 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -371,10 +371,10 @@
</span><span class="cx"> bool checkMore = true;
</span><span class="cx"> do {
</span><span class="cx"> checkMore = false;
</span><del>- for (AttachmentMap::iterator it = m_attachments.begin(); it != m_attachments.end(); ++it) {
- WebGLAttachment* attachmentObject = it->value.get();
</del><ins>+ for (auto& entry : m_attachments) {
+ WebGLAttachment* attachmentObject = entry.value.get();
</ins><span class="cx"> if (attachmentObject->isSharedObject(attachment)) {
</span><del>- GC3Denum attachmentType = it->key;
</del><ins>+ GC3Denum attachmentType = entry.key;
</ins><span class="cx"> attachmentObject->unattach(context()->graphicsContext3D(), attachmentType);
</span><span class="cx"> removeAttachmentFromBoundFramebuffer(attachmentType);
</span><span class="cx"> checkMore = true;
</span><span class="lines">@@ -423,9 +423,9 @@
</span><span class="cx"> bool haveDepth = false;
</span><span class="cx"> bool haveStencil = false;
</span><span class="cx"> bool haveDepthStencil = false;
</span><del>- for (AttachmentMap::const_iterator it = m_attachments.begin(); it != m_attachments.end(); ++it) {
- WebGLAttachment* attachment = it->value.get();
- if (!isAttachmentComplete(attachment, it->key, reason))
</del><ins>+ for (auto& entry : m_attachments) {
+ WebGLAttachment* attachment = entry.value.get();
+ if (!isAttachmentComplete(attachment, entry.key, reason))
</ins><span class="cx"> return GraphicsContext3D::FRAMEBUFFER_INCOMPLETE_ATTACHMENT;
</span><span class="cx"> if (!attachment->isValid()) {
</span><span class="cx"> *reason = "attachment is not valid";
</span><span class="lines">@@ -441,7 +441,7 @@
</span><span class="cx"> *reason = "attachment is an unsupported format";
</span><span class="cx"> return GraphicsContext3D::FRAMEBUFFER_INCOMPLETE_ATTACHMENT;
</span><span class="cx"> }
</span><del>- switch (it->key) {
</del><ins>+ switch (entry.key) {
</ins><span class="cx"> case GraphicsContext3D::DEPTH_ATTACHMENT:
</span><span class="cx"> haveDepth = true;
</span><span class="cx"> break;
</span><span class="lines">@@ -498,8 +498,8 @@
</span><span class="cx">
</span><span class="cx"> void WebGLFramebuffer::deleteObjectImpl(GraphicsContext3D* context3d, Platform3DObject object)
</span><span class="cx"> {
</span><del>- for (AttachmentMap::iterator it = m_attachments.begin(); it != m_attachments.end(); ++it)
- it->value->onDetached(context3d);
</del><ins>+ for (auto& attachment : m_attachments.values())
+ attachment->onDetached(context3d);
</ins><span class="cx">
</span><span class="cx"> context3d->deleteFramebuffer(object);
</span><span class="cx"> }
</span><span class="lines">@@ -509,9 +509,9 @@
</span><span class="cx"> ASSERT(object());
</span><span class="cx"> GC3Dbitfield mask = 0;
</span><span class="cx">
</span><del>- for (AttachmentMap::iterator it = m_attachments.begin(); it != m_attachments.end(); ++it) {
- GC3Denum attachmentType = it->key;
- WebGLAttachment* attachment = it->value.get();
</del><ins>+ for (auto& entry : m_attachments) {
+ GC3Denum attachmentType = entry.key;
+ WebGLAttachment* attachment = entry.value.get();
</ins><span class="cx"> if (!attachment->isInitialized())
</span><span class="cx"> mask |= GraphicsContext3D::getClearBitsByAttachmentType(attachmentType);
</span><span class="cx"> }
</span><span class="lines">@@ -600,8 +600,8 @@
</span><span class="cx"> {
</span><span class="cx"> m_drawBuffers = bufs;
</span><span class="cx"> m_filteredDrawBuffers.resize(m_drawBuffers.size());
</span><del>- for (size_t i = 0; i < m_filteredDrawBuffers.size(); ++i)
- m_filteredDrawBuffers[i] = GraphicsContext3D::NONE;
</del><ins>+ for (auto& buffer : m_filteredDrawBuffers)
+ buffer = GraphicsContext3D::NONE;
</ins><span class="cx"> drawBuffersIfNecessary(true);
</span><span class="cx"> }
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlcanvasWebGLRenderingContextBasecpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/canvas/WebGLRenderingContextBase.cpp (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/canvas/WebGLRenderingContextBase.cpp 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/html/canvas/WebGLRenderingContextBase.cpp 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -594,9 +594,9 @@
</span><span class="cx"> m_framebufferBinding = nullptr;
</span><span class="cx"> m_renderbufferBinding = nullptr;
</span><span class="cx">
</span><del>- for (size_t i = 0; i < m_textureUnits.size(); ++i) {
- m_textureUnits[i].texture2DBinding = nullptr;
- m_textureUnits[i].textureCubeMapBinding = nullptr;
</del><ins>+ for (auto& textureUnit : m_textureUnits) {
+ textureUnit.texture2DBinding = nullptr;
+ textureUnit.textureCubeMapBinding = nullptr;
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> m_blackTexture2D = nullptr;
</span><span class="lines">@@ -1514,11 +1514,11 @@
</span><span class="cx"> {
</span><span class="cx"> if (!deleteObject(texture))
</span><span class="cx"> return;
</span><del>- for (size_t i = 0; i < m_textureUnits.size(); ++i) {
- if (texture == m_textureUnits[i].texture2DBinding)
- m_textureUnits[i].texture2DBinding = nullptr;
- if (texture == m_textureUnits[i].textureCubeMapBinding)
- m_textureUnits[i].textureCubeMapBinding = nullptr;
</del><ins>+ for (auto& textureUnit : m_textureUnits) {
+ if (texture == textureUnit.texture2DBinding)
+ textureUnit.texture2DBinding = nullptr;
+ if (texture == textureUnit.textureCubeMapBinding)
+ textureUnit.textureCubeMapBinding = nullptr;
</ins><span class="cx"> }
</span><span class="cx"> if (m_framebufferBinding)
</span><span class="cx"> m_framebufferBinding->removeAttachmentFromBoundFramebuffer(texture);
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlcanvasWebGLVertexArrayObjectcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/canvas/WebGLVertexArrayObject.cpp (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/canvas/WebGLVertexArrayObject.cpp 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/html/canvas/WebGLVertexArrayObject.cpp 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -68,8 +68,7 @@
</span><span class="cx"> if (m_boundElementArrayBuffer)
</span><span class="cx"> m_boundElementArrayBuffer->onDetached(context3d);
</span><span class="cx">
</span><del>- for (size_t i = 0; i < m_vertexAttribState.size(); ++i) {
- VertexAttribState& state = m_vertexAttribState[i];
</del><ins>+ for (auto& state : m_vertexAttribState) {
</ins><span class="cx"> if (state.bufferBinding)
</span><span class="cx"> state.bufferBinding->onDetached(context3d);
</span><span class="cx"> }
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlcanvasWebGLVertexArrayObjectOEScpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/canvas/WebGLVertexArrayObjectOES.cpp (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/canvas/WebGLVertexArrayObjectOES.cpp 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/html/canvas/WebGLVertexArrayObjectOES.cpp 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -71,8 +71,7 @@
</span><span class="cx"> if (m_boundElementArrayBuffer)
</span><span class="cx"> m_boundElementArrayBuffer->onDetached(context3d);
</span><span class="cx">
</span><del>- for (size_t i = 0; i < m_vertexAttribState.size(); ++i) {
- VertexAttribState& state = m_vertexAttribState[i];
</del><ins>+ for (auto& state : m_vertexAttribState) {
</ins><span class="cx"> if (state.bufferBinding)
</span><span class="cx"> state.bufferBinding->onDetached(context3d);
</span><span class="cx"> }
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlparserAtomicHTMLTokenh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/parser/AtomicHTMLToken.h (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/parser/AtomicHTMLToken.h 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/html/parser/AtomicHTMLToken.h 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -186,8 +186,7 @@
</span><span class="cx"> return;
</span><span class="cx">
</span><span class="cx"> m_attributes.reserveInitialCapacity(size);
</span><del>- for (unsigned i = 0; i < size; ++i) {
- const HTMLToken::Attribute& attribute = attributes[i];
</del><ins>+ for (auto& attribute : attributes) {
</ins><span class="cx"> if (attribute.name.isEmpty())
</span><span class="cx"> continue;
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlparserHTMLConstructionSitecpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/parser/HTMLConstructionSite.cpp (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/parser/HTMLConstructionSite.cpp 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/html/parser/HTMLConstructionSite.cpp 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -274,8 +274,7 @@
</span><span class="cx"> if (token->attributes().isEmpty())
</span><span class="cx"> return;
</span><span class="cx">
</span><del>- for (unsigned i = 0; i < token->attributes().size(); ++i) {
- const Attribute& tokenAttribute = token->attributes().at(i);
</del><ins>+ for (auto& tokenAttribute : token->attributes()) {
</ins><span class="cx"> if (!element->elementData() || !element->findAttributeByName(tokenAttribute.name()))
</span><span class="cx"> element->setAttribute(tokenAttribute.name(), tokenAttribute.value());
</span><span class="cx"> }
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlparserHTMLFormattingElementListcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/parser/HTMLFormattingElementList.cpp (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/parser/HTMLFormattingElementList.cpp 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/html/parser/HTMLFormattingElementList.cpp 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -170,13 +170,8 @@
</span><span class="cx"> Vector<HTMLStackItem*> remainingCandidates;
</span><span class="cx"> remainingCandidates.reserveInitialCapacity(candidates.size());
</span><span class="cx">
</span><del>- const Vector<Attribute>& attributes = newItem->attributes();
- for (size_t i = 0; i < attributes.size(); ++i) {
- const Attribute& attribute = attributes[i];
-
- for (size_t j = 0; j < candidates.size(); ++j) {
- HTMLStackItem* candidate = candidates[j];
-
</del><ins>+ for (auto& attribute : newItem->attributes()) {
+ for (auto& candidate : candidates) {
</ins><span class="cx"> // These properties should already have been checked by tryToEnsureNoahsArkConditionQuickly.
</span><span class="cx"> ASSERT(newItem->attributes().size() == candidate->attributes().size());
</span><span class="cx"> ASSERT(newItem->localName() == candidate->localName() && newItem->namespaceURI() == candidate->namespaceURI());
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlparserHTMLPreloadScannercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/parser/HTMLPreloadScanner.cpp (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/parser/HTMLPreloadScanner.cpp 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/html/parser/HTMLPreloadScanner.cpp 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -103,9 +103,9 @@
</span><span class="cx"> if (m_tagId >= TagId::Unknown)
</span><span class="cx"> return;
</span><span class="cx">
</span><del>- for (HTMLToken::AttributeList::const_iterator iter = attributes.begin(); iter != attributes.end(); ++iter) {
- AtomicString attributeName(iter->name);
- String attributeValue = StringImpl::create8BitIfPossible(iter->value);
</del><ins>+ for (auto& attribute : attributes) {
+ AtomicString attributeName(attribute.name);
+ String attributeValue = StringImpl::create8BitIfPossible(attribute.value);
</ins><span class="cx"> processAttribute(attributeName, attributeValue);
</span><span class="cx"> }
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlparserXSSAuditorcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/parser/XSSAuditor.cpp (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/parser/XSSAuditor.cpp 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/html/parser/XSSAuditor.cpp 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -197,8 +197,8 @@
</span><span class="cx"> {
</span><span class="cx"> Vector<String> valueList;
</span><span class="cx"> value.split(';', valueList);
</span><del>- for (size_t i = 0; i < valueList.size(); ++i) {
- if (protocolIsJavaScript(valueList[i]))
</del><ins>+ for (auto& value : valueList) {
+ if (protocolIsJavaScript(value))
</ins><span class="cx"> return true;
</span><span class="cx"> }
</span><span class="cx"> return false;
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlshadowContentDistributorcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/shadow/ContentDistributor.cpp (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/shadow/ContentDistributor.cpp 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/html/shadow/ContentDistributor.cpp 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -78,13 +78,11 @@
</span><span class="cx"> m_validity = Valid;
</span><span class="cx">
</span><span class="cx"> if (ShadowRoot* root = host->shadowRoot()) {
</span><del>- const Vector<RefPtr<InsertionPoint>>& insertionPoints = ensureInsertionPointList(root);
- for (size_t i = 0; i < insertionPoints.size(); ++i) {
- InsertionPoint* point = insertionPoints[i].get();
</del><ins>+ for (auto& point : ensureInsertionPointList(root)) {
</ins><span class="cx"> if (!point->isActive())
</span><span class="cx"> continue;
</span><span class="cx">
</span><del>- distributeSelectionsTo(point, host);
</del><ins>+ distributeSelectionsTo(point.get(), host);
</ins><span class="cx"> }
</span><span class="cx"> }
</span><span class="cx"> }
</span><span class="lines">@@ -95,10 +93,9 @@
</span><span class="cx"> bool needsReattach = (m_validity == Undetermined) || !m_nodeToInsertionPoint.isEmpty();
</span><span class="cx">
</span><span class="cx"> if (ShadowRoot* root = host->shadowRoot()) {
</span><del>- const Vector<RefPtr<InsertionPoint>>& insertionPoints = ensureInsertionPointList(root);
- for (size_t i = 0; i < insertionPoints.size(); ++i) {
</del><ins>+ for (auto& point : ensureInsertionPointList(root)) {
</ins><span class="cx"> needsReattach = true;
</span><del>- insertionPoints[i]->clearDistribution();
</del><ins>+ point->clearDistribution();
</ins><span class="cx"> }
</span><span class="cx"> }
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceWebCorehtmlshadowMediaControlElementscpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/shadow/MediaControlElements.cpp (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/shadow/MediaControlElements.cpp 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/html/shadow/MediaControlElements.cpp 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -743,9 +743,7 @@
</span><span class="cx"> RefPtr<Element> offMenuItem;
</span><span class="cx"> bool trackMenuItemSelected = false;
</span><span class="cx">
</span><del>- for (unsigned i = 0, length = m_menuItems.size(); i < length; ++i) {
- RefPtr<Element> trackItem = m_menuItems[i];
-
</del><ins>+ for (auto& trackItem : m_menuItems) {
</ins><span class="cx"> RefPtr<TextTrack> textTrack;
</span><span class="cx"> MenuItemToTrackMap::iterator iter = m_menuToTrackMap.find(trackItem.get());
</span><span class="cx"> if (iter == m_menuToTrackMap.end())
</span><span class="lines">@@ -812,8 +810,7 @@
</span><span class="cx"> appendChild(WTF::move(captionsHeader));
</span><span class="cx"> Ref<Element> captionsMenuList = document().createElement(ulTag, ASSERT_NO_EXCEPTION);
</span><span class="cx">
</span><del>- for (unsigned i = 0, length = tracksForMenu.size(); i < length; ++i) {
- RefPtr<TextTrack> textTrack = tracksForMenu[i];
</del><ins>+ for (auto& textTrack : tracksForMenu) {
</ins><span class="cx"> Ref<Element> menuItem = document().createElement(liTag, ASSERT_NO_EXCEPTION);
</span><span class="cx"> menuItem->appendChild(document().createTextNode(captionPreferences->displayNameForTrack(textTrack.get())));
</span><span class="cx"> captionsMenuList->appendChild(menuItem.copyRef());
</span><span class="lines">@@ -1224,10 +1221,9 @@
</span><span class="cx"> float smallestDimension = std::min(m_videoDisplaySize.size().height(), m_videoDisplaySize.size().width());
</span><span class="cx"> float fontScale = document().page()->group().captionPreferences()->captionFontSizeScaleAndImportance(m_fontSizeIsImportant);
</span><span class="cx"> m_fontSize = lroundf(smallestDimension * fontScale);
</span><del>-
- CueList activeCues = mediaElement->currentlyActiveCues();
- for (size_t i = 0; i < activeCues.size(); ++i) {
- TextTrackCue* cue = activeCues[i].data();
</del><ins>+
+ for (auto& activeCue : mediaElement->currentlyActiveCues()) {
+ TextTrackCue* cue = activeCue.data();
</ins><span class="cx"> if (!cue->isRenderable())
</span><span class="cx"> continue;
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceWebCorehtmltrackAudioTrackListcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/track/AudioTrackList.cpp (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/track/AudioTrackList.cpp 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/html/track/AudioTrackList.cpp 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -74,8 +74,8 @@
</span><span class="cx">
</span><span class="cx"> AudioTrack* AudioTrackList::getTrackById(const AtomicString& id) const
</span><span class="cx"> {
</span><del>- for (size_t i = 0; i < m_inbandTracks.size(); ++i) {
- AudioTrack* track = toAudioTrack(m_inbandTracks[i].get());
</del><ins>+ for (auto& inbandTrack : m_inbandTracks) {
+ AudioTrack* track = toAudioTrack(inbandTrack.get());
</ins><span class="cx"> if (track->id() == id)
</span><span class="cx"> return track;
</span><span class="cx"> }
</span></span></pre></div>
<a id="trunkSourceWebCorehtmltrackLoadableTextTrackcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/track/LoadableTextTrack.cpp (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/track/LoadableTextTrack.cpp 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/html/track/LoadableTextTrack.cpp 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -110,9 +110,9 @@
</span><span class="cx"> if (!m_cues)
</span><span class="cx"> m_cues = TextTrackCueList::create();
</span><span class="cx">
</span><del>- for (size_t i = 0; i < newCues.size(); ++i) {
- newCues[i]->setTrack(this);
- m_cues->add(newCues[i]);
</del><ins>+ for (auto& newCue : newCues) {
+ newCue->setTrack(this);
+ m_cues->add(newCue);
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> if (client())
</span><span class="lines">@@ -136,9 +136,9 @@
</span><span class="cx"> Vector<RefPtr<VTTRegion>> newRegions;
</span><span class="cx"> m_loader->getNewRegions(newRegions);
</span><span class="cx">
</span><del>- for (size_t i = 0; i < newRegions.size(); ++i) {
- newRegions[i]->setTrack(this);
- regions()->add(newRegions[i]);
</del><ins>+ for (auto& newRegion : newRegions) {
+ newRegion->setTrack(this);
+ regions()->add(newRegion);
</ins><span class="cx"> }
</span><span class="cx"> }
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceWebCorehtmltrackTextTrackCueListcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/track/TextTrackCueList.cpp (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/track/TextTrackCueList.cpp 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/html/track/TextTrackCueList.cpp 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -54,9 +54,9 @@
</span><span class="cx">
</span><span class="cx"> TextTrackCue* TextTrackCueList::getCueById(const String& id) const
</span><span class="cx"> {
</span><del>- for (size_t i = 0; i < m_list.size(); ++i) {
- if (m_list[i]->id() == id)
- return m_list[i].get();
</del><ins>+ for (auto& cue : m_list) {
+ if (cue->id() == id)
+ return cue.get();
</ins><span class="cx"> }
</span><span class="cx"> return 0;
</span><span class="cx"> }
</span><span class="lines">@@ -67,8 +67,7 @@
</span><span class="cx"> m_activeCues = create();
</span><span class="cx">
</span><span class="cx"> m_activeCues->clear();
</span><del>- for (size_t i = 0; i < m_list.size(); ++i) {
- RefPtr<TextTrackCue> cue = m_list[i];
</del><ins>+ for (auto& cue : m_list) {
</ins><span class="cx"> if (cue->isActive())
</span><span class="cx"> m_activeCues->add(cue);
</span><span class="cx"> }
</span></span></pre></div>
<a id="trunkSourceWebCorehtmltrackTextTrackListcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/track/TextTrackList.cpp (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/track/TextTrackList.cpp 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/html/track/TextTrackList.cpp 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -72,29 +72,29 @@
</span><span class="cx"> // Calculate the "Let n be the number of text tracks whose text track mode is showing and that are in the media element's list of text tracks before track."
</span><span class="cx"> int trackIndex = 0;
</span><span class="cx">
</span><del>- for (size_t i = 0; i < m_elementTracks.size(); ++i) {
- if (!toTextTrack(m_elementTracks[i].get())->isRendered())
</del><ins>+ for (auto& elementTrack : m_elementTracks) {
+ if (!toTextTrack(elementTrack.get())->isRendered())
</ins><span class="cx"> continue;
</span><span class="cx">
</span><del>- if (m_elementTracks[i] == textTrack)
</del><ins>+ if (elementTrack == textTrack)
</ins><span class="cx"> return trackIndex;
</span><span class="cx"> ++trackIndex;
</span><span class="cx"> }
</span><span class="cx">
</span><del>- for (size_t i = 0; i < m_addTrackTracks.size(); ++i) {
- if (!toTextTrack(m_addTrackTracks[i].get())->isRendered())
</del><ins>+ for (auto& addTrack : m_addTrackTracks) {
+ if (!toTextTrack(addTrack.get())->isRendered())
</ins><span class="cx"> continue;
</span><span class="cx">
</span><del>- if (m_addTrackTracks[i] == textTrack)
</del><ins>+ if (addTrack == textTrack)
</ins><span class="cx"> return trackIndex;
</span><span class="cx"> ++trackIndex;
</span><span class="cx"> }
</span><span class="cx">
</span><del>- for (size_t i = 0; i < m_inbandTracks.size(); ++i) {
- if (!toTextTrack(m_inbandTracks[i].get())->isRendered())
</del><ins>+ for (auto& inbandTrack : m_inbandTracks) {
+ if (!toTextTrack(inbandTrack.get())->isRendered())
</ins><span class="cx"> continue;
</span><span class="cx">
</span><del>- if (m_inbandTracks[i] == textTrack)
</del><ins>+ if (inbandTrack == textTrack)
</ins><span class="cx"> return trackIndex;
</span><span class="cx"> ++trackIndex;
</span><span class="cx"> }
</span><span class="lines">@@ -149,14 +149,14 @@
</span><span class="cx">
</span><span class="cx"> if (track->trackType() == TextTrack::TrackElement) {
</span><span class="cx"> tracks = &m_elementTracks;
</span><del>- for (size_t i = 0; i < m_addTrackTracks.size(); ++i)
- toTextTrack(m_addTrackTracks[i].get())->invalidateTrackIndex();
- for (size_t i = 0; i < m_inbandTracks.size(); ++i)
- toTextTrack(m_inbandTracks[i].get())->invalidateTrackIndex();
</del><ins>+ for (auto& addTrack : m_addTrackTracks)
+ toTextTrack(addTrack.get())->invalidateTrackIndex();
+ for (auto& inbandTrack : m_inbandTracks)
+ toTextTrack(inbandTrack.get())->invalidateTrackIndex();
</ins><span class="cx"> } else if (track->trackType() == TextTrack::AddTrack) {
</span><span class="cx"> tracks = &m_addTrackTracks;
</span><del>- for (size_t i = 0; i < m_inbandTracks.size(); ++i)
- toTextTrack(m_inbandTracks[i].get())->invalidateTrackIndex();
</del><ins>+ for (auto& inbandTrack : m_inbandTracks)
+ toTextTrack(inbandTrack.get())->invalidateTrackIndex();
</ins><span class="cx"> } else if (track->trackType() == TextTrack::InBand)
</span><span class="cx"> tracks = &m_inbandTracks;
</span><span class="cx"> else
</span></span></pre></div>
<a id="trunkSourceWebCorehtmltrackTrackListBasecpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/track/TrackListBase.cpp (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/track/TrackListBase.cpp 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/html/track/TrackListBase.cpp 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -159,8 +159,7 @@
</span><span class="cx">
</span><span class="cx"> bool TrackListBase::isAnyTrackEnabled() const
</span><span class="cx"> {
</span><del>- for (size_t i = 0; i < m_inbandTracks.size(); ++i) {
- TrackBase* track = m_inbandTracks[i].get();
</del><ins>+ for (auto& track : m_inbandTracks) {
</ins><span class="cx"> if (track->enabled())
</span><span class="cx"> return true;
</span><span class="cx"> }
</span></span></pre></div>
<a id="trunkSourceWebCorehtmltrackVideoTrackListcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/html/track/VideoTrackList.cpp (190612 => 190613)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/html/track/VideoTrackList.cpp 2015-10-06 09:41:00 UTC (rev 190612)
+++ trunk/Source/WebCore/html/track/VideoTrackList.cpp 2015-10-06 13:31:14 UTC (rev 190613)
</span><span class="lines">@@ -73,8 +73,8 @@
</span><span class="cx">
</span><span class="cx"> VideoTrack* VideoTrackList::getTrackById(const AtomicString& id) const
</span><span class="cx"> {
</span><del>- for (size_t i = 0; i < length(); ++i) {
- VideoTrack* track = toVideoTrack(m_inbandTracks[i].get());
</del><ins>+ for (auto& inbandTracks : m_inbandTracks) {
+ VideoTrack* track = toVideoTrack(inbandTracks.get());
</ins><span class="cx"> if (track->id() == id)
</span><span class="cx"> return track;
</span><span class="cx"> }
</span></span></pre>
</div>
</div>
</body>
</html>