Targets should not depend on protobuf when enable_protobuf=0.

Targets should not depend on protobuf when enable_protobuf=0.

BUG=webrtc:4741
R=henrik.lundin@webrtc.org, stefan@webrtc.org, ivoc@webrtc.org

Committed: https://chromium.googlesource.com/external/webrtc/+/72a8cee42587f8512d69102e5cc346415ab71c26

[stefan-webrtc, 2015-07-03 13:39:08]

stefan@webrtc.org changed reviewers:
+ stefan@webrtc.org

[stefan-webrtc, 2015-07-03 13:39:09]

lgtm

[hlundin-webrtc, 2015-07-03 15:13:26]

lgtm with a nit (or maybe a note to ivoc@ and terelius@ going forward).

https://codereview.webrtc.org/1219333003/diff/40001/webrtc/modules/audio_codi...
File webrtc/modules/audio_coding/main/acm2/acm_dump.cc (right):

https://codereview.webrtc.org/1219333003/diff/40001/webrtc/modules/audio_codi...
webrtc/modules/audio_coding/main/acm2/acm_dump.cc:218: bool
AcmDump::ParseAcmDump(const std::string& file_name,
AcmDump::ParseAcmDump() no longer has an implementation when
RTC_AUDIOCODING_DEBUG_DUMP is 0. I guess this is fine for now, since the
function is never called unless the precompiler define is set, but it might be
good to fix this going forward.

[andresp, 2015-07-03 15:53:28]

Committed patchset #3 (id:40001) manually as
72a8cee42587f8512d69102e5cc346415ab71c26.