[tracing] Remove meaningless background tracing upload url
Change-Id: I598f973e0ee84105a62edd3a07d0926f334736bb
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4076487
Commit-Queue: Etienne Pierre-Doray <etiennep@chromium.org>
Reviewed-by: Eric Seckler <eseckler@chromium.org>
Reviewed-by: Alexander Timin <altimin@chromium.org>
Reviewed-by: Francois Pierre Doray <fdoray@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1079801}
diff --git a/components/performance_manager/graph/process_node_impl_unittest.cc b/components/performance_manager/graph/process_node_impl_unittest.cc
index 39e27c86..212f1ff 100644
--- a/components/performance_manager/graph/process_node_impl_unittest.cc
+++ b/components/performance_manager/graph/process_node_impl_unittest.cc
@@ -319,10 +319,6 @@
}
bool HasTraceToUpload() override { return false; }
std::string GetLatestTraceToUpload() override { return std::string(); }
- std::string GetBackgroundTracingUploadUrl(
- const std::string& trial_name) override {
- return std::string();
- }
std::unique_ptr<content::BackgroundTracingConfig> GetBackgroundTracingConfig(
const std::string& trial_name) override {
return nullptr;
diff --git a/content/browser/tracing/background_tracing_manager_impl.cc b/content/browser/tracing/background_tracing_manager_impl.cc
index 06846b4..4fd8fa60 100644
--- a/content/browser/tracing/background_tracing_manager_impl.cc
+++ b/content/browser/tracing/background_tracing_manager_impl.cc
@@ -50,7 +50,6 @@
namespace {
const char kBackgroundTracingConfig[] = "config";
-const char kBackgroundTracingUploadUrl[] = "upload_url";
} // namespace
@@ -327,12 +326,6 @@
}
}
-std::string BackgroundTracingManagerImpl::GetBackgroundTracingUploadUrl(
- const std::string& trial_name) {
- return variations::GetVariationParamValue(trial_name,
- kBackgroundTracingUploadUrl);
-}
-
std::unique_ptr<content::BackgroundTracingConfig>
BackgroundTracingManagerImpl::GetBackgroundTracingConfig(
const std::string& trial_name) {
diff --git a/content/browser/tracing/background_tracing_manager_impl.h b/content/browser/tracing/background_tracing_manager_impl.h
index fe9f2371d..85294444 100644
--- a/content/browser/tracing/background_tracing_manager_impl.h
+++ b/content/browser/tracing/background_tracing_manager_impl.h
@@ -124,8 +124,6 @@
bool HasTraceToUpload() override;
std::string GetLatestTraceToUpload() override;
void SetTraceToUpload(std::unique_ptr<std::string> trace_data);
- std::string GetBackgroundTracingUploadUrl(
- const std::string& trial_name) override;
std::unique_ptr<BackgroundTracingConfig> GetBackgroundTracingConfig(
const std::string& trial_name) override;
diff --git a/content/public/browser/background_tracing_manager.h b/content/public/browser/background_tracing_manager.h
index d7c380e..c11196e 100644
--- a/content/public/browser/background_tracing_manager.h
+++ b/content/public/browser/background_tracing_manager.h
@@ -117,11 +117,6 @@
// the serialized trace proto.
virtual std::string GetLatestTraceToUpload() = 0;
- // Returns the URL for uploading the trace files corresponding to the
- // experiment |trial_name|.
- virtual std::string GetBackgroundTracingUploadUrl(
- const std::string& trial_name) = 0;
-
// Returns background tracing configuration for the experiment |trial_name|.
virtual std::unique_ptr<BackgroundTracingConfig> GetBackgroundTracingConfig(
const std::string& trial_name) = 0;
