NEW 188399
Web Inspector: Consider raising the 4gb process memory limit for WebInspector process
https://bugs.webkit.org/show_bug.cgi?id=188399
Summary Web Inspector: Consider raising the 4gb process memory limit for WebInspector...
Saam Barati
Reported 2018-08-07 19:31:11 PDT
I ran into this limit a few times when taking heap snapshots of large heaps.
Attachments
Joseph Pecoraro
Comment 1 2018-08-08 15:14:06 PDT
In WebKit the Web Inspector frontend page is just a WebPageProxy: void WebInspectorProxy::createFrontendPage() { if (m_inspectorPage) return; m_inspectorPage = platformCreateFrontendPage(); ... trackInspectorPage(m_inspectorPage); ... } However these inspector frontend pages are put into special WebProcessPools meant for only Inspector Pages: WebProcessPool& inspectorProcessPool(unsigned inspectionLevel) { // Having our own process pool removes us from the main process pool and // guarantees no process sharing for our user interface. WebProcessPool*& pool = (inspectionLevel == 1) ? s_mainInspectorProcessPool : s_nestedInspectorProcessPool; if (!pool) { auto configuration = API::ProcessPoolConfiguration::createWithLegacyOptions(); pool = &WebProcessPool::create(configuration.get()).leakRef(); } return *pool; } This makes me think that if we can set a bit on the WebProcessProxy / WebProcess that says "You're a Debugger Process". Then when the first WebPage is created in the process pool it could propagate the property to the pressure handler: MemoryPressureHandler::singleton()->setDebuggerProcess(); This could then raise / affect the limit.
Blaze Burg
Comment 2 2018-08-08 16:22:01 PDT
Joe's investigation sounds like a good plan. Is there any reasonable way to test this without screwing up bots? Obviously we can try to take a huge heap snapshot, but considering how slow bots are already compared to dev boxes, I'm not sure a test like this would be stable or useful.
Alexey Proskuryakov
Comment 3 2018-08-08 16:55:47 PDT
I'm also skeptical of testing that uses huge amounts of memory.
Radar WebKit Bug Importer
Comment 4 2020-08-27 14:13:40 PDT
Note You need to log in before you can comment on or make changes to this bug.