Revert "[web snapshot] Dehandlify more parts of the deserializer"

This reverts commit 7ddacd68.

Reason for revert: GC Stress failures: https://ci.chromium.org/ui/p/v8/builders/ci/V8%20Linux64%20GC%20Stress%20-%20custom%20snapshot/40615/overview

Original change's description:
> [web snapshot] Dehandlify more parts of the deserializer
>
> - Use Heap::AddGCEpilogueCallback to update often accessed FixedArrays
>   in the WebSnapshotDeserializer.
> - ReadValue returns now a raw value to avoid handle creation in more
>   cases
> - Drop representation support for now in ReadValue
> - Avoid a few more handles when setting up objects
>
> Bug v8:11525
>
> Change-Id: I6955b56887834bc655bdaa9c390016d9a17db82d
> Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3416242
> Reviewed-by: Marja Hölttä <marja@chromium.org>
> Commit-Queue: Camillo Bruni <cbruni@chromium.org>
> Cr-Commit-Position: refs/heads/main@{#78862}

Change-Id: I44385ac118707ec60aa8c76e219b387bc3ba322e
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3423781
Auto-Submit: Leszek Swirski <leszeks@chromium.org>
Owners-Override: Leszek Swirski <leszeks@chromium.org>
Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Cr-Commit-Position: refs/heads/main@{#78863}

src/web-snapshot/web-snapshot.h

@@ -243,14 +243,6 @@ class V8_EXPORT WebSnapshotDeserializer
   uint32_t array_count() const { return array_count_; }
   uint32_t object_count() const { return object_count_; }
 
-  static void UpdatePointersCallback(v8::Isolate* isolate, v8::GCType type,
-                                     v8::GCCallbackFlags flags,
-                                     void* deserializer) {
-    reinterpret_cast<WebSnapshotDeserializer*>(deserializer)->UpdatePointers();
-  }
-
-  void UpdatePointers();
-
  private:
   WebSnapshotDeserializer(Isolate* isolate, Handle<Object> script_name,
                           base::Vector<const uint8_t> buffer);
@@ -263,7 +255,7 @@ class V8_EXPORT WebSnapshotDeserializer
   WebSnapshotDeserializer& operator=(const WebSnapshotDeserializer&) = delete;
 
   void DeserializeStrings();
-  String ReadString(bool internalize = false);
+  Handle<String> ReadString(bool internalize = false);
   void DeserializeMaps();
   void DeserializeContexts();
   Handle<ScopeInfo> CreateScopeInfo(uint32_t variable_count, bool has_parent,
@@ -277,47 +269,31 @@ class V8_EXPORT WebSnapshotDeserializer
   void DeserializeArrays();
   void DeserializeObjects();
   void DeserializeExports();
-  Object ReadValue(
+  void ReadValue(
+      Handle<Object>& value, Representation& representation,
       Handle<HeapObject> object_for_deferred_reference = Handle<HeapObject>(),
       uint32_t index_for_deferred_reference = 0);
   void ReadFunctionPrototype(Handle<JSFunction> function);
   bool SetFunctionPrototype(JSFunction function, JSReceiver prototype);
 
-  HeapObject AddDeferredReference(Handle<HeapObject> container, uint32_t index,
-                                  ValueType target_type,
-                                  uint32_t target_object_index);
+  void AddDeferredReference(Handle<HeapObject> container, uint32_t index,
+                            ValueType target_type,
+                            uint32_t target_object_index);
   void ProcessDeferredReferences();
   // Not virtual, on purpose (because it doesn't need to be).
   void Throw(const char* message);
 
-  Handle<FixedArray> strings_handle_;
-  FixedArray strings_;
-
-  Handle<FixedArray> maps_handle_;
-  FixedArray maps_;
-
-  Handle<FixedArray> contexts_handle_;
-  FixedArray contexts_;
-
-  Handle<FixedArray> functions_handle_;
-  FixedArray functions_;
-
-  Handle<FixedArray> classes_handle_;
-  FixedArray classes_;
-
-  Handle<FixedArray> arrays_handle_;
-  FixedArray arrays_;
-
-  Handle<FixedArray> objects_handle_;
-  FixedArray objects_;
-
+  Handle<FixedArray> strings_;
+  Handle<FixedArray> maps_;
+  Handle<FixedArray> contexts_;
+  Handle<FixedArray> functions_;
+  Handle<FixedArray> classes_;
+  Handle<FixedArray> arrays_;
+  Handle<FixedArray> objects_;
   Handle<ArrayList> deferred_references_;
 
-  Handle<WeakFixedArray> shared_function_infos_handle_;
-  WeakFixedArray shared_function_infos_;
-
+  Handle<WeakFixedArray> shared_function_infos_;
   Handle<ObjectHashTable> shared_function_info_table_;
-
   Handle<Script> script_;
   Handle<Object> script_name_;