Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(310)

Unified Diff: chrome/browser/devtools/devtools_targets_ui.cc

Issue 411283002: Remove disable-embedded-shared-worker flag and shared worker process related codes. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/about_flags.cc ('k') | chrome/browser/memory_details.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/devtools/devtools_targets_ui.cc
diff --git a/chrome/browser/devtools/devtools_targets_ui.cc b/chrome/browser/devtools/devtools_targets_ui.cc
index 9e3b8359ce54c124fc29e4604fec4c87592d4116..e2161c0845d2e06232d3c6e41eec269dab39654a 100644
--- a/chrome/browser/devtools/devtools_targets_ui.cc
+++ b/chrome/browser/devtools/devtools_targets_ui.cc
@@ -246,19 +246,12 @@ class WorkerObserver
// WorkerTargetsUIHandler -----------------------------------------------------
class WorkerTargetsUIHandler
- : public DevToolsTargetsUIHandler,
- public content::BrowserChildProcessObserver {
+ : public DevToolsTargetsUIHandler {
public:
explicit WorkerTargetsUIHandler(const Callback& callback);
virtual ~WorkerTargetsUIHandler();
private:
- // content::BrowserChildProcessObserver overrides.
- virtual void BrowserChildProcessHostConnected(
- const content::ChildProcessData& data) OVERRIDE;
- virtual void BrowserChildProcessHostDisconnected(
- const content::ChildProcessData& data) OVERRIDE;
-
void UpdateTargets(const DevToolsTargetImpl::List& targets);
scoped_refptr<WorkerObserver> observer_;
@@ -269,26 +262,12 @@ WorkerTargetsUIHandler::WorkerTargetsUIHandler(const Callback& callback)
observer_(new WorkerObserver()) {
observer_->Start(base::Bind(&WorkerTargetsUIHandler::UpdateTargets,
base::Unretained(this)));
- BrowserChildProcessObserver::Add(this);
}
WorkerTargetsUIHandler::~WorkerTargetsUIHandler() {
- BrowserChildProcessObserver::Remove(this);
observer_->Stop();
}
-void WorkerTargetsUIHandler::BrowserChildProcessHostConnected(
- const content::ChildProcessData& data) {
- if (data.process_type == content::PROCESS_TYPE_WORKER)
- observer_->Enumerate();
-}
-
-void WorkerTargetsUIHandler::BrowserChildProcessHostDisconnected(
- const content::ChildProcessData& data) {
- if (data.process_type == content::PROCESS_TYPE_WORKER)
- observer_->Enumerate();
-}
-
void WorkerTargetsUIHandler::UpdateTargets(
const DevToolsTargetImpl::List& targets) {
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
« no previous file with comments | « chrome/browser/about_flags.cc ('k') | chrome/browser/memory_details.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698