Make ClientHintsPreferences class work like a class.

Source/core/loader/FrameFetchContextTest.cpp

 /*
  * Copyright (c) 2015, Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 256 matching lines @@
     RefPtrWillBePersistent<DocumentLoader> documentLoader;
     RefPtrWillBePersistent<Document> document;
     Persistent<FrameFetchContext> fetchContext;
 };
 
 TEST_F(FrameFetchContextHintsTest, MonitorDPRHints)
 {
     expectHeader("http://www.example.com/1.gif", "DPR", false, "");
     ClientHintsPreferences preferences;
     preferences.setShouldSendDPR(true);
-    document->setClientHintsPreferences(preferences);
+    document->clientHintsPreferences().updateFrom(preferences);
     expectHeader("http://www.example.com/1.gif", "DPR", true, "1");
     dummyPageHolder->page().setDeviceScaleFactor(2.5);
     expectHeader("http://www.example.com/1.gif", "DPR", true, "2.5");
     expectHeader("http://www.example.com/1.gif", "Width", false, "");
     expectHeader("http://www.example.com/1.gif", "Viewport-Width", false, "");
 }
 
 TEST_F(FrameFetchContextHintsTest, MonitorResourceWidthHints)
 {
     expectHeader("http://www.example.com/1.gif", "Width", false, "");
     ClientHintsPreferences preferences;
     preferences.setShouldSendResourceWidth(true);
-    document->setClientHintsPreferences(preferences);
+    document->clientHintsPreferences().updateFrom(preferences);
     expectHeader("http://www.example.com/1.gif", "Width", true, "500", 500);
     expectHeader("http://www.example.com/1.gif", "Width", true, "667", 666.6666);
     expectHeader("http://www.example.com/1.gif", "DPR", false, "");
 }
 
 TEST_F(FrameFetchContextHintsTest, MonitorViewportWidthHints)
 {
     expectHeader("http://www.example.com/1.gif", "Viewport-Width", false, "");
     ClientHintsPreferences preferences;
     preferences.setShouldSendViewportWidth(true);
-    document->setClientHintsPreferences(preferences);
+    document->clientHintsPreferences().updateFrom(preferences);
     expectHeader("http://www.example.com/1.gif", "Viewport-Width", true, "500");
     dummyPageHolder->frameView().setLayoutSizeFixedToFrameSize(false);
     dummyPageHolder->frameView().setLayoutSize(IntSize(800, 800));
     expectHeader("http://www.example.com/1.gif", "Viewport-Width", true, "800");
     expectHeader("http://www.example.com/1.gif", "Viewport-Width", true, "800", 666.6666);
     expectHeader("http://www.example.com/1.gif", "DPR", false, "");
 }
 
 TEST_F(FrameFetchContextHintsTest, MonitorAllHints)
 {
     expectHeader("http://www.example.com/1.gif", "DPR", false, "");
     expectHeader("http://www.example.com/1.gif", "Viewport-Width", false, "");
     expectHeader("http://www.example.com/1.gif", "Width", false, "");
 
     ClientHintsPreferences preferences;
     preferences.setShouldSendDPR(true);
     preferences.setShouldSendResourceWidth(true);
     preferences.setShouldSendViewportWidth(true);
-    document->setClientHintsPreferences(preferences);
+    document->clientHintsPreferences().updateFrom(preferences);
     expectHeader("http://www.example.com/1.gif", "DPR", true, "1");
     expectHeader("http://www.example.com/1.gif", "Width", true, "400", 400);
     expectHeader("http://www.example.com/1.gif", "Viewport-Width", true, "500");
 }
 
 class StubFrameLoaderClientWithParent : public EmptyFrameLoaderClient {
 public:
     explicit StubFrameLoaderClientWithParent(Frame* parent)
         : m_parent(parent)
     {
@@ skipping 99 matching lines @@
     // Child frame as part of end to end reload
     document->frame()->loader().setLoadType(FrameLoadTypeReloadFromOrigin);
     EXPECT_EQ(ReloadBypassingCache, childFetchContext->resourceRequestCachePolicy(request, Resource::MainResource));
 
     childDocumentLoader->detachFromFrame();
     childDocumentLoader.clear();
     childFrame->detach(FrameDetachType::Remove);
 }
 
 } // namespace