Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(673)

Issue 2784773002: Fix leaks in mojo::edk::NodeController (Closed)

Created:
3 years, 8 months ago by Ken Rockot(use gerrit already)
Modified:
3 years, 8 months ago
Reviewers:
CC:
chromium-reviews
Target Ref:
refs/branch-heads/3029
Project:
chromium
Visibility:
Public.

Description

Fix leaks in mojo::edk::NodeController There are a few instances where the broker process (e.g. the browser in process Chrome) tracks some state -- including outgoing messages -- pertaining to a client process connection. In some cases this tracked data is not cleaned up once the client process is disconnected. This fixes that. BUG=704938 Review-Url: https://codereview.chromium.org/2780433003 Cr-Commit-Position: refs/heads/master@{#460001} (cherry picked from commit 69726f4ba3947fd4ddbfb07fd9f1a596289cdae9) Review-Url: https://codereview.chromium.org/2784773002 . Cr-Commit-Position: refs/branch-heads/3029@{#471} Cr-Branched-From: 939b32ee5ba05c396eef3fd992822fcca9a2e262-refs/heads/master@{#454471} Committed: https://chromium.googlesource.com/chromium/src/+/97716adf5756137e3dfde3526e3d5007e75988a6

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+14 lines, -0 lines) Patch
M mojo/edk/system/node_controller.cc View 3 chunks +14 lines, -0 lines 0 comments Download

Messages

Total messages: 2 (1 generated)
Ken Rockot(use gerrit already)
3 years, 8 months ago (2017-03-29 05:56:19 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
97716adf5756137e3dfde3526e3d5007e75988a6.

Powered by Google App Engine
This is Rietveld 408576698