Remove GPU_FEATURE_TYPE_3D_CSS and --disable-accelerated-layers

Remove GPU_FEATURE_TYPE_3D_CSS and --disable-accelerated-layers

We ignore GPU_FEATURE_TYPE_3D_CSS, so remove it.
Nothing sets --disable-accelerated-layers, and it doesn't correspond to
a production path, so remove it.
This lets us clean up some WebPreferences fields which are always set to
true.

Also remove WebPreferences::accelerated_compositing_for_plugins_enabled
which is always true.

BUG=362164, 365847
R=bbudge@chromium.org, danakj@chromium.org, tsepez@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=266226

[piman, 2014-04-16 23:00:30]

danakj: please review
darin: webkit/ OWNER
bbudge: ppapi/ OWNER
tsepez: content/public/common/common_param_traits_macros.h security OWNER

[bbudge, 2014-04-16 23:17:44]

ppapi LGTM
Thanks for deleting this little mess.

[danakj, 2014-04-16 23:40:24]

LGTM

[Tom Sepez, 2014-04-16 23:42:39]

Rubberstamp LGTM on deleting members.

[danakj, 2014-04-22 21:58:05]

Can we land?

[piman, 2014-04-22 22:41:34]

On Tue, Apr 22, 2014 at 2:58 PM, <danakj@chromium.org> wrote:

> Can we land?
>

It was written assuming 236363019 which is still in the q.
I can try to see if I can out-of-order it.


> https://codereview.chromium.org/239973005/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[piman, 2014-04-23 01:32:25]

The CQ bit was checked by piman@chromium.org

[commit-bot: I haz the power, 2014-04-23 01:34:36]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/piman@chromium.org/239973005/20001

[commit-bot: I haz the power, 2014-04-23 01:50:00]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-04-23 01:50:01]

Try jobs failed on following builders:
  tryserver.chromium on chromium_presubmit

[piman, 2014-04-23 02:40:25]

The CQ bit was checked by piman@chromium.org

[commit-bot: I haz the power, 2014-04-23 02:41:29]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/piman@chromium.org/239973005/40001

[commit-bot: I haz the power, 2014-04-23 02:44:33]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-04-23 02:44:34]

Try jobs failed on following builders:
  tryserver.chromium on linux_chromium_chromeos_clang_dbg

[piman, 2014-04-23 03:31:42]

The CQ bit was checked by piman@chromium.org

[piman, 2014-04-23 03:32:21]

Oh, still waiting for OWNERS in webkit. darin: ping?

[commit-bot: I haz the power, 2014-04-23 03:32:39]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/piman@chromium.org/239973005/40001

[commit-bot: I haz the power, 2014-04-23 03:49:06]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-04-23 03:49:07]

Try jobs failed on following builders:
  tryserver.chromium on chromium_presubmit

[danakj, 2014-04-24 19:36:32]

The CQ bit was checked by danakj@chromium.org

[commit-bot: I haz the power, 2014-04-24 21:07:56]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/piman@chromium.org/239973005/40001

[commit-bot: I haz the power, 2014-04-24 21:08:18]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-04-24 21:08:19]

Failed to apply patch for content/browser/gpu/gpu_data_manager_impl_private.cc:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file content/browser/gpu/gpu_data_manager_impl_private.cc
  Hunk #1 FAILED at 757.
  1 out of 1 hunk FAILED -- saving rejects to file
content/browser/gpu/gpu_data_manager_impl_private.cc.rej

Patch:       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
f6b39ce7f59c4dcabf6d12060fbde902b1739e04..67237897678c0a2862a717155fdef8471aec3e43
100644
--- a/content/browser/gpu/gpu_data_manager_impl_private.cc
+++ b/content/browser/gpu/gpu_data_manager_impl_private.cc
@@ -757,9 +757,6 @@ void GpuDataManagerImplPrivate::UpdateRendererWebPrefs(
     prefs->gl_multisampling_enabled = false;
 
   prefs->accelerated_compositing_enabled = true;
-  prefs->accelerated_compositing_for_3d_transforms_enabled = true;
-  prefs->accelerated_compositing_for_animation_enabled = true;
-  prefs->accelerated_compositing_for_plugins_enabled = true;
   prefs->accelerated_compositing_for_video_enabled = true;
 
 #if defined(USE_AURA)

[piman, 2014-04-24 23:27:33]

The CQ bit was checked by piman@chromium.org

[commit-bot: I haz the power, 2014-04-24 23:27:57]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/piman@chromium.org/239973005/60001

[commit-bot: I haz the power, 2014-04-25 00:31:11]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-04-25 00:31:12]

Try jobs failed on following builders:
  tryserver.chromium on win_chromium_rel

[piman, 2014-04-25 06:12:09]

The CQ bit was checked by piman@chromium.org

[commit-bot: I haz the power, 2014-04-25 08:22:03]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/piman@chromium.org/239973005/60001

[piman, 2014-04-25 20:02:28]

Committed patchset #5 manually as r266226 (presubmit successful).