net: disable SSL compression

chrome/browser/page_info_model.cc

Index: chrome/browser/page_info_model.cc
diff --git a/chrome/browser/page_info_model.cc b/chrome/browser/page_info_model.cc
index 89ce07b0ef68480ba5ef166a5c3a7f9b6cc1e26f..34521a3996acdab59bfb8c8106954802ff2a2ac8 100644
--- a/chrome/browser/page_info_model.cc
+++ b/chrome/browser/page_info_model.cc
@@ -241,18 +241,15 @@ PageInfoModel::PageInfoModel(Profile* profile,
         IDS_PAGE_INFO_SECURITY_TAB_ENCRYPTION_DETAILS,
         ASCIIToUTF16(cipher), ASCIIToUTF16(mac), ASCIIToUTF16(key_exchange));
 
-    description += ASCIIToUTF16("\n\n");
     uint8 compression_id =
         net::SSLConnectionStatusToCompression(ssl.connection_status);
     if (compression_id) {
       const char* compression;
       net::SSLCompressionToString(&compression, compression_id);
+      description += ASCIIToUTF16("\n\n");
       description += l10n_util::GetStringFUTF16(
           IDS_PAGE_INFO_SECURITY_TAB_COMPRESSION_DETAILS,
           ASCIIToUTF16(compression));
-    } else {
-      description += l10n_util::GetStringUTF16(
-          IDS_PAGE_INFO_SECURITY_TAB_NO_COMPRESSION);

[wtc, 2012/08/01 17:31:28]

Just to doublecheck: if TLS compression is not used, you
don't want the page info model to say that.  Correct?

[agl, 2012/08/01 23:04:59]

Yes. It seems silly to mention that the connection is not compressed if we don't
support it. In the future we'll probably want to clean this up but I don't want
to make this patch hard to merge.

     }
 
     if (did_fallback) {