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

Issue 196145: Gears File Drag Drop... (Closed)

Created:
11 years, 3 months ago by michaeln
Modified:
9 years, 7 months ago
Reviewers:
noel.gordon
CC:
chromium-reviews_googlegroups.com, darin (slow to review), jam, Ben Goodger (Google)
Visibility:
Public.

Description

Gears File Drag Drop r24899 now keys the child security policy database based on renderer id, rather than renderer process id. Update gears drag drop to use the renderer id for the file policy access checks. This is a clone for submission of of noel's original CL http://codereview.chromium.org/195079 BUG=7995 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=26397

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+36 lines, -26 lines) Patch
M chrome/browser/plugin_process_host.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/plugin_process_host.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M chrome/common/plugin_messages_internal.h View 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/plugin/chrome_plugin_host.cc View 2 chunks +3 lines, -3 lines 0 comments Download
M chrome/plugin/plugin_channel.h View 4 chunks +8 lines, -3 lines 0 comments Download
M chrome/plugin/plugin_channel.cc View 1 chunk +13 lines, -6 lines 0 comments Download
M chrome/plugin/plugin_thread.h View 2 chunks +2 lines, -4 lines 0 comments Download
M chrome/plugin/webplugin_proxy.h View 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/plugin/webplugin_proxy.cc View 1 chunk +3 lines, -3 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
michaeln
Noel... fyi...
11 years, 3 months ago (2009-09-16 19:35:56 UTC) #1
noel.gordon_gmail.com
11 years, 3 months ago (2009-09-16 22:45:38 UTC) #2
LGTM and thanks.

Powered by Google App Engine
This is Rietveld 408576698