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

Issue 783540 link

Starred by 2 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Feature

Blocked on: View detail
issue 630357
issue 788026
issue 845077
issue 775848
issue 777921
issue 779717
issue 779732
issue 788010
issue 788014
issue 788015
issue 788019
issue 788021
issue 788022
issue 788023
issue 788024
issue 788025
issue 788027
issue 795090

Blocking:
issue 832892
issue 839559


Show other hotlists

Hotlists containing this issue:
Fixing-touch


Sign in to add a comment

Eng: Enable Harmony Dialogs on ChromeOS

Project Member Reported by tapted@chromium.org, Nov 10 2017

Issue description

Tracking bug for engineering work for Issue 839559 "Enable harmony on ChromeOS".

ChromeOS equivalent of  Issue 775848  "Eng: Enable Harmony Dialogs on Windows, Linux and Mac."

See also Issue 775847 - launch bug - "Enable Harmony Dialogs on Windows, Linux and Mac."

Speculatively targeting M-66.
 

Comment 1 by tapted@chromium.org, Nov 10 2017

Blockedon: 777921

Comment 2 by tetsui@chromium.org, Nov 13 2017

Owner: tetsui@chromium.org

Comment 3 by tapted@chromium.org, Nov 23 2017

Blockedon: 630357
Blockedon: 779717
Blockedon: 795090

Comment 7 by tetsui@chromium.org, Jan 18 2018

Labels: -M-66 M-67
Owner: glevin@chromium.org

Comment 9 by tapted@chromium.org, May 18 2018

Description: Show this description
Blockedon: 845077
Project Member

Comment 11 by bugdroid1@chromium.org, Jun 19 2018

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

commit db0e213a037aabba4e00d21c8e180af33e7a74ca
Author: glevin <glevin@chromium.org>
Date: Tue Jun 19 19:49:42 2018

SecondaryUiMd on by default for CrOS

Enables Harmony dialog styling by default on Chrome OS dialogs.  Also
see launch bug crbug.com/839559.

Bug: 783540
Test: Summon any dialog (blockers of 783540) that has been Harmonized.
Verify that it has the Harmonious styling.

Change-Id: I2c9340d33d0d404edec2b110fe4dd9d6727bc645
Reviewed-on: https://chromium-review.googlesource.com/1072727
Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Elly Fong-Jones <ellyjones@chromium.org>
Commit-Queue: Greg Levin <glevin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#568576}
[modify] https://crrev.com/db0e213a037aabba4e00d21c8e180af33e7a74ca/ui/base/ui_base_features.cc

Labels: -M-67 M-68
The merge request is for the above CL turning the SecondaryUiMd flag on by default.
Labels: Merge-Request-68
Project Member

Comment 14 by sheriffbot@chromium.org, Jun 26 2018

Labels: -Merge-Request-68 Merge-Reject-68 Hotlist-Merge-Reject
The bug is marked as P3 or Feature. It should not be merged as M68 is in beta. 
Please contact the approriate milestone owner if you have questions.
Owners: cmasso@(Android), kariahda@(iOS), bhthompson@(ChromeOS), abdulsyed@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Hotlist-Merge-Reject -Merge-Reject-68 Merge-Approved-68
Project Member

Comment 16 by bugdroid1@chromium.org, Jun 27 2018

Labels: -merge-approved-68 merge-merged-3440
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b66cab98e9a8bf2501721a22bbb7ad2f68e47651

commit b66cab98e9a8bf2501721a22bbb7ad2f68e47651
Author: glevin <glevin@chromium.org>
Date: Wed Jun 27 19:47:18 2018

SecondaryUiMd on by default for CrOS

Enables Harmony dialog styling by default on Chrome OS dialogs.  Also
see launch bug crbug.com/839559.

Bug: 783540
Test: Summon any dialog (blockers of 783540) that has been Harmonized.
Verify that it has the Harmonious styling.

Change-Id: I2c9340d33d0d404edec2b110fe4dd9d6727bc645
Reviewed-on: https://chromium-review.googlesource.com/1072727
Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Elly Fong-Jones <ellyjones@chromium.org>
Commit-Queue: Greg Levin <glevin@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#568576}(cherry picked from commit db0e213a037aabba4e00d21c8e180af33e7a74ca)
Reviewed-on: https://chromium-review.googlesource.com/1117519
Reviewed-by: Greg Levin <glevin@chromium.org>
Cr-Commit-Position: refs/branch-heads/3440@{#558}
Cr-Branched-From: 010ddcfda246975d194964ccf20038ebbdec6084-refs/heads/master@{#561733}
[modify] https://crrev.com/b66cab98e9a8bf2501721a22bbb7ad2f68e47651/ui/base/ui_base_features.cc

Cc: bsep@chromium.org bhthompson@chromium.org
The merge of https://chromium-review.googlesource.com/1072727 also required merging fixes of two tests which this change breaks:
  https://chromium-review.googlesource.com/1103586
  https://chromium-review.googlesource.com/1105457
The additional merging of these two was approved via email by bhthompson@ and bsep@.
Status: Assigned (was: Available)
Blocking: 832892

Sign in to add a comment