[clang-tidy] Remove unneeded constructor

The constructor of ByteData isn't doing anything interesting, so
can be removed.

Bug: v8:10488
Change-Id: Ic114b947ff6471075c7df49c98ea7c59c5b522bc
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2233978
Commit-Queue: Zhi An Ng <zhin@chromium.org>
Reviewed-by: Yang Guo <yangguo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#68312}

src/parsing/preparse-data-impl.h

@@ -37,8 +37,6 @@ class BaseConsumedPreparseData : public ConsumedPreparseData {
  public:
   class ByteData : public PreparseByteDataConstants {
    public:
-    ByteData() {}
-
     // Reading from the ByteData is only allowed when a ReadingScope is on the
     // stack. This ensures that we have a DisallowHeapAllocation in place
     // whenever ByteData holds a raw pointer into the heap.