Refactor FeedbackReport, FeedbackUpload to remove chrome/net deps
Remove use of BrowserContext and BrowserThread from FeedbackReport and
FeedbackUpload. Moves net/ specific functionality into a new class
FeedbackUploadChrome, and moves using the browser context out of
the base classes.
CrOS should be able to directly use feedback_report.cc/h &
feedback_uploader.cc/h, providing a curl-based upload version.
BUG=341554
Review URL: https://codereview.chromium.org/186663011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@257534 0039d316-1c4b-4281-b951-d872f2087c98
diff --git a/chrome/browser/feedback/feedback_uploader_factory.cc b/chrome/browser/feedback/feedback_uploader_factory.cc
index 4a715ff..bc20ecc 100644
--- a/chrome/browser/feedback/feedback_uploader_factory.cc
+++ b/chrome/browser/feedback/feedback_uploader_factory.cc
@@ -6,6 +6,7 @@
#include "base/memory/singleton.h"
#include "chrome/browser/feedback/feedback_uploader.h"
+#include "chrome/browser/feedback/feedback_uploader_chrome.h"
#include "chrome/browser/profiles/incognito_helpers.h"
#include "components/keyed_service/content/browser_context_dependency_manager.h"
@@ -19,7 +20,7 @@
// static
FeedbackUploader* FeedbackUploaderFactory::GetForBrowserContext(
content::BrowserContext* context) {
- return static_cast<FeedbackUploader*>(
+ return static_cast<FeedbackUploaderChrome*>(
GetInstance()->GetServiceForBrowserContext(context, true));
}
@@ -32,7 +33,7 @@
KeyedService* FeedbackUploaderFactory::BuildServiceInstanceFor(
content::BrowserContext* context) const {
- return new FeedbackUploader(context);
+ return new FeedbackUploaderChrome(context);
}
content::BrowserContext* FeedbackUploaderFactory::GetBrowserContextToUse(
