Restore switch kDisableWebRtcHWEncoding

content/browser/gpu/gpu_data_manager_impl_private.cc

Index: content/browser/gpu/gpu_data_manager_impl_private.cc
diff --git a/content/browser/gpu/gpu_data_manager_impl_private.cc b/content/browser/gpu/gpu_data_manager_impl_private.cc
index 83ef7cac45db21659575c4acf55ac4562c748e8d..5229871f27b932fafde80f53a3a34871eab8efac 100644
--- a/content/browser/gpu/gpu_data_manager_impl_private.cc
+++ b/content/browser/gpu/gpu_data_manager_impl_private.cc
@@ -778,12 +778,15 @@ void GpuDataManagerImplPrivate::AppendGpuCommandLine(
 
 #if BUILDFLAG(ENABLE_WEBRTC)
   if (IsFeatureBlacklisted(gpu::GPU_FEATURE_TYPE_ACCELERATED_VIDEO_ENCODE) &&

[Pawel Osciak, 2017/01/11 09:55:35]

Sorry, I'm confused by the intended logic of this block.

The original logic was to append kDisableWebRtcHWEncoding if encoding was
blacklisted, but the kDisableWebRtcHWEncoding switch was not on commandline
(https://chromium.googlesource.com/chromium/src/+blame/71d41b68d19128f43405820...).

Now, unless I'm misreading something, it seems that we'll only append
kDisableWebRtcHWEncoding if kDisableWebRtcHWVP8Encoding is not on commandline.
However, if kDisableWebRtcHWVP8Encoding is on commandline, then we won't append
kDisableWebRtcHWEncoding regardless of anything else?

Could you explain the intention of the whole block and add comments please?

[braveyao, 2017/01/11 20:46:06]

First of all, now the blacklist is temporarily for blacklisting HW VP8 only on
Android (in the past it will blacklist all HW codec on Android). 
And on Android, generally you can't append commandline flag by clicking App icon
to launch it.

So the logic here is mainly to append the switch to disable HW VP8 on Android.
And then set gpu_preferences or append kDisableWebRtcHWEncoding depending on HW
H264 status.

And I'm preparing to remove the blacklist soon to enable HW VP8 for
mediaRecording only on Android (no WebRTC yet).

-      !command_line->HasSwitch(switches::kDisableWebRtcHWVP8Encoding) &&
-      !base::FeatureList::IsEnabled(features::kWebRtcHWH264Encoding)) {
-    if (gpu_preferences) {
-      gpu_preferences->disable_web_rtc_hw_encoding = true;
-    } else {
-      command_line->AppendSwitch(switches::kDisableWebRtcHWVP8Encoding);
+      !command_line->HasSwitch(switches::kDisableWebRtcHWVP8Encoding)) {
+    command_line->AppendSwitch(switches::kDisableWebRtcHWVP8Encoding);
+
+    if (!base::FeatureList::IsEnabled(features::kWebRtcHWH264Encoding)) {
+      if (gpu_preferences) {
+        gpu_preferences->disable_web_rtc_hw_encoding = true;
+      } else if (!command_line->HasSwitch(switches::kDisableWebRtcHWEncoding)) {
+        command_line->AppendSwitch(switches::kDisableWebRtcHWEncoding);
+      }
     }
   }
 #endif