Revert "[VM] Ensure Top level class is finalized for loadedscripts in library"

This reverts commit 9f00d1b.

Reason for revert: This change triggered some incorrect runtime typecheck failures in Fuchsia.

Original change's description:
> [VM] Ensure Top level class is finalized for loadedscripts in library
> 
> The breakpoints sent from IDE will be checked with existing scripts.
> Because the loadedscripts() doesn't contain the scripts info for part/part of.
> Then the breakpoint will be considerred as a latent breakpoint and didn't get
> resolved later. The solution is to finalize the toplevel class before loadedscripts()
> uses Dictionary iterator.
> 
> Bug: #35859
> Change-Id: I90b67ee9e9e6afe2556ca806cdd87eb5661304a8
> Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/97402
> Reviewed-by: Alexander Markov <[email protected]>
> Commit-Queue: Zichang Guo <[email protected]>

[email protected],[email protected],[email protected]

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: #35859
Change-Id: Ib28c2257f94b8d7ee0a3607be4a92153b7229e54
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/98360
Reviewed-by: Zach Anderson <[email protected]>

Author: zanderso

runtime/vm/object.cc

@@ -10389,7 +10389,6 @@ RawArray* Library::LoadedScripts() const {
     Object& entry = Object::Handle();
     Class& cls = Class::Handle();
     Script& owner_script = Script::Handle();
-    EnsureTopLevelClassIsFinalized();
     DictionaryIterator it(*this);
     while (it.HasNext()) {
       entry = it.GetNext();