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

Issue 14869007: Merge 197686 "Remove Pepper URLLoader from resource tracker early." (Closed)

Created:
7 years, 7 months ago by Chris Evans
Modified:
7 years, 7 months ago
Reviewers:
bbudge
CC:
chromium-reviews, darin-cc_chromium.org
Visibility:
Public.

Description

Merge 197686 "Remove Pepper URLLoader from resource tracker early." > Remove Pepper URLLoader from resource tracker early. > This protects against double delete if the instance is destroyed > as a result of canceling a load. > > BUG=159429, 227350 > > Review URL: https://chromiumcodereview.appspot.com/14695002 TBR=bbudge@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=198490

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+20 lines, -7 lines) Patch
M ppapi/shared_impl/resource.h View 1 chunk +8 lines, -0 lines 0 comments Download
M ppapi/shared_impl/resource.cc View 2 chunks +5 lines, -1 line 0 comments Download
M webkit/plugins/ppapi/ppb_url_loader_impl.cc View 1 chunk +7 lines, -6 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Chris Evans
7 years, 7 months ago (2013-05-06 18:45:51 UTC) #1
Chris Evans
7 years, 7 months ago (2013-05-06 18:45:59 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 manually as r198490.

Powered by Google App Engine
This is Rietveld 408576698