New issue
Advanced search Search tips
Starred by 1 user

Issue metadata

Status: Fixed
Closed: Jan 23
EstimatedDays: ----
NextAction: ----
Pri: 3
Type: Bug

Sign in to add a comment

Issue 912502: [ios] DCHECK hit in constructor of ChromeIOSTranslateClient when APP startup.

Reported by, Dec 6 Project Member

Issue description

Chrome Version: 73.0.3633.0

DCHECK code:

What steps will reproduce the problem?
(1)Run "gclient sync" and rebuild the APP;
(2)Start the APP on simulator;
(3)DCHECK is hit;
(4)Restart the APP and the DCHECK won't be hit anymore.

Comment 1 by, Dec 6

Status: Assigned (was: Untriaged)
Anthony could you PTAL?

Comment 2 by, Dec 6

Status: Started (was: Assigned)
This is caused by my change. I will take a look.

Comment 3 by, Dec 7

mahmadi: Does this affect M72?

Comment 4 by, Dec 10

Comment 5 by, Dec 11

Project Member
The following revision refers to this bug:

commit 2545cc4292cd8343302b111104201f60a1701a63
Author: Moe Ahmadi <>
Date: Tue Dec 11 12:31:31 2018

[iOS][Translate] Moves LanguageSelectionHnadler out of infobarcoordinator

Instead of passing the id<LanguageSelectionHandler> to the
ChromeIOSTranslateClient during construction which is prone to error due to
dependency on the order in which ChromeIOSTranslateClient and
the InfobarCoordinator are initialized, this CL modifies the
LanguageSelectionCoordinator to observe the WebStateList and set/unset the
id<LanguageSelectionHandler> on instances of ChromeIOSTranslateClient when

This fixes a DCHECK during ChromeIOSTranslateClient construction and helps
make the feature more self-contained as the InfobarCoordinator/Mediator no
longer needs to know about the languageSelectionHandler.


Bug:  912502 
Change-Id: Id3769f10a0ed638821afdd9578bd263700768299
Commit-Queue: Moe Ahmadi <>
Reviewed-by: edchin <>
Reviewed-by: Sergio Collazos <>
Reviewed-by: anthonyvd <>
Cr-Commit-Position: refs/heads/master@{#615500}

Comment 6 by, Jan 23

Status: Fixed (was: Started)

Sign in to add a comment