DescriptionRevert of Allow handles through for official builds too (partial revert) (patchset #3 id:40001 of https://codereview.chromium.org/2198603002/ )
Reason for revert:
Potentially causing https://crbug.com/645319.
Graph to watch to see if it helps: https://goto.google.com/nezdg
BUG=645319
Original issue's description:
> Allow handles through for official builds too (partial revert)
>
> This is a (very) partial revert of
> https://codereview.chromium.org/1291553003 to fix the crash reported in
> https://bugs.chromium.org/p/chromium/issues/detail?id=625172#c21. I
> started trying to dig into making renderers log, but it's still a hairy
> mess.
>
> R=wfh@chromium.org
> BUG=625172, 358267, 579196
> TEST=out\Release\chrome.exe http://www.bbc.co.uk/news/world-asia-china-36189636 --enable-logging=stderr > log.txt 2>&1 where chrome is is_official_build=true shouldn't crash.
>
> Committed: https://crrev.com/5f2967d15fe53b2bd0cec15dd22fdff3b86e1ca9
> Cr-Commit-Position: refs/heads/master@{#409318}
TBR=wfh@chromium.org,sky@chromium.org,thestig@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=625172, 358267, 579196
Review-Url: https://codereview.chromium.org/2767663002
Cr-Commit-Position: refs/heads/master@{#458513}
Committed: https://chromium.googlesource.com/chromium/src/+/00c0c98ea999d1d8edde6d1a2c9d3e4ff1449117
Patch Set 1 #Patch Set 2 : rebase #
Messages
Total messages: 22 (15 generated)
|