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

Issue 22070002: Avoid wasted work in FreePluginLibraryHelper(). (Closed)

Created:
7 years, 4 months ago by pchang9
Modified:
7 years, 4 months ago
Reviewers:
Bernhard Bauer
CC:
chromium-reviews, joi+watch-content_chromium.org, darin-cc_chromium.org, jam
Base URL:
https://src.chromium.org/chrome/trunk/src/
Visibility:
Public.

Description

Avoid wasted work in FreePluginLibraryHelper(). BUG=268091 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=215625

Patch Set 1 #

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -1 line) Patch
M AUTHORS View 1 1 chunk +1 line, -0 lines 0 comments Download
M content/child/npapi/plugin_lib.cc View 1 1 chunk +3 lines, -1 line 0 comments Download

Messages

Total messages: 7 (0 generated)
pchang9
7 years, 4 months ago (2013-08-04 02:40:00 UTC) #1
Bernhard Bauer
lgtm
7 years, 4 months ago (2013-08-04 08:35:17 UTC) #2
Bernhard Bauer
Thanks for your contribution! In order for us to accept it, you need to add ...
7 years, 4 months ago (2013-08-04 08:43:25 UTC) #3
pchang9
On 2013/08/04 08:43:25, Bernhard Bauer wrote: > Thanks for your contribution! In order for us ...
7 years, 4 months ago (2013-08-04 14:14:49 UTC) #4
Bernhard Bauer
On 2013/08/04 14:14:49, pochang0403 wrote: > On 2013/08/04 08:43:25, Bernhard Bauer wrote: > > Thanks ...
7 years, 4 months ago (2013-08-05 08:08:52 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/pochang0403@gmail.com/22070002/7001
7 years, 4 months ago (2013-08-05 13:53:11 UTC) #6
commit-bot: I haz the power
7 years, 4 months ago (2013-08-05 16:36:55 UTC) #7
Message was sent while issue was closed.
Change committed as 215625

Powered by Google App Engine
This is Rietveld 408576698