New issue
Advanced search Search tips

Issue 865913 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jul 27
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

Manually passing --js-flags=--no-opt to chrome or content_shell has no effect

Project Member Reported by neis@chromium.org, Jul 20

Issue description

This is due to V8Initializer::Initialize in gin/v8_initializer.cc explicitly passing --opt to V8 (unless disabled through some other means?).
 
Labels: -Pri-3 Pri-1
Perhaps the issue is merely that the --js-flags get processed before the above function is called.
I'll have a look at this, but not sure I can do it today (and out next week). As a workaround, you can use the chrome flag "--disable-features=V8OptimizeJavascript" to disable optimization in gin.
No problem, thanks!
re #2 - that's probable, however the alternative (processing --js-flags after V8 has been initialized) has other issues, e.g., the flags aren't correctly set during initialization and so might have not effect.
Project Member

Comment 6 by bugdroid1@chromium.org, Jul 27

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

commit 8f0d3d4bfeaee0f35de94171d15e1deef9f912a9
Author: Ross McIlroy <rmcilroy@chromium.org>
Date: Fri Jul 27 01:51:47 2018

[Gin] Avoid explicitly setting --opt if features::kV8OptimizeJavascript is enabled.

This is the default, and doing so prevents passing --no-opt via other means.

BUG= chromium:865913 

Change-Id: I017d940809020d4015e245e9cb4ac3b166ad67b8
Reviewed-on: https://chromium-review.googlesource.com/1148730
Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Commit-Queue: Ross McIlroy <rmcilroy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#578517}
[modify] https://crrev.com/8f0d3d4bfeaee0f35de94171d15e1deef9f912a9/gin/v8_initializer.cc

Status: Fixed (was: Assigned)

Sign in to add a comment