tools: check for std::vector<v8::Local> in lint

[Aditi-1400]

Adds a lint rule to disallow the usage of std::vector<v8::Local<T>>.
Follow up from the clean up of std::vector<v8::Local<T>> we did before.
From pull request: #57578:

According to V8's public API documentation, local handles (i.e., objects of type v8::Local) "should never be allocated on the heap". This replaces the usage of heap-allocated data structures containing instances of v8::Local, specifically the std::vector<v8::Localv8::String> with recently introduced v8::LocalVector.

[Aditi-1400]

Oops, looks like I missed a few instances of std::vector<v8::Local<T>>. I'll take care of those before this pull request is merged

[juanarbol]

lgtm

[anonrig]

I don't think we need this. We can just enable V8_ENABLE_LOCAL_OFF_STACK_CHECK check on v8 that enforces vector not to be used.

[joyeecheung]

We can just enable V8_ENABLE_LOCAL_OFF_STACK_CHECK check on v8 that enforces vector not to be used.

I think we can have both, in case it ends up in a path that we don't actually build/test in the CI.

[joyeecheung]

src/quic/streams.h:335:  Do not use std::vector<v8::Local<T>>. Use v8::LocalVector<T> instead.  [runtime/local_vector] [5]
test/addons/make-callback/binding.cc:14:  Do not use std::vector<v8::Local<T>>. Use v8::LocalVector<T> instead.  [runtime/local_vector] [5]
test/addons/openssl-providers/binding.cc:29:  Do not use std::vector<v8::Local<T>>. Use v8::LocalVector<T> instead.  [runtime/local_vector] [5]

This is getting being caught in the CI of this PR BTW (kinda proves my point about "what if the path is not built in the CI"..)

[joyeecheung]

I think the GitHub actions require manual rebase. @Aditi-1400 can you rebase again?