Respect QuicAllowed policy for new streams

chrome/browser/net/net_pref_observer.h

+// 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.
+
+#ifndef CHROME_BROWSER_NET_NET_PREF_OBSERVER_H_
+#define CHROME_BROWSER_NET_NET_PREF_OBSERVER_H_
+
+#include "base/macros.h"
+#include "components/prefs/pref_member.h"
+
+class PrefService;
+
+namespace user_prefs {
+class PrefRegistrySyncable;
+}
+
+// Monitors network-related preferences for changes and applies them.
+// The supplied PrefService must outlive this NetPrefObserver.
+// Must be used only on the UI thread.
+class NetPrefObserver {
+ public:
+  // |prefs| must be non-NULL and |*prefs| must outlive this.
+  explicit NetPrefObserver(PrefService* prefs);
+  virtual ~NetPrefObserver();

[Ryan Hamilton, 2016/12/20 15:38:12]

Does this need to be virtual? It doesn't have a base class and I don't seen any
derived classes.

[pmarko, 2016/12/20 18:00:37]

Right - I resurrected this from an older CL where it was virtual and didn't
notice. No need for this to be virtual. Done.

+
+  static void RegisterProfilePrefs(user_prefs::PrefRegistrySyncable* registry);

[Ryan Hamilton, 2016/12/20 15:38:12]

Can you add a comment about this method? I'm not super familiar with this code,
but from the name, I'm not sure what it does.

[pmarko, 2016/12/20 18:00:37]

Good point, I've added a comment. The name seems to be a common name across the
codebase with components registering "their own" prefs (see
https://cs.chromium.org/search/?q=RegisterProfilePrefs&p=2&sq=package:chromiu...
).

+
+ private:
+  void ApplySettings();
+
+  BooleanPrefMember quic_allowed_;
+
+  DISALLOW_COPY_AND_ASSIGN(NetPrefObserver);
+};
+
+#endif  // CHROME_BROWSER_NET_NET_PREF_OBSERVER_H_