Changing VLOG from level 1 to 9 for text to be translated.

chrome/renderer/translate_helper.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/renderer/translate_helper.h"
 
 #include "base/bind.h"
 #include "base/compiler_specific.h"
 #include "base/logging.h"
 #include "base/message_loop.h"
@@ skipping 51 matching lines @@
   WebDocument document = render_view()->GetWebView()->mainFrame()->document();
   // If the page explicitly specifies a language, use it, otherwise we'll
   // determine it based on the text content using the CLD.
   std::string language = GetPageLanguageFromMetaTag(&document);
   if (language.empty()) {
     base::TimeTicks begin_time = base::TimeTicks::Now();
     language = DetermineTextLanguage(contents);
     UMA_HISTOGRAM_MEDIUM_TIMES("Renderer4.LanguageDetection",
                                base::TimeTicks::Now() - begin_time);
   } else {
-    VLOG(1) << "PageLanguageFromMetaTag: " << language;
+    VLOG(9) << "PageLanguageFromMetaTag: " << language;
   }
 
   Send(new ChromeViewHostMsg_TranslateLanguageDetermined(
       routing_id(), language, IsPageTranslatable(&document)));
 }
 
 void TranslateHelper::CancelPendingTranslation() {
   weak_method_factory_.InvalidateWeakPtrs();
   translation_pending_ = false;
   page_id_ = -1;
@@ skipping 70 matching lines @@
   if (is_reliable && text_bytes >= 100 && cld_language != NUM_LANGUAGES &&
       cld_language != UNKNOWN_LANGUAGE && cld_language != TG_UNKNOWN_LANGUAGE) {
     // We should not use LanguageCode_ISO_639_1 because it does not cover all
     // the languages CLD can detect. As a result, it'll return the invalid
     // language code for tradtional Chinese among others.
     // |LanguageCodeWithDialect| will go through ISO 639-1, ISO-639-2 and
     // 'other' tables to do the 'right' thing. In addition, it'll return zh-CN
     // for Simplified Chinese.
     language = LanguageCodeWithDialects(cld_language);
   }
-  VLOG(1) << "Detected lang_id: " << language << ", from Text:\n" << text
+  VLOG(9) << "Detected lang_id: " << language << ", from Text:\n" << text

[rvargas (doing something else), 2012/08/10 19:11:30]

Actually, I would still want to understand why are we dumping the whole text
here and why do we have to do that on the released product (but the bug report
is a better place for that).

I guess 9 is better than 1 so I'm not blocking the commit :)

           << "\n*************************************\n";
   return language;
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // TranslateHelper, protected:
 //
 bool TranslateHelper::IsTranslateLibAvailable() {
   bool lib_available = false;
   if (!ExecuteScriptAndGetBoolResult(
@@ skipping 269 matching lines @@
   WebView* web_view = render_view()->GetWebView();
   if (!web_view) {
     // When the WebView is going away, the render view should have called
     // CancelPendingTranslation() which should have stopped any pending work, so
     // that case should not happen.
     NOTREACHED();
     return NULL;
   }
   return web_view->mainFrame();
 }