Restore switch kDisableWebRtcHWEncoding

content/public/browser/gpu_utils.cc

Index: content/public/browser/gpu_utils.cc
diff --git a/content/public/browser/gpu_utils.cc b/content/public/browser/gpu_utils.cc
index 8605b2d52b10d3dfb73b068ef24467ad417b62c0..19c6ab65d6297bbf6eeca200f23bed8e0e7037a0 100644
--- a/content/public/browser/gpu_utils.cc
+++ b/content/public/browser/gpu_utils.cc
@@ -47,8 +47,9 @@ const gpu::GpuPreferences GetGpuPreferencesFromCommandLine() {
 #endif
 #if BUILDFLAG(ENABLE_WEBRTC)
   gpu_preferences.disable_web_rtc_hw_encoding =
-      command_line->HasSwitch(switches::kDisableWebRtcHWVP8Encoding) &&
-      !base::FeatureList::IsEnabled(features::kWebRtcHWH264Encoding);
+      command_line->HasSwitch(switches::kDisableWebRtcHWEncoding) ||

[Pawel Osciak, 2016/12/15 05:45:53]

Could we keep only kDisableWebRtcHWEncoding here as well please?

[braveyao, 2016/12/15 20:46:24]

Probably we can. The consequence, as I mentioned yesterday, will be if we
disable H264(not append kDisableWebRtcHWEncoding. And so far VP8 is  disabled by
default), then VEA will be created even no one will use it. This changed the
previous behavior.
What if we always create VEA on Android? Will it add CPU load or power
consumption? I suppose the principle is we only create the resource when we
about to use it.

[Pawel Osciak, 2016/12/20 01:49:17]

These switches control not only Android. VP8 encode is enabled by default on
CrOS.
Sorry, I'm not familiar how this would affect Android in particular for
power/CPU, but if you needed to disable encoding in general and not create a VEA
at all, kDisableWebRtcHWEncoding should be more appropriate, instead of a
codec-specific flag.

[braveyao, 2016/12/20 19:09:14]

Again on CrOS, nothing is changed. You control with kDisableWebRtcHWEncoing
only.

On Android, we can control VEA creation not only by kDisableWebRtcHWEncoding as
before, but also by checking if none of VP8/H264 is enabled. This just covers
another use case.

+      (command_line->HasSwitch(switches::kDisableWebRtcHWVP8Encoding) &&
+       !base::FeatureList::IsEnabled(features::kWebRtcHWH264Encoding));
 #endif
 #if defined(OS_WIN)
   uint32_t enable_accelerated_vpx_decode_val =