Skip to content

Commit

Permalink
cefsimple: Enable Chrome runtime by default (see #3685)
Browse files Browse the repository at this point in the history
Run with `--disable-chrome-runtime` to use the Alloy runtime.
  • Loading branch information
magreenblatt committed May 9, 2024
1 parent 331f668 commit 44f7b57
Show file tree
Hide file tree
Showing 4 changed files with 30 additions and 26 deletions.
12 changes: 5 additions & 7 deletions tests/cefsimple/cefsimple_linux.cc
Original file line number Diff line number Diff line change
Expand Up @@ -58,13 +58,11 @@ int main(int argc, char* argv[]) {
// Specify CEF global settings here.
CefSettings settings;

// Use the CEF Chrome runtime if "--enable-chrome-runtime" is specified via
// the command-line. Otherwise, use the CEF Alloy runtime. For more
// information about CEF runtimes see
// https://bitbucket.org/chromiumembedded/cef/wiki/Architecture.md#markdown-header-cef3
if (command_line->HasSwitch("enable-chrome-runtime")) {
settings.chrome_runtime = true;
}
// Use the CEF Chrome bootstrap unless "--disable-chrome-runtime" is specified
// via the command-line. Otherwise, use the CEF Alloy bootstrap. The Alloy
// bootstrap is deprecated and will be removed in ~M127. See
// https://github.com/chromiumembedded/cef/issues/3685
settings.chrome_runtime = !command_line->HasSwitch("disable-chrome-runtime");

// When generating projects with CMake the CEF_USE_SANDBOX value will be defined
// automatically. Pass -DUSE_SANDBOX=OFF to the CMake command-line to disable
Expand Down
13 changes: 6 additions & 7 deletions tests/cefsimple/cefsimple_mac.mm
Original file line number Diff line number Diff line change
Expand Up @@ -150,13 +150,12 @@ int main(int argc, char* argv[]) {
// Specify CEF global settings here.
CefSettings settings;

// Use the CEF Chrome runtime if "--enable-chrome-runtime" is specified via
// the command-line. Otherwise, use the CEF Alloy runtime. For more
// information about CEF runtimes see
// https://bitbucket.org/chromiumembedded/cef/wiki/Architecture.md#markdown-header-cef3
if (command_line->HasSwitch("enable-chrome-runtime")) {
settings.chrome_runtime = true;
}
// Use the CEF Chrome bootstrap unless "--disable-chrome-runtime" is
// specified via the command-line. Otherwise, use the CEF Alloy bootstrap.
// The Alloy bootstrap is deprecated and will be removed in ~M127. See
// https://github.com/chromiumembedded/cef/issues/3685
settings.chrome_runtime =
!command_line->HasSwitch("disable-chrome-runtime");

// When generating projects with CMake the CEF_USE_SANDBOX value will be
// defined automatically. Pass -DUSE_SANDBOX=OFF to the CMake command-line
Expand Down
12 changes: 5 additions & 7 deletions tests/cefsimple/cefsimple_win.cc
Original file line number Diff line number Diff line change
Expand Up @@ -74,13 +74,11 @@ int APIENTRY wWinMain(HINSTANCE hInstance,
// Specify CEF global settings here.
CefSettings settings;

// Use the CEF Chrome runtime if "--enable-chrome-runtime" is specified via
// the command-line. Otherwise, use the CEF Alloy runtime. For more
// information about CEF runtimes see
// https://bitbucket.org/chromiumembedded/cef/wiki/Architecture.md#markdown-header-cef3
if (command_line->HasSwitch("enable-chrome-runtime")) {
settings.chrome_runtime = true;
}
// Use the CEF Chrome bootstrap unless "--disable-chrome-runtime" is specified
// via the command-line. Otherwise, use the CEF Alloy bootstrap. The Alloy
// bootstrap is deprecated and will be removed in ~M127. See
// https://github.com/chromiumembedded/cef/issues/3685
settings.chrome_runtime = !command_line->HasSwitch("disable-chrome-runtime");

#if !defined(CEF_USE_SANDBOX)
settings.no_sandbox = true;
Expand Down
19 changes: 14 additions & 5 deletions tests/cefsimple/simple_app.cc
Original file line number Diff line number Diff line change
Expand Up @@ -113,11 +113,14 @@ void SimpleApp::OnContextInitialized() {
CefRefPtr<CefCommandLine> command_line =
CefCommandLine::GetGlobalCommandLine();

const bool enable_chrome_runtime =
!command_line->HasSwitch("disable-chrome-runtime");

// Check if Alloy style will be used. Alloy style is always used with the
// Alloy runtime bootstrap and optional with the Chrome runtime bootstrap.
bool use_alloy_style = true;
cef_runtime_style_t runtime_style = CEF_RUNTIME_STYLE_DEFAULT;
if (command_line->HasSwitch("enable-chrome-runtime")) {
if (enable_chrome_runtime) {
use_alloy_style = command_line->HasSwitch("use-alloy-style");
if (use_alloy_style) {
runtime_style = CEF_RUNTIME_STYLE_ALLOY;
Expand All @@ -139,10 +142,16 @@ void SimpleApp::OnContextInitialized() {
url = "http://www.google.com";
}

// Create the browser using the Views framework if "--use-views" is specified
// via the command-line. Otherwise, create the browser using the native
// platform framework.
if (command_line->HasSwitch("use-views")) {
// Views is enabled by default with the Chrome bootstrap (add `--use-native`
// to disable). Views is disabled by default with the Alloy bootstrap (add
// `--use-views` to enable).
const bool use_views =
(enable_chrome_runtime && !command_line->HasSwitch("use-native")) ||
(!enable_chrome_runtime && command_line->HasSwitch("use-views"));

// If using Views create the browser using the Views framework, otherwise
// create the browser using the native platform framework.
if (use_views) {
// Create the BrowserView.
CefRefPtr<CefBrowserView> browser_view = CefBrowserView::CreateBrowserView(
handler, url, browser_settings, nullptr, nullptr,
Expand Down

0 comments on commit 44f7b57

Please sign in to comment.