Made code in DocumentMarkerController.cpp conform to blink's coding

guidelines and improved performance in WebViewImpl.cpp a bit.

Review URL: https://codereview.chromium.org/479803002

git-svn-id: svn://svn.chromium.org/blink/trunk@180447 bbb929c8-8fbe-4397-9dbb-9b2b20218538

third_party/WebKit/Source/core/dom/DocumentMarkerController.cpp

@@ -46,8 +46,8 @@ MarkerRemoverPredicate::MarkerRemoverPredicate(const Vector<String>& words)
 {
 }
 
-bool MarkerRemoverPredicate::operator()(const DocumentMarker& documentMarker,
-    const Text& textNode) const {
+bool MarkerRemoverPredicate::operator()(const DocumentMarker& documentMarker, const Text& textNode) const
+{
     unsigned start  = documentMarker.startOffset();
     unsigned length = documentMarker.endOffset() - documentMarker.startOffset();
 

third_party/WebKit/Source/web/WebViewImpl.cpp

@@ -3422,14 +3422,12 @@ void WebViewImpl::spellingMarkers(WebVector<uint32_t>* markers)
 
 void WebViewImpl::removeSpellingMarkersUnderWords(const WebVector<WebString>& words)
 {
-    for (Frame* frame = m_page->mainFrame(); frame; frame = frame->tree().traverseNext()) {
-        if (!frame->isLocalFrame())
-            continue;
+    Vector<String> convertedWords;
+    convertedWords.append(words.data(), words.size());
 
-        Vector<String> convertedWords;
-        convertedWords.append(words.data(), words.size());
-
-        toLocalFrame(frame)->removeSpellingMarkersUnderWords(convertedWords);
+    for (Frame* frame = m_page->mainFrame(); frame; frame = frame->tree().traverseNext()) {
+        if (frame->isLocalFrame())
+            toLocalFrame(frame)->removeSpellingMarkersUnderWords(convertedWords);
     }
 }