Add brotli content-encoding filter.

chrome/browser/about_flags.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/about_flags.h"
 
 #include <iterator>
 #include <map>
 #include <set>
 #include <utility>
@@ skipping 37 matching lines @@
 #include "components/signin/core/common/signin_switches.h"
 #include "components/sync_driver/sync_driver_switches.h"
 #include "components/tracing/tracing_switches.h"
 #include "components/version_info/version_info.h"
 #include "content/public/browser/user_metrics.h"
 #include "content/public/common/content_features.h"
 #include "content/public/common/content_switches.h"
 #include "grit/components_strings.h"
 #include "media/base/media_switches.h"
 #include "media/midi/midi_switches.h"
+#include "net/http/http_features.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/ui_base_switches.h"
 #include "ui/display/display_switches.h"
 #include "ui/events/event_switches.h"
 #include "ui/gfx/switches.h"
 #include "ui/gl/gl_switches.h"
 #include "ui/keyboard/keyboard_switches.h"
 #include "ui/native_theme/native_theme_switches.h"
 #include "ui/views/views_switches.h"
 
@@ skipping 1945 matching lines @@
      kOsAndroid,
      ENABLE_DISABLE_VALUE_TYPE(switches::kEnableWebNotificationCustomLayouts,
                                switches::kDisableWebNotificationCustomLayouts)},
 #endif  // defined(ENABLE_NOTIFICATIONS) && defined(OS_ANDROID)
 #if defined(OS_WIN)
     {"enable-appcontainer", IDS_FLAGS_ENABLE_APPCONTAINER_NAME,
      IDS_FLAGS_ENABLE_APPCONTAINER_DESCRIPTION, kOsWin,
      ENABLE_DISABLE_VALUE_TYPE(switches::kEnableAppContainer,
                                switches::kDisableAppContainer)},
 #endif  // defined(OS_WIN)
+#if !defined(DISABLE_BROTLI_SUPPORT)

[Ryan Sleevi, 2015/12/17 01:04:40]

BUG: Please review
https://groups.google.com/a/chromium.org/d/msg/chromium-dev/5c4ySpPsV14/JJfAe...

#if BUILDFLAG(DISABLE_BROTLI_SUPPORT)

[eustas, 2015/12/17 16:44:54]

All 3 exmaples of buildflag look simple - I don't see a way to change the value
of the flag for concrete target (e.g. for cronet_small). Is it possible?

[Ryan Sleevi, 2015/12/17 22:49:33]

That's not possible today either. That's part of why it's a BUG - what you're
trying to do (defining at the top-most target whether or not a feature is
enabled) is explicitly not supported.

+    {"enable-brotli",
+     IDS_FLAGS_ENABLE_BROTLI_NAME,
+     IDS_FLAGS_ENABLE_BROTLI_DESCRIPTION,
+     kOsAll,
+     FEATURE_VALUE_TYPE(net::features::kBrotliEncodingFeature)},
+#endif  // !defined(DISABLE_BROTLI_SUPPORT)
 #if defined(OS_ANDROID)
     {"disable-auto-hiding-toolbar-threshold",
      IDS_FLAGS_DISABLE_AUTO_HIDING_TOOLBAR_NAME,
      IDS_FLAGS_DISABLE_AUTO_HIDING_TOOLBAR_DESCRIPTION,
      kOsAndroid,
      SINGLE_VALUE_TYPE_AND_VALUE(switches::kDisableAutoHidingToolbarThreshold,
                                  "800")},
 #endif
 #if defined(TOOLKIT_VIEWS)
     {"enable-autofill-credit-card-upload",
@@ skipping 247 matching lines @@
 const base::HistogramBase::Sample kBadSwitchFormatHistogramId = 0;
 
 const FeatureEntry* GetFeatureEntries(size_t* count) {
   *count = arraysize(kFeatureEntries);
   return kFeatureEntries;
 }
 
 }  // namespace testing
 
 }  // namespace about_flags