New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 590825 link

Starred by 4 users

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug
mus

Blocking:
issue 586369



Sign in to add a comment

Get rid of gpu related switches

Project Member Reported by penghuang@chromium.org, Feb 29 2016

Issue description

Get rid of gpu related switches by passing gpu::GpuPreferences via IPC, at same time get rid of duplcate code in deferred_gpu_command_service.cc & content/gpu/gpu_child_thread.cc.

 
Status: Started (was: Assigned)
Project Member

Comment 2 by bugdroid1@chromium.org, Mar 7 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/248b114c9d3034964c7259222712dfcccfad8f48

commit 248b114c9d3034964c7259222712dfcccfad8f48
Author: penghuang <penghuang@chromium.org>
Date: Mon Mar 07 21:49:34 2016

Get rid of gpu related switches by passing gpu::GpuPreferences via IPC.

BUG= 590825 
CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel

Review URL: https://codereview.chromium.org/1753383003

Cr-Commit-Position: refs/heads/master@{#379651}

[modify] https://crrev.com/248b114c9d3034964c7259222712dfcccfad8f48/android_webview/browser/deferred_gpu_command_service.cc
[modify] https://crrev.com/248b114c9d3034964c7259222712dfcccfad8f48/content/browser/android/synchronous_compositor_base.cc
[modify] https://crrev.com/248b114c9d3034964c7259222712dfcccfad8f48/content/browser/gpu/gpu_process_host.cc
[modify] https://crrev.com/248b114c9d3034964c7259222712dfcccfad8f48/content/browser/gpu/gpu_process_host.h
[modify] https://crrev.com/248b114c9d3034964c7259222712dfcccfad8f48/content/common/gpu/gpu_host_messages.h
[modify] https://crrev.com/248b114c9d3034964c7259222712dfcccfad8f48/content/content_browser.gypi
[modify] https://crrev.com/248b114c9d3034964c7259222712dfcccfad8f48/content/gpu/gpu_child_thread.cc
[modify] https://crrev.com/248b114c9d3034964c7259222712dfcccfad8f48/content/gpu/gpu_child_thread.h
[modify] https://crrev.com/248b114c9d3034964c7259222712dfcccfad8f48/content/gpu/in_process_gpu_thread.cc
[modify] https://crrev.com/248b114c9d3034964c7259222712dfcccfad8f48/content/gpu/in_process_gpu_thread.h
[add] https://crrev.com/248b114c9d3034964c7259222712dfcccfad8f48/content/public/browser/gpu_utils.cc
[add] https://crrev.com/248b114c9d3034964c7259222712dfcccfad8f48/content/public/browser/gpu_utils.h
[modify] https://crrev.com/248b114c9d3034964c7259222712dfcccfad8f48/gpu/command_buffer/service/gpu_preferences.h
[modify] https://crrev.com/248b114c9d3034964c7259222712dfcccfad8f48/gpu/command_buffer/service/gpu_switches.cc
[modify] https://crrev.com/248b114c9d3034964c7259222712dfcccfad8f48/gpu/command_buffer/service/gpu_switches.h

Project Member

Comment 3 by bugdroid1@chromium.org, Mar 9 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/7a1173688cc0e8ff83260231c66fc3e118219c75

commit 7a1173688cc0e8ff83260231c66fc3e118219c75
Author: simonhatch <simonhatch@chromium.org>
Date: Wed Mar 09 00:28:26 2016

Revert of Get rid of gpu related switches by passing gpu::GpuPreferences via IPC. (patchset #8 id:140001 of https://codereview.chromium.org/1753383003/ )

Reason for revert:
Suspecting this is breaking a lot of perf bots.

BUG= 593045 

Original issue's description:
> Get rid of gpu related switches by passing gpu::GpuPreferences via IPC.
>
> BUG= 590825 
> CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel
>
> Committed: https://crrev.com/248b114c9d3034964c7259222712dfcccfad8f48
> Cr-Commit-Position: refs/heads/master@{#379651}

TBR=piman@chromium.org,boliu@chromium.org,palmer@chromium.org,penghuang@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG= 590825 

Review URL: https://codereview.chromium.org/1778693002

Cr-Commit-Position: refs/heads/master@{#379993}

[modify] https://crrev.com/7a1173688cc0e8ff83260231c66fc3e118219c75/android_webview/browser/deferred_gpu_command_service.cc
[modify] https://crrev.com/7a1173688cc0e8ff83260231c66fc3e118219c75/content/browser/android/synchronous_compositor_base.cc
[modify] https://crrev.com/7a1173688cc0e8ff83260231c66fc3e118219c75/content/browser/gpu/gpu_process_host.cc
[modify] https://crrev.com/7a1173688cc0e8ff83260231c66fc3e118219c75/content/browser/gpu/gpu_process_host.h
[modify] https://crrev.com/7a1173688cc0e8ff83260231c66fc3e118219c75/content/common/gpu/gpu_host_messages.h
[modify] https://crrev.com/7a1173688cc0e8ff83260231c66fc3e118219c75/content/content_browser.gypi
[modify] https://crrev.com/7a1173688cc0e8ff83260231c66fc3e118219c75/content/gpu/gpu_child_thread.cc
[modify] https://crrev.com/7a1173688cc0e8ff83260231c66fc3e118219c75/content/gpu/gpu_child_thread.h
[modify] https://crrev.com/7a1173688cc0e8ff83260231c66fc3e118219c75/content/gpu/in_process_gpu_thread.cc
[modify] https://crrev.com/7a1173688cc0e8ff83260231c66fc3e118219c75/content/gpu/in_process_gpu_thread.h
[delete] https://crrev.com/692a9e9ca5bb6d48b1d3c1e565be7205034e9a39/content/public/browser/gpu_utils.cc
[delete] https://crrev.com/692a9e9ca5bb6d48b1d3c1e565be7205034e9a39/content/public/browser/gpu_utils.h
[modify] https://crrev.com/7a1173688cc0e8ff83260231c66fc3e118219c75/gpu/command_buffer/service/gpu_preferences.h
[modify] https://crrev.com/7a1173688cc0e8ff83260231c66fc3e118219c75/gpu/command_buffer/service/gpu_switches.cc
[modify] https://crrev.com/7a1173688cc0e8ff83260231c66fc3e118219c75/gpu/command_buffer/service/gpu_switches.h

Cc: rjkroege@chromium.org sadrul@chromium.org vollick@chromium.org markdittmer@chromium.org piman@chromium.org vmi...@chromium.org siev...@chromium.org
Labels: mustash1 mus gpurefactor
Fady asked me to attach my grep results for reference.

content/common/gpu > ag-src '[#]include.*content' | grep 'switches'
image_transport_surface_android.cc:13:#include "content/public/common/content_switches.h"
image_transport_surface_win.cc:15:#include "content/public/common/content_switches.h"
client/webgraphicscontext3d_command_buffer_impl.cc:28:#include "content/public/common/content_switches.h"
media/gpu_video_decode_accelerator.cc:22:#include "content/public/common/content_switches.h"
media/vaapi_wrapper.cc:21:#include "content/public/common/content_switches.h"
media/gpu_video_encode_accelerator.cc:18:#include "content/public/common/content_switches.h"
media/android_video_encode_accelerator.cc:15:#include "content/public/common/content_switches.h"
media/v4l2_video_encode_accelerator.cc:21:#include "content/public/common/content_switches.h"
media/video_decode_accelerator_unittest.cc:52:#include "content/public/common/content_switches.h"
media/vt_video_decode_accelerator_mac.cc:23:#include "content/public/common/content_switches.h"
media/android_video_decode_accelerator.cc:22:#include "content/public/common/content_switches.h"
media/dxva_video_decode_accelerator_win.cc:37:#include "content/public/common/content_switches.h"
Project Member

Comment 6 by bugdroid1@chromium.org, Mar 10 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/2084d99587adfa5eeb4025d5a12c8856fe8d2f3a

commit 2084d99587adfa5eeb4025d5a12c8856fe8d2f3a
Author: penghuang <penghuang@chromium.org>
Date: Thu Mar 10 03:32:57 2016

Get rid of gpu related switches by passing gpu::GpuPreferences via IPC.

BUG= 590825 
CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel

Committed: https://crrev.com/248b114c9d3034964c7259222712dfcccfad8f48
Cr-Commit-Position: refs/heads/master@{#379651}

Review URL: https://codereview.chromium.org/1753383003

Cr-Commit-Position: refs/heads/master@{#380320}

[modify] https://crrev.com/2084d99587adfa5eeb4025d5a12c8856fe8d2f3a/android_webview/browser/deferred_gpu_command_service.cc
[modify] https://crrev.com/2084d99587adfa5eeb4025d5a12c8856fe8d2f3a/content/browser/android/synchronous_compositor_base.cc
[modify] https://crrev.com/2084d99587adfa5eeb4025d5a12c8856fe8d2f3a/content/browser/gpu/gpu_process_host.cc
[modify] https://crrev.com/2084d99587adfa5eeb4025d5a12c8856fe8d2f3a/content/browser/gpu/gpu_process_host.h
[modify] https://crrev.com/2084d99587adfa5eeb4025d5a12c8856fe8d2f3a/content/common/gpu/gpu_host_messages.h
[modify] https://crrev.com/2084d99587adfa5eeb4025d5a12c8856fe8d2f3a/content/content_browser.gypi
[modify] https://crrev.com/2084d99587adfa5eeb4025d5a12c8856fe8d2f3a/content/gpu/gpu_child_thread.cc
[modify] https://crrev.com/2084d99587adfa5eeb4025d5a12c8856fe8d2f3a/content/gpu/gpu_child_thread.h
[modify] https://crrev.com/2084d99587adfa5eeb4025d5a12c8856fe8d2f3a/content/gpu/in_process_gpu_thread.cc
[modify] https://crrev.com/2084d99587adfa5eeb4025d5a12c8856fe8d2f3a/content/gpu/in_process_gpu_thread.h
[add] https://crrev.com/2084d99587adfa5eeb4025d5a12c8856fe8d2f3a/content/public/browser/gpu_utils.cc
[add] https://crrev.com/2084d99587adfa5eeb4025d5a12c8856fe8d2f3a/content/public/browser/gpu_utils.h
[modify] https://crrev.com/2084d99587adfa5eeb4025d5a12c8856fe8d2f3a/gpu/command_buffer/service/gpu_preferences.h
[modify] https://crrev.com/2084d99587adfa5eeb4025d5a12c8856fe8d2f3a/gpu/command_buffer/service/gpu_switches.cc
[modify] https://crrev.com/2084d99587adfa5eeb4025d5a12c8856fe8d2f3a/gpu/command_buffer/service/gpu_switches.h

Cc: penghuang@chromium.org dalecur...@chromium.org w...@chromium.org
 Issue 593829  has been merged into this issue.
Project Member

Comment 8 by bugdroid1@chromium.org, Mar 13 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/9288ad04012900a9656959f441f4519f4239dccb

commit 9288ad04012900a9656959f441f4519f4239dccb
Author: penghuang <penghuang@chromium.org>
Date: Sun Mar 13 21:50:20 2016

content gpu refactor: get rid of gpu related content switches.

BUG= 590825 
CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel
CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:winx64_10_perf_cq;tryserver.chromium.perf:mac_retina_perf_cq;tryserver.chromium.perf:linux_perf_cq

Review URL: https://codereview.chromium.org/1784193003

Cr-Commit-Position: refs/heads/master@{#380906}

[modify] https://crrev.com/9288ad04012900a9656959f441f4519f4239dccb/content/common/gpu/gpu_host_messages.h
[modify] https://crrev.com/9288ad04012900a9656959f441f4519f4239dccb/content/common/gpu/image_transport_surface_android.cc
[modify] https://crrev.com/9288ad04012900a9656959f441f4519f4239dccb/content/common/gpu/image_transport_surface_win.cc
[modify] https://crrev.com/9288ad04012900a9656959f441f4519f4239dccb/content/common/gpu/media/android_video_decode_accelerator.cc
[modify] https://crrev.com/9288ad04012900a9656959f441f4519f4239dccb/content/common/gpu/media/android_video_encode_accelerator.cc
[modify] https://crrev.com/9288ad04012900a9656959f441f4519f4239dccb/content/common/gpu/media/dxva_video_decode_accelerator_win.cc
[modify] https://crrev.com/9288ad04012900a9656959f441f4519f4239dccb/content/common/gpu/media/dxva_video_decode_accelerator_win.h
[modify] https://crrev.com/9288ad04012900a9656959f441f4519f4239dccb/content/common/gpu/media/gpu_video_decode_accelerator.cc
[modify] https://crrev.com/9288ad04012900a9656959f441f4519f4239dccb/content/common/gpu/media/gpu_video_decode_accelerator.h
[modify] https://crrev.com/9288ad04012900a9656959f441f4519f4239dccb/content/common/gpu/media/gpu_video_encode_accelerator.cc
[modify] https://crrev.com/9288ad04012900a9656959f441f4519f4239dccb/content/common/gpu/media/gpu_video_encode_accelerator.h
[modify] https://crrev.com/9288ad04012900a9656959f441f4519f4239dccb/content/common/gpu/media/vaapi_wrapper.cc
[modify] https://crrev.com/9288ad04012900a9656959f441f4519f4239dccb/content/common/gpu/media/video_decode_accelerator_unittest.cc
[modify] https://crrev.com/9288ad04012900a9656959f441f4519f4239dccb/content/common/gpu/media/vt_video_decode_accelerator_mac.cc
[modify] https://crrev.com/9288ad04012900a9656959f441f4519f4239dccb/content/gpu/gpu_child_thread.cc
[modify] https://crrev.com/9288ad04012900a9656959f441f4519f4239dccb/content/gpu/gpu_main.cc
[modify] https://crrev.com/9288ad04012900a9656959f441f4519f4239dccb/content/public/browser/gpu_utils.cc
[modify] https://crrev.com/9288ad04012900a9656959f441f4519f4239dccb/gpu/command_buffer/service/gpu_preferences.h

Project Member

Comment 9 by bugdroid1@chromium.org, Mar 13 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/f749fe6715f1f3c81bda3576fd60f260ac47282a

commit f749fe6715f1f3c81bda3576fd60f260ac47282a
Author: rockot <rockot@chromium.org>
Date: Sun Mar 13 23:40:53 2016

Revert of content gpu refactor: get rid of gpu related content switches. (patchset #8 id:100002 of https://codereview.chromium.org/1784193003/ )

Reason for revert:
Closed the tree: https://build.chromium.org/p/chromium/builders/Win/builds/41257/steps/compile/logs/stdio

Original issue's description:
> content gpu refactor: get rid of gpu related content switches.
>
> BUG= 590825 
> CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel
> CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:winx64_10_perf_cq;tryserver.chromium.perf:mac_retina_perf_cq;tryserver.chromium.perf:linux_perf_cq
>
> Committed: https://crrev.com/9288ad04012900a9656959f441f4519f4239dccb
> Cr-Commit-Position: refs/heads/master@{#380906}

TBR=dcheng@chromium.org,mcasas@chromium.org,piman@chromium.org,penghuang@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG= 590825 

Review URL: https://codereview.chromium.org/1796633002

Cr-Commit-Position: refs/heads/master@{#380908}

[modify] https://crrev.com/f749fe6715f1f3c81bda3576fd60f260ac47282a/content/common/gpu/gpu_host_messages.h
[modify] https://crrev.com/f749fe6715f1f3c81bda3576fd60f260ac47282a/content/common/gpu/image_transport_surface_android.cc
[modify] https://crrev.com/f749fe6715f1f3c81bda3576fd60f260ac47282a/content/common/gpu/image_transport_surface_win.cc
[modify] https://crrev.com/f749fe6715f1f3c81bda3576fd60f260ac47282a/content/common/gpu/media/android_video_decode_accelerator.cc
[modify] https://crrev.com/f749fe6715f1f3c81bda3576fd60f260ac47282a/content/common/gpu/media/android_video_encode_accelerator.cc
[modify] https://crrev.com/f749fe6715f1f3c81bda3576fd60f260ac47282a/content/common/gpu/media/dxva_video_decode_accelerator_win.cc
[modify] https://crrev.com/f749fe6715f1f3c81bda3576fd60f260ac47282a/content/common/gpu/media/dxva_video_decode_accelerator_win.h
[modify] https://crrev.com/f749fe6715f1f3c81bda3576fd60f260ac47282a/content/common/gpu/media/gpu_video_decode_accelerator.cc
[modify] https://crrev.com/f749fe6715f1f3c81bda3576fd60f260ac47282a/content/common/gpu/media/gpu_video_decode_accelerator.h
[modify] https://crrev.com/f749fe6715f1f3c81bda3576fd60f260ac47282a/content/common/gpu/media/gpu_video_encode_accelerator.cc
[modify] https://crrev.com/f749fe6715f1f3c81bda3576fd60f260ac47282a/content/common/gpu/media/gpu_video_encode_accelerator.h
[modify] https://crrev.com/f749fe6715f1f3c81bda3576fd60f260ac47282a/content/common/gpu/media/vaapi_wrapper.cc
[modify] https://crrev.com/f749fe6715f1f3c81bda3576fd60f260ac47282a/content/common/gpu/media/video_decode_accelerator_unittest.cc
[modify] https://crrev.com/f749fe6715f1f3c81bda3576fd60f260ac47282a/content/common/gpu/media/vt_video_decode_accelerator_mac.cc
[modify] https://crrev.com/f749fe6715f1f3c81bda3576fd60f260ac47282a/content/gpu/gpu_child_thread.cc
[modify] https://crrev.com/f749fe6715f1f3c81bda3576fd60f260ac47282a/content/gpu/gpu_main.cc
[modify] https://crrev.com/f749fe6715f1f3c81bda3576fd60f260ac47282a/content/public/browser/gpu_utils.cc
[modify] https://crrev.com/f749fe6715f1f3c81bda3576fd60f260ac47282a/gpu/command_buffer/service/gpu_preferences.h

Project Member

Comment 10 by bugdroid1@chromium.org, Mar 14 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/34fb6bacfdd34c536f75504073ce1211898f413b

commit 34fb6bacfdd34c536f75504073ce1211898f413b
Author: penghuang <penghuang@chromium.org>
Date: Mon Mar 14 16:41:44 2016

content gpu refactor: get rid of gpu related content switches.

BUG= 590825 
CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel
CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:winx64_10_perf_cq;tryserver.chromium.perf:mac_retina_perf_cq;tryserver.chromium.perf:linux_perf_cq

Committed: https://crrev.com/9288ad04012900a9656959f441f4519f4239dccb
Cr-Commit-Position: refs/heads/master@{#380906}

Review URL: https://codereview.chromium.org/1784193003

Cr-Commit-Position: refs/heads/master@{#380988}

[modify] https://crrev.com/34fb6bacfdd34c536f75504073ce1211898f413b/content/common/gpu/gpu_host_messages.h
[modify] https://crrev.com/34fb6bacfdd34c536f75504073ce1211898f413b/content/common/gpu/image_transport_surface_android.cc
[modify] https://crrev.com/34fb6bacfdd34c536f75504073ce1211898f413b/content/common/gpu/image_transport_surface_win.cc
[modify] https://crrev.com/34fb6bacfdd34c536f75504073ce1211898f413b/content/common/gpu/media/android_video_decode_accelerator.cc
[modify] https://crrev.com/34fb6bacfdd34c536f75504073ce1211898f413b/content/common/gpu/media/android_video_encode_accelerator.cc
[modify] https://crrev.com/34fb6bacfdd34c536f75504073ce1211898f413b/content/common/gpu/media/dxva_video_decode_accelerator_win.cc
[modify] https://crrev.com/34fb6bacfdd34c536f75504073ce1211898f413b/content/common/gpu/media/dxva_video_decode_accelerator_win.h
[modify] https://crrev.com/34fb6bacfdd34c536f75504073ce1211898f413b/content/common/gpu/media/gpu_video_decode_accelerator.cc
[modify] https://crrev.com/34fb6bacfdd34c536f75504073ce1211898f413b/content/common/gpu/media/gpu_video_decode_accelerator.h
[modify] https://crrev.com/34fb6bacfdd34c536f75504073ce1211898f413b/content/common/gpu/media/gpu_video_encode_accelerator.cc
[modify] https://crrev.com/34fb6bacfdd34c536f75504073ce1211898f413b/content/common/gpu/media/gpu_video_encode_accelerator.h
[modify] https://crrev.com/34fb6bacfdd34c536f75504073ce1211898f413b/content/common/gpu/media/vaapi_wrapper.cc
[modify] https://crrev.com/34fb6bacfdd34c536f75504073ce1211898f413b/content/common/gpu/media/video_decode_accelerator_unittest.cc
[modify] https://crrev.com/34fb6bacfdd34c536f75504073ce1211898f413b/content/common/gpu/media/vt_video_decode_accelerator_mac.cc
[modify] https://crrev.com/34fb6bacfdd34c536f75504073ce1211898f413b/content/gpu/gpu_child_thread.cc
[modify] https://crrev.com/34fb6bacfdd34c536f75504073ce1211898f413b/content/gpu/gpu_main.cc
[modify] https://crrev.com/34fb6bacfdd34c536f75504073ce1211898f413b/content/public/browser/gpu_utils.cc
[modify] https://crrev.com/34fb6bacfdd34c536f75504073ce1211898f413b/gpu/command_buffer/service/gpu_preferences.h

Status: Fixed (was: Started)
Status: Assigned (was: Fixed)
Another content switch: https://code.google.com/p/chromium/codesearch#chromium/src/content/common/gpu/media/v4l2_video_encode_accelerator.cc&q=file:content/common/gpu%20switches&sq=package:chromium&type=cs&l=22&ssfr=1

content/common/gpu/media/v4l2_video_encode_accelerator.cc

refers to

#include "content/public/common/content_switches.h"
Project Member

Comment 14 by bugdroid1@chromium.org, Mar 21 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/a246dbbc4f8b168271c62e527e5c20c8f9c8e387

commit a246dbbc4f8b168271c62e527e5c20c8f9c8e387
Author: penghuang <penghuang@chromium.org>
Date: Mon Mar 21 17:26:54 2016

Remove a content_switches.h  from v4l2_video_encode_accelerator.cc

BUG= 590825 

Review URL: https://codereview.chromium.org/1820733003

Cr-Commit-Position: refs/heads/master@{#382316}

[modify] https://crrev.com/a246dbbc4f8b168271c62e527e5c20c8f9c8e387/content/common/gpu/media/v4l2_video_encode_accelerator.cc

Status: Fixed (was: Assigned)
Marking as FIXED.
I found a reference to this issue in the code at [1]. Without understanding all the details, it appears to me that there are still command-line switches being propagated to the GPU process. If I understand the changes associated with this issue correctly, the GPU process is now supposed to pull all configuration flags through GpuPreferences via IPC instead of being invoked with command-line switches. With the issue being marked as fixed, shouldn't the code that propagates command-line switches have been removed? If not, why is it still needed?

The reason I ask is that I may need to make a modification to the switches, and I need to understand if propagating them to the GPU process is still needed.

[1] https://cs.chromium.org/chromium/src/content/browser/gpu/gpu_process_host.cc?sq=package:chromium&dr=C&rcl=1467126994&l=1008

Comment 17 by piman@chromium.org, Jun 28 2016

There are some switches that are needed before the GpuMsg_Initialize message is sent (which sends GpuPreferences) - e.g. whether to turn on the mailbox, logging related, etc.
But generally new switches should go to GpuPreferences.

Sign in to add a comment