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

Unified Diff: chrome/renderer/safe_browsing/phishing_dom_feature_extractor.cc

Issue 377683002: Fixes for re-enabling more MSVC level 4 warnings: chrome/ edition (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/renderer/safe_browsing/phishing_dom_feature_extractor.cc
diff --git a/chrome/renderer/safe_browsing/phishing_dom_feature_extractor.cc b/chrome/renderer/safe_browsing/phishing_dom_feature_extractor.cc
index 65742936e50cbba1ab290baedd0ec0e3fce30d9c..821fe2edd1c231eac68e4b9bf9d0c36df545ebe1 100644
--- a/chrome/renderer/safe_browsing/phishing_dom_feature_extractor.cc
+++ b/chrome/renderer/safe_browsing/phishing_dom_feature_extractor.cc
@@ -393,7 +393,8 @@ blink::WebDocument PhishingDOMFeatureExtractor::GetNextDocument() {
blink::WebFrame* frame = cur_document_.frame();
// Advance to the next frame that contains a document, with no wrapping.
if (frame) {
- while ((frame = frame->traverseNext(false))) {
+ for (frame = frame->traverseNext(false); frame;
+ frame = frame->traverseNext(false)) {
if (!frame->document().isNull()) {
return frame->document();
}

Powered by Google App Engine
This is Rietveld 408576698