Issue 475004: Revert 34161 - Enable Pepper support by default, including building the test ...

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Issue 475004: Revert 34161 - Enable Pepper support by default, including building the test ... (Closed)

Created:
11 years ago by sehr (please use chromium)

Modified:
9 years, 7 months ago

Reviewers:
sehr (please use chromium)

CC:
chromium-reviews_googlegroups.com, brettw+cc_chromium.org, ben+cc_chromium.org, Erik does not do reviews, jam, Aaron Boodman, pam+watch_chromium.org, darin (slow to review)

Base URL:
svn://svn.chromium.org/chrome/trunk/src/

Visibility:
Public.

Description

Revert 34161 - Enable Pepper support by default, including building the test plugin. This is needed because the NaCl plugin code that runs in the renderer needs to use Pepper APIs all the time, and NaCl support has been enabled by default for several months now. To cause an untrusted Pepper plugin to run in the renderer one needs to specify the internalpepper flag. I have also removed the enable_pepper flag from gyp. As the build of the GPU process was tied to this flag, I have renamed the flag to enable_gpu. TEST=none BUG=none Review URL: http://codereview.chromium.org/464074 TBR=sehr@google.com Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=34162

Patch Set 1 #

Created: 11 years ago

Download [raw] [tar.bz2]

	Unified diffs	Side-by-side diffs	Stats (+58 lines, -49 lines)			Patch
M	build/all.gyp	View	2 chunks	+2 lines, -2 lines	0 comments	Download
M	build/common.gypi	View	2 chunks	+5 lines, -2 lines	0 comments	Download
M	chrome/browser/renderer_host/browser_render_process_host.cc	View	1 chunk	+0 lines, -1 line	0 comments	Download
M	chrome/chrome_renderer.gypi	View	1 chunk	+1 line, -1 line	0 comments	Download
M	chrome/common/chrome_switches.h	View	1 chunk	+0 lines, -1 line	0 comments	Download
M	chrome/common/chrome_switches.cc	View	1 chunk	+0 lines, -3 lines	0 comments	Download
M	chrome/renderer/render_view.cc	View	2 chunks	+12 lines, -20 lines	0 comments	Download
M	webkit/glue/plugins/npapi_extension_thunk.cc	View	2 chunks	+17 lines, -0 lines	0 comments	Download
M	webkit/glue/plugins/plugin_host.cc	View	2 chunks	+5 lines, -9 lines	0 comments	Download
M	webkit/glue/webplugin.h	View	1 chunk	+2 lines, -2 lines	0 comments	Download
M	webkit/glue/webplugin_impl.cc	View	2 chunks	+1 line, -3 lines	0 comments	Download
M	webkit/tools/pepper_test_plugin/README	View	1 chunk	+8 lines, -2 lines	0 comments	Download
M	webkit/tools/pepper_test_plugin/event_handler.cc	View	2 chunks	+2 lines, -3 lines	0 comments	Download
M	webkit/tools/pepper_test_plugin/pepper_test_plugin.gyp	View	1 chunk	+3 lines, -0 lines	0 comments	Download

Messages

Total messages: 1 (0 generated)

Expand Messages | Collapse Messages