Delegate no-arg PackagerStatusCheck constructor to shared OkHttpClient (#55577)

Summary:
Pull Request resolved: https://github.com/facebook/react-native/pull/55577

PackagerStatusCheck had two constructors: a no-arg one that created its own OkHttpClient, and one that accepted an external client. The no-arg constructor is used by callers in fbandroid/ (FbDevBottomSheetViewController, LocalJsPackagerManager, FbPackagerStatusCheck, FbDevModeNuxController). Instead of removing it, delegate it to `DevSupportHttpClient.httpClient` so that all callers share the same connection pool without needing code changes.

Also removes the now-unused `HTTP_CONNECT_TIMEOUT_MS` constant since the timeout configuration is centralized in DevSupportHttpClient.

Changelog:
[Internal] [Changed] -

Reviewed By: cipolleschi

Differential Revision: D93480248

fbshipit-source-id: 07b3a6c01cec19a5de1604c560229622974e500e