Split methods for ClientHintsPreferences from FrameFetchContext

third_party/WebKit/Source/core/loader/FrameClientHintsPreferencesContext.h

+// Copyright 2016 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 FrameClientHintsPreferencesContext_h
+#define FrameClientHintsPreferencesContext_h
+
+#include "core/fetch/ClientHintsPreferences.h"
+#include "core/frame/Frame.h"
+#include "platform/heap/Persistent.h"
+#include "wtf/Allocator.h"
+
+namespace blink {
+
+class FrameClientHintsPreferencesContext

[yhirano, 2016/12/08 07:47:12]

+final

[tyoshino (SeeGerritForStatus), 2016/12/09 09:43:24]

Done.

+    : public ClientHintsPreferences::Context {
+  STACK_ALLOCATED();
+
+ public:
+  explicit FrameClientHintsPreferencesContext(Frame*);
+
+  void countClientHintsDPR() override;
+  void countClientHintsResourceWidth() override;
+  void countClientHintsViewportWidth() override;
+
+ private:
+  Persistent<Frame> m_frame;

[yhirano, 2016/12/08 07:47:12]

I think a STACK_ALLOCATED class can have an onheap-member as Member without a
trace function (but I couldn't find a document).

[tyoshino (SeeGerritForStatus), 2016/12/09 09:43:24]

Tried, but it seems we can't.

../../third_party/WebKit/Source/core/loader/FrameClientHintsPreferencesContext.h:28:3:
note: [blink-gc] Raw pointer field 'm_frame' to a GC managed class declared
here:
  Frame* m_frame;

[tyoshino (SeeGerritForStatus), 2016/12/09 09:46:38]

Sorry I misunderstood the suggestion. Done

+};
+
+}  // namespace blink
+
+#endif  // FrameClientHintsPreferencesContext_h