Make _delete() idempotent

[hsubox76]

_delete() should be idempotent - it should just be a noop if the service has already been deleted. This prevents errors trying to delete an already deleted service.

Also firestore-compat and database-compat can't have their INTERNAL.delete() method removed easily because it's in a class shared by v8, exp, and compat. But storage-compat has it in a compat-only file, where it's not needed, so removed it.

[changeset-bot]

⚠️ No Changeset found

Latest commit: c9f76a3

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[github-actions]

Changeset File Check ✅

• No modified packages are missing from the changeset file.
• No changeset formatting errors detected.

[google-oss-bot]

Binary Size Report

Affected SDKs

• @firebase/database

  Type	Base (d1ae4de)	Head (60170d2)	Diff
  browser	296 kB	296 kB	-2 B (-0.0%)
  esm2017	265 kB	265 kB	-2 B (-0.0%)
  main	299 kB	299 kB	-2 B (-0.0%)
  module	296 kB	296 kB	-2 B (-0.0%)

• @firebase/database-exp

  Type	Base (d1ae4de)	Head (60170d2)	Diff
  browser	246 kB	246 kB	-2 B (-0.0%)
  main	278 kB	278 kB	-2 B (-0.0%)
  module	246 kB	246 kB	-2 B (-0.0%)

• @firebase/firestore

  Type	Base (d1ae4de)	Head (60170d2)	Diff
  browser	284 kB	285 kB	+240 B (+0.1%)
  esm2017	226 kB	227 kB	+252 B (+0.1%)
  main	531 kB	531 kB	+424 B (+0.1%)
  module	284 kB	285 kB	+240 B (+0.1%)
  react-native	227 kB	227 kB	+256 B (+0.1%)

• @firebase/firestore-exp

  Type	Base (d1ae4de)	Head (60170d2)	Diff
  browser	224 kB	224 kB	+251 B (+0.1%)
  main	510 kB	511 kB	+424 B (+0.1%)
  module	224 kB	224 kB	+251 B (+0.1%)
  react-native	224 kB	224 kB	+281 B (+0.1%)

• @firebase/firestore-lite

  Type	Base (d1ae4de)	Head (60170d2)	Diff
  browser	72.6 kB	72.7 kB	+153 B (+0.2%)
  main	149 kB	149 kB	+324 B (+0.2%)
  module	72.6 kB	72.7 kB	+153 B (+0.2%)
  react-native	72.8 kB	72.9 kB	+153 B (+0.2%)

• @firebase/firestore/bundle

  Type	Base (d1ae4de)	Head (60170d2)	Diff
  browser	290 kB	291 kB	+286 B (+0.1%)
  esm2017	177 kB	177 kB	+157 B (+0.1%)
  main	527 kB	527 kB	+424 B (+0.1%)
  module	290 kB	291 kB	+286 B (+0.1%)
  react-native	177 kB	177 kB	+161 B (+0.1%)

• @firebase/firestore/memory

  Type	Base (d1ae4de)	Head (60170d2)	Diff
  browser	216 kB	217 kB	+457 B (+0.2%)
  esm2017	172 kB	172 kB	+157 B (+0.1%)
  main	325 kB	325 kB	+290 B (+0.1%)
  module	216 kB	217 kB	+457 B (+0.2%)
  react-native	172 kB	172 kB	+161 B (+0.1%)

• @firebase/firestore/memory-bundle

  Type	Base (d1ae4de)	Head (60170d2)	Diff
  browser	224 kB	225 kB	+185 B (+0.1%)
  esm2017	177 kB	177 kB	+157 B (+0.1%)
  main	322 kB	322 kB	+290 B (+0.1%)
  module	224 kB	225 kB	+185 B (+0.1%)
  react-native	177 kB	177 kB	+161 B (+0.1%)

• @firebase/storage

  Type	Base (d1ae4de)	Head (60170d2)	Diff
  browser	63.7 kB	63.6 kB	-64 B (-0.1%)
  esm2017	54.9 kB	54.9 kB	-33 B (-0.1%)
  main	55.4 kB	55.4 kB	-33 B (-0.1%)
  module	63.7 kB	63.6 kB	-64 B (-0.1%)

• @firebase/storage-compat

  Type	Base (d1ae4de)	Head (60170d2)	Diff
  browser	10.3 kB	10.2 kB	-52 B (-0.5%)
  main	29.3 kB	29.2 kB	-64 B (-0.2%)
  module	10.3 kB	10.2 kB	-52 B (-0.5%)

• @firebase/storage-exp

  Type	Base (d1ae4de)	Head (60170d2)	Diff
  browser	51.6 kB	51.6 kB	+19 B (+0.0%)
  main	53.0 kB	53.0 kB	+19 B (+0.0%)
  module	51.6 kB	51.6 kB	+19 B (+0.0%)

• firebase

  Type	Base (d1ae4de)	Head (60170d2)	Diff
  firebase-database.js	187 kB	187 kB	-3 B (-0.0%)
  firebase-firestore.js	336 kB	336 kB	+465 B (+0.1%)
  firebase-firestore.memory.js	271 kB	271 kB	+175 B (+0.1%)
  firebase-storage.js	41.6 kB	41.5 kB	-58 B (-0.1%)
  firebase.js	893 kB	894 kB	+278 B (+0.0%)

Test Logs

• Base (d1ae4dea): https://github.com/firebase/firebase-js-sdk/actions/runs/1035329621
• Head (60170d2b): https://github.com/firebase/firebase-js-sdk/actions/runs/1046228665

[google-oss-bot]

Size Analysis Report

Affected Products

No changes between base commit (d1ae4de) and head commit (60170d2).

Test Logs

• Base (d1ae4de): https://github.com/firebase/firebase-js-sdk/actions/runs/1035329621
• Head (60170d2): https://github.com/firebase/firebase-js-sdk/actions/runs/1046228665