New issue
Advanced search Search tips

Issue 735519 link

Starred by 1 user
Status: WontFix
Owner:
rouslan@chromium.org
Closed: Jun 2017
Cc:
rouslan@chromium.org
dpranke@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 3
Type: Bug



Sign in to add a comment

Strings are not available in components_browsertests

Project Member Reported by rouslan@chromium.org, Jun 21 2017 
Although components_browsertests depends on //components/strings, using any of those strings results in a DCHECK:

[87598:87598:0621/123659.882339:1743264151380:FATAL:resource_bundle.cc(559)] Check failed: false. unable to find resource: 15608

Code:

l10n_util::GetSTringUTF16(IDS_AUTILITY_PROCESS_PAYMENT_MANIFEST_PARSER_NAME);

Comment 1 by dpranke@chromium.org, Jun 21 2017

Cc: dpranke@chromium.org
Labels: -Build-Tools-GN
Owner: rouslan@chromium.org
This sounds like possibly a problem with the build files, but not a problem with the GN binary itself.

Is this error causing tests to fail? Can you link to a log, or give me a way to reproduce the problem?

Comment 2 by rouslan@chromium.org, Jun 21 2017 

This is not causing existing tests to fail. I was working on http://crrev.com/c/541520/ and found out that l10n_utils::GetStringUTF16() in components/payments/content/payment_manifest_parser_host.cc causes the DCHECK in components_browsertests. I found this strange, because components_browsertests depends on //components/strings, which contains IDS_AUTILITY_PROCESS_PAYMENT_MANIFEST_PARSER_NAME.

I've since moved my browser test from components_browsertests to browser_tests for a different reason, so I'm no longer affected. In order to reproduce the issues, put the l10n_util::GetStringUTF16(IDS_AUTILITY_PROCESS_PAYMENT_MANIFEST_PARSER_NAME) line in any of the sources used by in components_browsertests.

Given that this no longer blocks anyone, I assume this bug can be shelved. Agreed?

Comment 3 by dpranke@chromium.org, Jun 21 2017 

fine by me.

Comment 4 by rouslan@chromium.org, Jun 21 2017

Status: WontFix (was: Untriaged)
►

Sign in to add a comment