PK longpress diacritics: Add pref metric [chromium/src : main]

0 views
Skip to first unread message

jhtin (Gerrit)

unread,
Aug 8, 2022, 2:49:09 AM8/8/22
to asvitkine...@chromium.org, keithle...@chromium.org, nona+...@chromium.org, shuche...@chromium.org, tranbaod...@chromium.org, yhanad...@chromium.org, yusuke...@chromium.org, Essential Inputs Reviews, Tricium, Chromium Metrics Reviews, chromium...@chromium.org, Sadrul Chowdhury

Attention is currently required from: Essential Inputs Reviews.

View Change

    To view, visit change 3815109. To unsubscribe, or for help writing mail filters, visit settings.

    Gerrit-Project: chromium/src
    Gerrit-Branch: main
    Gerrit-Change-Id: I89c1188603ff34b4befa6abb00f2f4b700135bdd
    Gerrit-Change-Number: 3815109
    Gerrit-PatchSet: 1
    Gerrit-Owner: jhtin <jh...@chromium.org>
    Gerrit-Reviewer: Essential Inputs Reviews <e14s-...@google.com>
    Gerrit-CC: Chromium Metrics Reviews <chromium-met...@google.com>
    Gerrit-CC: Sadrul Chowdhury <sad...@chromium.org>
    Gerrit-Attention: Essential Inputs Reviews <e14s-...@google.com>
    Gerrit-Comment-Date: Mon, 08 Aug 2022 06:48:59 +0000
    Gerrit-HasComments: No
    Gerrit-Has-Labels: No
    Gerrit-MessageType: comment

    gwsq (Gerrit)

    unread,
    Aug 8, 2022, 2:52:47 AM8/8/22
    to jhtin, asvitkine...@chromium.org, keithle...@chromium.org, nona+...@chromium.org, shuche...@chromium.org, tranbaod...@chromium.org, yhanad...@chromium.org, yusuke...@chromium.org, Essential Inputs Reviews, Mehrab N, Keith Lee, Tricium, Chromium Metrics Reviews, chromium...@chromium.org, Sadrul Chowdhury

    Attention is currently required from: Keith Lee, Mehrab N.


    Reviewer source(s):
    keit...@chromium.org, meh...@chromium.org is from context

    View Change

      To view, visit change 3815109. To unsubscribe, or for help writing mail filters, visit settings.

      Gerrit-Project: chromium/src
      Gerrit-Branch: main
      Gerrit-Change-Id: I89c1188603ff34b4befa6abb00f2f4b700135bdd
      Gerrit-Change-Number: 3815109
      Gerrit-PatchSet: 1
      Gerrit-Owner: jhtin <jh...@chromium.org>
      Gerrit-Reviewer: Keith Lee <keit...@chromium.org>
      Gerrit-Reviewer: Mehrab N <meh...@chromium.org>
      Gerrit-CC: Chromium Metrics Reviews <chromium-met...@google.com>
      Gerrit-CC: Essential Inputs Reviews <e14s-...@google.com>
      Gerrit-CC: Sadrul Chowdhury <sad...@chromium.org>
      Gerrit-CC: gwsq
      Gerrit-Attention: Keith Lee <keit...@chromium.org>
      Gerrit-Attention: Mehrab N <meh...@chromium.org>
      Gerrit-Comment-Date: Mon, 08 Aug 2022 06:52:33 +0000

      jhtin (Gerrit)

      unread,
      Aug 8, 2022, 3:01:09 AM8/8/22
      to asvitkine...@chromium.org, keithle...@chromium.org, nona+...@chromium.org, shuche...@chromium.org, tranbaod...@chromium.org, yhanad...@chromium.org, yusuke...@chromium.org, Essential Inputs Reviews, Mehrab N, Keith Lee, Tricium, Chromium Metrics Reviews, chromium...@chromium.org, Sadrul Chowdhury

      Attention is currently required from: Keith Lee, Mehrab N.

      Patch set 2:Auto-Submit +1Commit-Queue +1

      View Change

        To view, visit change 3815109. To unsubscribe, or for help writing mail filters, visit settings.

        Gerrit-Project: chromium/src
        Gerrit-Branch: main
        Gerrit-Change-Id: I89c1188603ff34b4befa6abb00f2f4b700135bdd
        Gerrit-Change-Number: 3815109
        Gerrit-PatchSet: 2
        Gerrit-Owner: jhtin <jh...@chromium.org>
        Gerrit-Reviewer: Keith Lee <keit...@chromium.org>
        Gerrit-Reviewer: Mehrab N <meh...@chromium.org>
        Gerrit-Reviewer: jhtin <jh...@chromium.org>
        Gerrit-CC: Chromium Metrics Reviews <chromium-met...@google.com>
        Gerrit-CC: Essential Inputs Reviews <e14s-...@google.com>
        Gerrit-CC: Sadrul Chowdhury <sad...@chromium.org>
        Gerrit-CC: gwsq
        Gerrit-Attention: Keith Lee <keit...@chromium.org>
        Gerrit-Attention: Mehrab N <meh...@chromium.org>
        Gerrit-Comment-Date: Mon, 08 Aug 2022 07:00:59 +0000
        Gerrit-HasComments: No
        Gerrit-Has-Labels: Yes
        Gerrit-MessageType: comment

        jhtin (Gerrit)

        unread,
        Aug 8, 2022, 3:07:40 AM8/8/22
        to asvitkine...@chromium.org, keithle...@chromium.org, nona+...@chromium.org, shuche...@chromium.org, tranbaod...@chromium.org, yhanad...@chromium.org, yusuke...@chromium.org, Chromium Metrics Reviews, Chromium LUCI CQ, Essential Inputs Reviews, Mehrab N, Keith Lee, Tricium, chromium...@chromium.org, Sadrul Chowdhury

        Attention is currently required from: Keith Lee, Mehrab N.

        Patch set 3:Auto-Submit +1

        View Change

          To view, visit change 3815109. To unsubscribe, or for help writing mail filters, visit settings.

          Gerrit-Project: chromium/src
          Gerrit-Branch: main
          Gerrit-Change-Id: I89c1188603ff34b4befa6abb00f2f4b700135bdd
          Gerrit-Change-Number: 3815109
          Gerrit-PatchSet: 3
          Gerrit-Owner: jhtin <jh...@chromium.org>
          Gerrit-Reviewer: Chromium Metrics Reviews <chromium-met...@google.com>
          Gerrit-Reviewer: Keith Lee <keit...@chromium.org>
          Gerrit-Reviewer: Mehrab N <meh...@chromium.org>
          Gerrit-Reviewer: jhtin <jh...@chromium.org>
          Gerrit-CC: Essential Inputs Reviews <e14s-...@google.com>
          Gerrit-CC: Sadrul Chowdhury <sad...@chromium.org>
          Gerrit-CC: gwsq
          Gerrit-Attention: Keith Lee <keit...@chromium.org>
          Gerrit-Attention: Mehrab N <meh...@chromium.org>
          Gerrit-Comment-Date: Mon, 08 Aug 2022 07:07:30 +0000

          gwsq (Gerrit)

          unread,
          Aug 8, 2022, 3:12:43 AM8/8/22
          to jhtin, asvitkine...@chromium.org, keithle...@chromium.org, nona+...@chromium.org, shuche...@chromium.org, tranbaod...@chromium.org, yhanad...@chromium.org, yusuke...@chromium.org, Chromium Metrics Reviews, Caitlin Fischer, Chromium LUCI CQ, Essential Inputs Reviews, Mehrab N, Keith Lee, Tricium, chromium...@chromium.org, Sadrul Chowdhury

          Attention is currently required from: Caitlin Fischer, Keith Lee, Mehrab N.

          From analysis/uma/chrome-metrics.gwsq:
          Histograms should by default be reviewed by the owners of the subdirectories. The chromium-met...@google.com gwsq should be used when there are no individual owners, or for escalation to the Metrics team.
          If you are interested in becoming a metrics reviewer, please review the instructions at https://chromium.googlesource.com/chromium/src.git/+/HEAD/tools/metrics/histograms/README.md#Becoming-a-Metrics-Reviewer

          Reviewer source(s):
          caitlin...@google.com is from context

          View Change

            To view, visit change 3815109. To unsubscribe, or for help writing mail filters, visit settings.

            Gerrit-Project: chromium/src
            Gerrit-Branch: main
            Gerrit-Change-Id: I89c1188603ff34b4befa6abb00f2f4b700135bdd
            Gerrit-Change-Number: 3815109
            Gerrit-PatchSet: 3
            Gerrit-Owner: jhtin <jh...@chromium.org>
            Gerrit-Reviewer: Caitlin Fischer <caitlin...@google.com>
            Gerrit-Reviewer: Keith Lee <keit...@chromium.org>
            Gerrit-Reviewer: Mehrab N <meh...@chromium.org>
            Gerrit-Reviewer: jhtin <jh...@chromium.org>
            Gerrit-CC: Chromium Metrics Reviews <chromium-met...@google.com>
            Gerrit-CC: Essential Inputs Reviews <e14s-...@google.com>
            Gerrit-CC: Sadrul Chowdhury <sad...@chromium.org>
            Gerrit-CC: gwsq
            Gerrit-Attention: Keith Lee <keit...@chromium.org>
            Gerrit-Attention: Mehrab N <meh...@chromium.org>
            Gerrit-Attention: Caitlin Fischer <caitlin...@google.com>
            Gerrit-Comment-Date: Mon, 08 Aug 2022 07:12:34 +0000

            Caitlin Fischer (Gerrit)

            unread,
            Aug 8, 2022, 7:14:39 AM8/8/22
            to jhtin, asvitkine...@chromium.org, keithle...@chromium.org, nona+...@chromium.org, shuche...@chromium.org, tranbaod...@chromium.org, yhanad...@chromium.org, yusuke...@chromium.org, Chromium Metrics Reviews, Chromium LUCI CQ, Essential Inputs Reviews, Mehrab N, Keith Lee, Tricium, chromium...@chromium.org, Sadrul Chowdhury

            Attention is currently required from: Keith Lee, Mehrab N, jhtin.

            View Change

            1 comment:

            To view, visit change 3815109. To unsubscribe, or for help writing mail filters, visit settings.

            Gerrit-Project: chromium/src
            Gerrit-Branch: main
            Gerrit-Change-Id: I89c1188603ff34b4befa6abb00f2f4b700135bdd
            Gerrit-Change-Number: 3815109
            Gerrit-PatchSet: 4
            Gerrit-Owner: jhtin <jh...@chromium.org>
            Gerrit-Reviewer: Keith Lee <keit...@chromium.org>
            Gerrit-Reviewer: Mehrab N <meh...@chromium.org>
            Gerrit-Reviewer: jhtin <jh...@chromium.org>
            Gerrit-CC: Chromium Metrics Reviews <chromium-met...@google.com>
            Gerrit-CC: Essential Inputs Reviews <e14s-...@google.com>
            Gerrit-CC: Sadrul Chowdhury <sad...@chromium.org>
            Gerrit-CC: gwsq
            Gerrit-Attention: Keith Lee <keit...@chromium.org>
            Gerrit-Attention: jhtin <jh...@chromium.org>
            Gerrit-Attention: Mehrab N <meh...@chromium.org>
            Gerrit-Comment-Date: Mon, 08 Aug 2022 11:14:24 +0000
            Gerrit-HasComments: Yes
            Gerrit-Has-Labels: No
            Gerrit-MessageType: comment

            jhtin (Gerrit)

            unread,
            Aug 8, 2022, 8:21:17 AM8/8/22
            to asvitkine...@chromium.org, keithle...@chromium.org, nona+...@chromium.org, shuche...@chromium.org, tranbaod...@chromium.org, yhanad...@chromium.org, yusuke...@chromium.org, Darren Shen, Chromium Metrics Reviews, Chromium LUCI CQ, Essential Inputs Reviews, Mehrab N, Keith Lee, Tricium, chromium...@chromium.org, Sadrul Chowdhury

            Attention is currently required from: Darren Shen, Keith Lee, Mehrab N.

            Patch set 4:Auto-Submit +1

            View Change

              To view, visit change 3815109. To unsubscribe, or for help writing mail filters, visit settings.

              Gerrit-Project: chromium/src
              Gerrit-Branch: main
              Gerrit-Change-Id: I89c1188603ff34b4befa6abb00f2f4b700135bdd
              Gerrit-Change-Number: 3815109
              Gerrit-PatchSet: 4
              Gerrit-Owner: jhtin <jh...@chromium.org>
              Gerrit-Reviewer: Darren Shen <sh...@chromium.org>
              Gerrit-Reviewer: Keith Lee <keit...@chromium.org>
              Gerrit-Reviewer: Mehrab N <meh...@chromium.org>
              Gerrit-Reviewer: jhtin <jh...@chromium.org>
              Gerrit-CC: Chromium Metrics Reviews <chromium-met...@google.com>
              Gerrit-CC: Essential Inputs Reviews <e14s-...@google.com>
              Gerrit-CC: Sadrul Chowdhury <sad...@chromium.org>
              Gerrit-CC: gwsq
              Gerrit-Attention: Keith Lee <keit...@chromium.org>
              Gerrit-Attention: Mehrab N <meh...@chromium.org>
              Gerrit-Attention: Darren Shen <sh...@chromium.org>
              Gerrit-Comment-Date: Mon, 08 Aug 2022 12:21:06 +0000

              Darren Shen (Gerrit)

              unread,
              Aug 8, 2022, 7:17:15 PM8/8/22
              to jhtin, asvitkine...@chromium.org, keithle...@chromium.org, nona+...@chromium.org, shuche...@chromium.org, tranbaod...@chromium.org, yhanad...@chromium.org, yusuke...@chromium.org, Chromium Metrics Reviews, Chromium LUCI CQ, Essential Inputs Reviews, Mehrab N, Keith Lee, Tricium, chromium...@chromium.org, Sadrul Chowdhury

              Attention is currently required from: Keith Lee, Mehrab N, jhtin.

              Patch set 4:Code-Review +1

              View Change

                To view, visit change 3815109. To unsubscribe, or for help writing mail filters, visit settings.

                Gerrit-Project: chromium/src
                Gerrit-Branch: main
                Gerrit-Change-Id: I89c1188603ff34b4befa6abb00f2f4b700135bdd
                Gerrit-Change-Number: 3815109
                Gerrit-PatchSet: 4
                Gerrit-Owner: jhtin <jh...@chromium.org>
                Gerrit-Reviewer: Darren Shen <sh...@chromium.org>
                Gerrit-Reviewer: Keith Lee <keit...@chromium.org>
                Gerrit-Reviewer: Mehrab N <meh...@chromium.org>
                Gerrit-Reviewer: jhtin <jh...@chromium.org>
                Gerrit-CC: Chromium Metrics Reviews <chromium-met...@google.com>
                Gerrit-CC: Essential Inputs Reviews <e14s-...@google.com>
                Gerrit-CC: Sadrul Chowdhury <sad...@chromium.org>
                Gerrit-CC: gwsq
                Gerrit-Attention: Keith Lee <keit...@chromium.org>
                Gerrit-Attention: jhtin <jh...@chromium.org>
                Gerrit-Attention: Mehrab N <meh...@chromium.org>
                Gerrit-Comment-Date: Mon, 08 Aug 2022 23:17:03 +0000

                Mehrab N (Gerrit)

                unread,
                Aug 8, 2022, 7:45:49 PM8/8/22
                to jhtin, asvitkine...@chromium.org, keithle...@chromium.org, nona+...@chromium.org, shuche...@chromium.org, tranbaod...@chromium.org, yhanad...@chromium.org, yusuke...@chromium.org, Darren Shen, Chromium Metrics Reviews, Chromium LUCI CQ, Essential Inputs Reviews, Keith Lee, Tricium, chromium...@chromium.org, Sadrul Chowdhury

                Attention is currently required from: Keith Lee, jhtin.

                Patch set 4:Code-Review +1

                View Change

                  To view, visit change 3815109. To unsubscribe, or for help writing mail filters, visit settings.

                  Gerrit-Project: chromium/src
                  Gerrit-Branch: main
                  Gerrit-Change-Id: I89c1188603ff34b4befa6abb00f2f4b700135bdd
                  Gerrit-Change-Number: 3815109
                  Gerrit-PatchSet: 4
                  Gerrit-Owner: jhtin <jh...@chromium.org>
                  Gerrit-Reviewer: Darren Shen <sh...@chromium.org>
                  Gerrit-Reviewer: Keith Lee <keit...@chromium.org>
                  Gerrit-Reviewer: Mehrab N <meh...@chromium.org>
                  Gerrit-Reviewer: jhtin <jh...@chromium.org>
                  Gerrit-CC: Chromium Metrics Reviews <chromium-met...@google.com>
                  Gerrit-CC: Essential Inputs Reviews <e14s-...@google.com>
                  Gerrit-CC: Sadrul Chowdhury <sad...@chromium.org>
                  Gerrit-CC: gwsq
                  Gerrit-Attention: Keith Lee <keit...@chromium.org>
                  Gerrit-Attention: jhtin <jh...@chromium.org>
                  Gerrit-Comment-Date: Mon, 08 Aug 2022 23:45:25 +0000

                  Keith Lee (Gerrit)

                  unread,
                  Aug 8, 2022, 11:10:20 PM8/8/22
                  to jhtin, asvitkine...@chromium.org, keithle...@chromium.org, nona+...@chromium.org, shuche...@chromium.org, tranbaod...@chromium.org, yhanad...@chromium.org, yusuke...@chromium.org, Mehrab N, Darren Shen, Chromium Metrics Reviews, Chromium LUCI CQ, Essential Inputs Reviews, Tricium, chromium...@chromium.org, Sadrul Chowdhury

                  Attention is currently required from: jhtin.

                  Patch set 4:Code-Review +1Commit-Queue +2

                  View Change

                    To view, visit change 3815109. To unsubscribe, or for help writing mail filters, visit settings.

                    Gerrit-Project: chromium/src
                    Gerrit-Branch: main
                    Gerrit-Change-Id: I89c1188603ff34b4befa6abb00f2f4b700135bdd
                    Gerrit-Change-Number: 3815109
                    Gerrit-PatchSet: 4
                    Gerrit-Owner: jhtin <jh...@chromium.org>
                    Gerrit-Reviewer: Darren Shen <sh...@chromium.org>
                    Gerrit-Reviewer: Keith Lee <keit...@chromium.org>
                    Gerrit-Reviewer: Mehrab N <meh...@chromium.org>
                    Gerrit-Reviewer: jhtin <jh...@chromium.org>
                    Gerrit-CC: Chromium Metrics Reviews <chromium-met...@google.com>
                    Gerrit-CC: Essential Inputs Reviews <e14s-...@google.com>
                    Gerrit-CC: Sadrul Chowdhury <sad...@chromium.org>
                    Gerrit-CC: gwsq
                    Gerrit-Attention: jhtin <jh...@chromium.org>
                    Gerrit-Comment-Date: Tue, 09 Aug 2022 03:10:07 +0000

                    Chromium LUCI CQ (Gerrit)

                    unread,
                    Aug 9, 2022, 12:31:14 AM8/9/22
                    to jhtin, asvitkine...@chromium.org, keithle...@chromium.org, nona+...@chromium.org, shuche...@chromium.org, tranbaod...@chromium.org, yhanad...@chromium.org, yusuke...@chromium.org, Keith Lee, Mehrab N, Darren Shen, Chromium Metrics Reviews, Essential Inputs Reviews, Tricium, chromium...@chromium.org, Sadrul Chowdhury

                    Chromium LUCI CQ submitted this change.

                    View Change


                    Approvals: Keith Lee: Looks good to me; Commit jhtin: Send CL to CQ automatically after approval Darren Shen: Looks good to me Mehrab N: Looks good to me
                    PK longpress diacritics: Add pref metric

                    Bug: b/241729161
                    Change-Id: I89c1188603ff34b4befa6abb00f2f4b700135bdd
                    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3815109
                    Reviewed-by: Darren Shen <sh...@chromium.org>
                    Reviewed-by: Keith Lee <keit...@chromium.org>
                    Commit-Queue: Keith Lee <keit...@chromium.org>
                    Reviewed-by: Mehrab N <meh...@chromium.org>
                    Auto-Submit: jhtin <jh...@chromium.org>
                    Cr-Commit-Position: refs/heads/main@{#1032872}
                    ---
                    M chrome/browser/ash/input_method/assistive_suggester.cc
                    M chrome/browser/ash/input_method/assistive_suggester_unittest.cc
                    M tools/metrics/histograms/metadata/input/histograms.xml
                    3 files changed, 69 insertions(+), 0 deletions(-)


                    To view, visit change 3815109. To unsubscribe, or for help writing mail filters, visit settings.

                    Gerrit-Project: chromium/src
                    Gerrit-Branch: main
                    Gerrit-Change-Id: I89c1188603ff34b4befa6abb00f2f4b700135bdd
                    Gerrit-Change-Number: 3815109
                    Gerrit-PatchSet: 5
                    Gerrit-Owner: jhtin <jh...@chromium.org>
                    Gerrit-Reviewer: Chromium LUCI CQ <chromiu...@luci-project-accounts.iam.gserviceaccount.com>
                    Gerrit-Reviewer: Darren Shen <sh...@chromium.org>
                    Gerrit-Reviewer: Keith Lee <keit...@chromium.org>
                    Gerrit-Reviewer: Mehrab N <meh...@chromium.org>
                    Gerrit-Reviewer: jhtin <jh...@chromium.org>
                    Gerrit-CC: Chromium Metrics Reviews <chromium-met...@google.com>
                    Gerrit-CC: Essential Inputs Reviews <e14s-...@google.com>
                    Gerrit-CC: Sadrul Chowdhury <sad...@chromium.org>
                    Gerrit-CC: gwsq
                    Gerrit-MessageType: merged
                    Reply all
                    Reply to author
                    Forward
                    0 new messages