telemetry: Remove shared storage support and tests from Telemetry [catapult : main]

0 views
Skip to first unread message

Cammie Smith Barnes (Gerrit)

unread,
Jul 7, 2026, 5:04:57 PM (4 days ago) Jul 7
to Wenbin Zhang, catapul...@luci-project-accounts.iam.gserviceaccount.com, catapult...@chromium.org, telemetr...@chromium.org
Attention needed from Wenbin Zhang

Cammie Smith Barnes voted and added 1 comment

Votes added by Cammie Smith Barnes

Commit-Queue+1

1 comment

Patchset-level comments
File-level comment, Patchset 1 (Latest):
Cammie Smith Barnes . resolved

wenbinzhang@, please take a look, thanks!

Open in Gerrit

Related details

Attention is currently required from:
  • Wenbin Zhang
Submit Requirements:
  • requirement is not satisfiedCode-Owners
  • requirement is not satisfiedCode-Review
  • requirement is not satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: catapult
Gerrit-Branch: main
Gerrit-Change-Id: I039334400ef38d66a70ff39ecc4a4c1e2c688f00
Gerrit-Change-Number: 8058765
Gerrit-PatchSet: 1
Gerrit-Owner: Cammie Smith Barnes <cam...@chromium.org>
Gerrit-Reviewer: Cammie Smith Barnes <cam...@chromium.org>
Gerrit-Reviewer: Wenbin Zhang <wenbi...@google.com>
Gerrit-Attention: Wenbin Zhang <wenbi...@google.com>
Gerrit-Comment-Date: Tue, 07 Jul 2026 21:04:55 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
unsatisfied_requirement
open
diffy

Wenbin Zhang (Gerrit)

unread,
Jul 9, 2026, 6:05:38 PM (2 days ago) Jul 9
to Cammie Smith Barnes, Code Review Nudger, catapul...@luci-project-accounts.iam.gserviceaccount.com, catapult...@chromium.org, telemetr...@chromium.org
Attention needed from Cammie Smith Barnes

Wenbin Zhang voted Code-Review+1

Code-Review+1
Open in Gerrit

Related details

Attention is currently required from:
  • Cammie Smith Barnes
Submit Requirements:
  • requirement satisfiedCode-Owners
  • requirement satisfiedCode-Review
  • requirement satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: catapult
Gerrit-Branch: main
Gerrit-Change-Id: I039334400ef38d66a70ff39ecc4a4c1e2c688f00
Gerrit-Change-Number: 8058765
Gerrit-PatchSet: 1
Gerrit-Owner: Cammie Smith Barnes <cam...@chromium.org>
Gerrit-Reviewer: Cammie Smith Barnes <cam...@chromium.org>
Gerrit-Reviewer: Wenbin Zhang <wenbi...@google.com>
Gerrit-CC: Code Review Nudger <android-build...@prod.google.com>
Gerrit-Attention: Cammie Smith Barnes <cam...@chromium.org>
Gerrit-Comment-Date: Thu, 09 Jul 2026 22:05:34 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
open
diffy

Cammie Smith Barnes (Gerrit)

unread,
Jul 9, 2026, 7:45:30 PM (2 days ago) Jul 9
to Wenbin Zhang, Code Review Nudger, catapul...@luci-project-accounts.iam.gserviceaccount.com, catapult...@chromium.org, telemetr...@chromium.org

Cammie Smith Barnes voted Commit-Queue+2

Commit-Queue+2
Open in Gerrit

Related details

Attention set is empty
Submit Requirements:
  • requirement satisfiedCode-Owners
  • requirement satisfiedCode-Review
  • requirement satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: catapult
Gerrit-Branch: main
Gerrit-Change-Id: I039334400ef38d66a70ff39ecc4a4c1e2c688f00
Gerrit-Change-Number: 8058765
Gerrit-PatchSet: 1
Gerrit-Owner: Cammie Smith Barnes <cam...@chromium.org>
Gerrit-Reviewer: Cammie Smith Barnes <cam...@chromium.org>
Gerrit-Reviewer: Wenbin Zhang <wenbi...@google.com>
Gerrit-CC: Code Review Nudger <android-build...@prod.google.com>
Gerrit-Comment-Date: Thu, 09 Jul 2026 23:45:25 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
open
diffy

catapult-scoped@luci-project-accounts.iam.gserviceaccount.com (Gerrit)

unread,
Jul 9, 2026, 8:09:35 PM (2 days ago) Jul 9
to Cammie Smith Barnes, Wenbin Zhang, Code Review Nudger, catapult...@chromium.org, telemetr...@chromium.org

catapul...@luci-project-accounts.iam.gserviceaccount.com submitted the change

Change information

Commit message:
telemetry: Remove shared storage support and tests from Telemetry

Remove Shared Storage support from Telemetry, including inspector
backends, tab wrappers, and associated unit tests.

These methods were temporarily disabled due to Chromium changes
(crbug.com/401011862) and are currently unused. Moreover, shared storage
is deprecated and will soon be reduced to stubs pending removal.
Bug: chromium:462465887
Change-Id: I039334400ef38d66a70ff39ecc4a4c1e2c688f00
Reviewed-by: Wenbin Zhang <wenbi...@google.com>
Commit-Queue: Cammie Smith Barnes <cam...@chromium.org>
Files:
  • M telemetry/telemetry/internal/backends/chrome_inspector/inspector_backend.py
  • M telemetry/telemetry/internal/backends/chrome_inspector/inspector_storage.py
  • M telemetry/telemetry/internal/browser/tab.py
  • M telemetry/telemetry/internal/browser/tab_unittest.py
Change size: L
Delta: 4 files changed, 5 insertions(+), 652 deletions(-)
Branch: refs/heads/main
Submit Requirements:
  • requirement satisfiedCode-Review: +1 by Wenbin Zhang
Open in Gerrit
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: merged
Gerrit-Project: catapult
Gerrit-Branch: main
Gerrit-Change-Id: I039334400ef38d66a70ff39ecc4a4c1e2c688f00
Gerrit-Change-Number: 8058765
Gerrit-PatchSet: 2
Gerrit-Owner: Cammie Smith Barnes <cam...@chromium.org>
Gerrit-Reviewer: Cammie Smith Barnes <cam...@chromium.org>
Gerrit-Reviewer: Wenbin Zhang <wenbi...@google.com>
open
diffy
satisfied_requirement
Reply all
Reply to author
Forward
0 new messages