Do not white list offline archive directory

chrome/browser/net/chrome_network_delegate.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/browser/net/chrome_network_delegate.h"
 
 #include <stddef.h>
 #include <stdlib.h>
 
 #include <vector>
@@ skipping 497 matching lines @@
       return true;
     }
   }
 #elif defined(OS_ANDROID)
   // Access to files in external storage is allowed.
   base::FilePath external_storage_path;
   PathService::Get(base::DIR_ANDROID_EXTERNAL_STORAGE, &external_storage_path);
   if (external_storage_path.IsParent(path))
     return true;
 
-  // Allow to load offline pages, which are stored in the $PROFILE_PATH/Offline
-  // Pages/archives.
-  if (!profile_path_.empty()) {
-    const base::FilePath offline_page_archives =
-        profile_path_.Append(chrome::kOfflinePageArchivesDirname);

[mmenke, 2016/10/06 15:28:31]

Do we still need the chrome/common/chrome_constants.h include?  If it was added
just for this, I suspect not.

[jianli, 2016/10/06 22:31:33]

Done.

-    if (offline_page_archives.IsParent(path))
-      return true;
-  }
-
   // Whitelist of other allowed directories.
   static const char* const kLocalAccessWhiteList[] = {
       "/sdcard",
       "/mnt/sdcard",
   };
 #endif
 
   for (size_t i = 0; i < arraysize(kLocalAccessWhiteList); ++i) {
     const base::FilePath white_listed_path(kLocalAccessWhiteList[i]);
     // base::FilePath::operator== should probably handle trailing separators.
@@ skipping 49 matching lines @@
   if (!data_use_aggregator_)
     return;
 
   if (is_data_usage_off_the_record_) {
     data_use_aggregator_->ReportOffTheRecordDataUse(tx_bytes, rx_bytes);
     return;
   }
 
   data_use_aggregator_->ReportDataUse(request, tx_bytes, rx_bytes);
 }