[bundle-size][database] Refine bundle definition

[yifanyang]

No description provided.

[changeset-bot]

⚠️ No Changeset found

Latest commit: f60db90

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

[google-oss-bot]

Binary Size Report

Affected SDKs

• bundle

  Type	Base (6ae54d0)	Head (72fa76e)	Diff
  database (Listen for value events + Detach listeners)	?	152 kB	? (?)

Test Logs

• Base (6ae54d04): https://github.com/firebase/firebase-js-sdk/actions/runs/1533121342
• Head (72fa76ea): https://github.com/firebase/firebase-js-sdk/actions/runs/1533122287

[google-oss-bot]

Size Analysis Report

Affected Products

Diffs between base commit (6ae54d0) and head commit (72fa76e) are too large (1,138,758 characters) to display.

Please check below links to see details from the original test log.

Test Logs

• Head (72fa76e): https://github.com/firebase/firebase-js-sdk/actions/runs/1533122287

[schmidt-sebastian]

Would it make sense to add another definition that just uses getDatabase+ref+onValue+off? off is probably not used by most users, so I don't want to make it the default.

[yifanyang]

Done. Made it a separate test case now.

p.s. I had to force push the new change which is rebased onto a more recent commit on master branch. This is to let the diff report to pick the right comparison target commit. See here for more detail.