Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 /* | 1 /* |
| 2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org) | 2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org) |
| 3 * (C) 1999 Antti Koivisto (koivisto@kde.org) | 3 * (C) 1999 Antti Koivisto (koivisto@kde.org) |
| 4 * Copyright (C) 2003, 2004, 2005, 2006, 2007, 2008 Apple Inc. All rights reserv ed. | 4 * Copyright (C) 2003, 2004, 2005, 2006, 2007, 2008 Apple Inc. All rights reserv ed. |
| 5 * | 5 * |
| 6 * This library is free software; you can redistribute it and/or | 6 * This library is free software; you can redistribute it and/or |
| 7 * modify it under the terms of the GNU Library General Public | 7 * modify it under the terms of the GNU Library General Public |
| 8 * License as published by the Free Software Foundation; either | 8 * License as published by the Free Software Foundation; either |
| 9 * version 2 of the License, or (at your option) any later version. | 9 * version 2 of the License, or (at your option) any later version. |
| 10 * | 10 * |
| (...skipping 257 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 268 { | 268 { |
| 269 static HashSet<StringImpl*>* htmlCaseInsensitiveAttributesSet = createHtmlCa seInsensitiveAttributesSet(); | 269 static HashSet<StringImpl*>* htmlCaseInsensitiveAttributesSet = createHtmlCa seInsensitiveAttributesSet(); |
| 270 bool isPossibleHTMLAttr = !attributeName.hasPrefix() && (attributeName.names paceURI() == nullAtom); | 270 bool isPossibleHTMLAttr = !attributeName.hasPrefix() && (attributeName.names paceURI() == nullAtom); |
| 271 return !isPossibleHTMLAttr || !htmlCaseInsensitiveAttributesSet->contains(at tributeName.localName().impl()); | 271 return !isPossibleHTMLAttr || !htmlCaseInsensitiveAttributesSet->contains(at tributeName.localName().impl()); |
| 272 } | 272 } |
| 273 | 273 |
| 274 void HTMLDocument::write(LocalDOMWindow* callingWindow, const Vector<String>& te xt, ExceptionState& exceptionState) | 274 void HTMLDocument::write(LocalDOMWindow* callingWindow, const Vector<String>& te xt, ExceptionState& exceptionState) |
| 275 { | 275 { |
| 276 ASSERT(callingWindow); | 276 ASSERT(callingWindow); |
| 277 StringBuilder builder; | 277 StringBuilder builder; |
| 278 for (size_t i = 0; i < text.size(); ++i) | 278 for (const auto& string : text) |
|
Mike West
2014/11/04 15:25:52
No need for `auto` here. `const String&` is cleare
riju_
2014/11/04 17:50:53
Done.
| |
| 279 builder.append(text[i]); | 279 builder.append(string); |
| 280 write(builder.toString(), callingWindow->document(), exceptionState); | 280 write(builder.toString(), callingWindow->document(), exceptionState); |
| 281 } | 281 } |
| 282 | 282 |
| 283 void HTMLDocument::writeln(LocalDOMWindow* callingWindow, const Vector<String>& text, ExceptionState& exceptionState) | 283 void HTMLDocument::writeln(LocalDOMWindow* callingWindow, const Vector<String>& text, ExceptionState& exceptionState) |
| 284 { | 284 { |
| 285 ASSERT(callingWindow); | 285 ASSERT(callingWindow); |
| 286 StringBuilder builder; | 286 StringBuilder builder; |
| 287 for (size_t i = 0; i < text.size(); ++i) | 287 for (const auto& string : text) |
|
Mike West
2014/11/04 15:25:52
Same here.
riju_
2014/11/04 17:50:53
Done.
| |
| 288 builder.append(text[i]); | 288 builder.append(string); |
| 289 writeln(builder.toString(), callingWindow->document(), exceptionState); | 289 writeln(builder.toString(), callingWindow->document(), exceptionState); |
| 290 } | 290 } |
| 291 | 291 |
| 292 } | 292 } |
| OLD | NEW |