[Bindings] Move and remove some private symbols from V8HiddenValue

third_party/WebKit/Source/modules/serviceworkers/FetchEvent.cpp

 // Copyright 2014 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 "modules/serviceworkers/FetchEvent.h"
 
 #include "bindings/core/v8/ScriptState.h"
 #include "bindings/core/v8/ToV8.h"
-#include "bindings/core/v8/V8HiddenValue.h"
+#include "bindings/core/v8/V8PrivateProperty.h"
 #include "modules/fetch/BytesConsumerForDataConsumerHandle.h"
 #include "modules/fetch/Request.h"
 #include "modules/fetch/Response.h"
 #include "modules/serviceworkers/FetchRespondWithObserver.h"
 #include "modules/serviceworkers/ServiceWorkerError.h"
 #include "modules/serviceworkers/ServiceWorkerGlobalScope.h"
 #include "public/platform/WebURLResponse.h"
 #include "public/platform/modules/serviceworker/WebServiceWorkerError.h"
 #include "wtf/PtrUtil.h"
 #include "wtf/RefPtr.h"
@@ skipping 70 matching lines @@
     if (event.IsEmpty()) {
       // |toV8| can return an empty handle when the worker is terminating.
       // We don't want the renderer to crash in such cases.
       // TODO(yhirano): Replace this branch with an assertion when the
       // graceful shutdown mechanism is introduced.
       return;
     }
     DCHECK(event->IsObject());
     // Sets a hidden value in order to teach V8 the dependency from
     // the event to the request.
-    V8HiddenValue::setHiddenValue(
-        scriptState, event.As<v8::Object>(),
-        V8HiddenValue::requestInFetchEvent(scriptState->isolate()), request);
+    V8PrivateProperty::getFetchEventRequest(scriptState->isolate())
+        .set(event.As<v8::Object>(), request);

[haraken, 2017/03/30 14:04:45]

Ditto.

[peria, 2017/04/04 05:48:58]

Acknowledged.

     // From the same reason as above, setHiddenValue can return false.
     // TODO(yhirano): Add an assertion that it returns true once the
     // graceful shutdown mechanism is introduced.
   }
 }
 
 void FetchEvent::onNavigationPreloadResponse(
     ScriptState* scriptState,
     std::unique_ptr<WebURLResponse> response,
     std::unique_ptr<WebDataConsumerHandle> dataConsumeHandle) {
@@ skipping 33 matching lines @@
 }
 
 DEFINE_TRACE(FetchEvent) {
   visitor->trace(m_observer);
   visitor->trace(m_request);
   visitor->trace(m_preloadResponseProperty);
   ExtendableEvent::trace(visitor);
 }
 
 }  // namespace blink