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

Issue 10831224: Add pepper permissions to WebPluginInfo (Closed)

Created:
8 years, 4 months ago by brettw
Modified:
8 years, 4 months ago
Reviewers:
cpu_(ooo_6.6-7.5)
CC:
chromium-reviews, joi+watch-content_chromium.org, darin-cc_chromium.org, jam
Visibility:
Public.

Description

Add pepper permissions to WebPluginInfo This allows us to round-trip from PepperPluginInfo to WebPluginInfo and back. This happens in the component updater when it registers Flash. BUG=138902 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=150922

Patch Set 1 #

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+30 lines, -10 lines) Patch
M chrome/browser/component_updater/pepper_flash_component_installer.cc View 1 2 chunks +3 lines, -4 lines 0 comments Download
M chrome/common/chrome_content_client.cc View 1 1 chunk +1 line, -3 lines 0 comments Download
M chrome/common/pepper_flash.h View 1 1 chunk +5 lines, -0 lines 0 comments Download
M chrome/common/pepper_flash.cc View 1 2 chunks +7 lines, -0 lines 0 comments Download
M content/common/pepper_plugin_registry.cc View 2 chunks +3 lines, -0 lines 0 comments Download
M webkit/plugins/webplugininfo.h View 1 chunk +3 lines, -0 lines 0 comments Download
M webkit/plugins/webplugininfo.cc View 4 chunks +8 lines, -3 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
brettw
8 years, 4 months ago (2012-08-08 23:44:08 UTC) #1
cpu_(ooo_6.6-7.5)
lgtm
8 years, 4 months ago (2012-08-09 19:26:45 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/brettw@chromium.org/10831224/2001
8 years, 4 months ago (2012-08-09 19:43:21 UTC) #3
commit-bot: I haz the power
8 years, 4 months ago (2012-08-09 22:33:05 UTC) #4
Change committed as 150922

Powered by Google App Engine
This is Rietveld 408576698