Add ENABLE_JAVA_BRIDGE

Add ENABLE_JAVA_BRIDGE

This flag allows us to avoid compiling and executing JavaBridge code when the
feature is not required. The flag defaults to off, but is enabled on Android.

BUG=96703
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=107010

[Steve Block, 2011-10-24 12:57:48]

See http://codereview.chromium.org/8366034 for context

[jam, 2011-10-24 17:43:21]

http://codereview.chromium.org/8383001/diff/1/content/content_browser.gypi
File content/content_browser.gypi (right):

http://codereview.chromium.org/8383001/diff/1/content/content_browser.gypi#ne...
content/content_browser.gypi:725:
'browser/renderer_host/java_bridge_channel_host.cc',
the convention is to list all the file above (i.e. where you just removed it
from), and then use flags to exclude the files (i.e. see safe_browsing as an
example). this way the files still get put in the IDE projects, which allows
people doing find-and-replace to still find the files.

http://codereview.chromium.org/8383001/diff/1/content/renderer/render_view_im...
File content/renderer/render_view_impl.cc (right):

http://codereview.chromium.org/8383001/diff/1/content/renderer/render_view_im...
content/renderer/render_view_impl.cc:674: #if defined(ENABLE_JAVA_BRIDGE)
ditto

http://codereview.chromium.org/8383001/diff/1/content/renderer/render_view_im...
content/renderer/render_view_impl.cc:4658: #if defined(ENABLE_JAVA_BRIDGE)
this can just be around line 4662

http://codereview.chromium.org/8383001/diff/1/content/renderer/render_view_im...
File content/renderer/render_view_impl.h (right):

http://codereview.chromium.org/8383001/diff/1/content/renderer/render_view_im...
content/renderer/render_view_impl.h:867: #if defined(ENABLE_JAVA_BRIDGE)
nit: it's better to have as few ifdefs as possible per feature, so you don't
need this

[Steve Block, 2011-10-24 22:55:33]

Checked with bradnelson, and 'sources!' is preferred to 'exclude' in this case.

[jam, 2011-10-24 22:59:09]

lgtm