WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
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
Add attachment
proposed patch, testcase, etc.
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
<
rdar://problem/67896698
>
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug