DescriptionEnsure low-memory renderers retry failed loads correctly.
LibraryLoader uses isUsingBrowserSharedRelros() to decide whether to retry
a fixed address load for the browser. On a low-memory target, this function
will currently return true for both browser and renderer processes. This
means that any renderer fixed load retry for low-memory will also occur in
LibraryLoader, rather than in ChildProcessService.
This works, but a side effect is that UMA statistics for failed renderer
fixed address load are not updated correctly by ChildProcessService (such
an event will however not register as a failed browser load either,
because the function to push UMA values on browser load is not called for
a child process).
Fixed by making isUsingBrowserSharedRelros() return false if not in the
browser process, regardless of any low-memory/normal-memory setting.
Additionally, make ...ForTesting functions final, and clean up a few
portions of testing code for tighter operation.
BUG=460499
Committed: https://crrev.com/1f11873066a51ebe84e302128957ddc5ee594065
Cr-Commit-Position: refs/heads/master@{#349420}
Patch Set 1 #
Total comments: 2
Patch Set 2 : Update for review comments. #
Messages
Total messages: 10 (3 generated)
|