perf: remove GC timer that fired once per minute. (#26051)

Co-authored-by: Charles Kerr <charles@charleskerr.com>
electron · Oct 20, 2020 · 65fdfc1 · 65fdfc1
1 parent 139e203
commit 65fdfc1
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 7 deletions.
diff --git a/shell/browser/electron_browser_main_parts.cc b/shell/browser/electron_browser_main_parts.cc
@@ -471,11 +471,6 @@ void ElectronBrowserMainParts::PreMainMessageLoopRun() {
   ui::TouchFactory::SetTouchDeviceListFromCommandLine();
 #endif
 
-  // Start idle gc.
-  gc_timer_.Start(FROM_HERE, base::TimeDelta::FromMinutes(1),
-                  base::BindRepeating(&v8::Isolate::LowMemoryNotification,
-                                      base::Unretained(js_env_->isolate())));
-
   content::WebUIControllerFactory::RegisterFactory(
       ElectronWebUIControllerFactory::GetInstance());
 

diff --git a/shell/browser/electron_browser_main_parts.h b/shell/browser/electron_browser_main_parts.h
@@ -160,8 +160,6 @@ class ElectronBrowserMainParts : public content::BrowserMainParts {
   std::unique_ptr<ElectronExtensionsBrowserClient> extensions_browser_client_;
 #endif
 
-  base::RepeatingTimer gc_timer_;
-
   // List of callbacks should be executed before destroying JS env.
   std::list<base::OnceClosure> destructors_;