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

Issue 684654 link

Starred by 3 users

Issue metadata

Status: Verified
Owner:
Closed: Feb 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug

Blocking:
issue 684848



Sign in to add a comment

$i18n{greetingHeader} appears in the OOBE flow

Project Member Reported by hennessywill@chromium.org, Jan 24 2017

Issue description

Chrome Version: 57.0.2984.4
OS: Chrome OS

What steps will reproduce the problem?
(1) begin OOBE flow

I see $i18n{greetingHeader} render on my screen in the OOBE flow.

Sounds like this is a bug in the new template system from 677338.

See feedback report and screenshot here:  https://feedback.corp.google.com/#/Report/51969988157

I am running 9199.1.0 branch on our new Apollo Lake devices.


 
attaching image and logs below
51969988157-system_logs.zip
299 KB Download
ed1d530c3894f49cfcf43b01a30c08c74f4131a579862139f689edaa671d4bd2.jpg
84.0 KB View Download
Status: Fixed (was: Assigned)
Reverting the change that I believe caused this.
(https://codereview.chromium.org/2653123003/)

Comment 3 by bleung@chromium.org, Jan 30 2017

Labels: Blocks-FSI M-57 OS-Chrome
Status: Assigned (was: Fixed)
This still happens on the M-57 branch, as of 9202.9. Can you help please pick this back for 57?
Labels: Merge-Request-57

Comment 5 by bleung@chromium.org, Jan 30 2017

Cc: bleung@chromium.org keta...@chromium.org
Labels: Merge-Requested
+ketakid.
Labels: -Merge-Request-57 Merge-Approved-57
Approving merge to M57 Chrome OS.
Project Member

Comment 7 by sheriffbot@chromium.org, Feb 6 2017

This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Merge-Requested -Merge-Approved-57
Status: Fixed (was: Assigned)
I used drover to merge the patch. I rarely do this kind of patch (this might be the second time). If it's not correctly patched, please let me know so that I can learn and correct it. (If it doesn't look fixed, then I would suspect that it's not correctly patched since the fix should be solid).
Cc: st...@chromium.org alemate@chromium.org
 Issue 689275  has been merged into this issue.
Labels: ReleaseBlock-Beta
Status: Assigned (was: Fixed)
Your CL does not appear to be in branch. This is still broken.
Cc: w...@chromium.org
alemate@ can you please help dschuyler@ drover this change asap?
Labels: -ReleaseBlock-Beta ReleaseBlock-Stable
alemate@/dschuyler@ - please merge the fix soonest possible. moving to stable.
Labels: Merge-Request-57
Status: Fixed (was: Assigned)
Owner: alemate@chromium.org
Requesting merge.
Requesting merge for https://codereview.chromium.org/2653123003 .
Cc: dschuyler@chromium.org
Labels: -Merge-Request-57 merge-merged-2987
Dave landed fix in https://codereview.chromium.org/2681863006 .
Closing this.
Blocking: 684848
Status: Verified (was: Fixed)
9202.43.0 / 57.0.2987.85 beta-channel
Cc: r...@chromium.org
Cc: -st...@chromium.org
Components: UI>Shell>OOBE

Sign in to add a comment