tree 97bfc5fcfc02f6a6205d5b31b15d3256b3a5131c
parent 65116e80fe8b54af11a497442ed64981bcd9da78
author Filip Filmar <fmil@google.com> 1617904292 +0000
committer CQ Bot <commit-bot@chromium.org> 1617904292 +0000

Reland "[ime] Migrates ImeService.ViewFocusChanged to ...focus.Controller.Notify"

This reverts commit b28fb6b2c90f0b7c1fc8cd8937cebe7ac3dd4c91.

Reason for revert: the root cause of the empty screen has been removed.
In short, the change was missing protocol routing declarations, which
have now been added.

Requires fxr/512198 to ensure Workstation does not break.

Tested:
  Verified locally and confirmed that the workstation screen is back.

Original change's description:
> Revert "[ime] Migrates ImeService.ViewFocusChanged to ...focus.Controller.Notify"
>
> This reverts commit c7c1697ebaa04471c9bb501cbfd2ddeefde98064.
>
> Reason for revert: Seems like the change causes workstation to show
> black screen only.
>
> Original change's description:
> > [ime] Migrates ImeService.ViewFocusChanged to ...focus.Controller.Notify
> >
> > We are factoring out the protocol member ImeService.ViewFocusChanged
> > into its own separate protocol for finer-grained access control.
> >
> > Looks like no users of this protocol remain out of tree, so we're making
> > the change and removing the respective protocol member at once.
> >
> > Tested: CI, GI, LSC presubmit
> > Fixed: 71970, 70966
> > Change-Id: I53b5103768b8b950e60f9b68490774ec8f19d833
> > Reviewed-on: https://fuchsia-review.googlesource.com/c/fuchsia/+/510795
> > Fuchsia-Auto-Submit: Filip Filmar <fmil@google.com>
> > Reviewed-by: Alice Neels <neelsa@google.com>
> > Reviewed-by: Gabe Schine <thatguy@google.com>
> > API-Review: Alice Neels <neelsa@google.com>
> > Commit-Queue: Auto-Submit <auto-submit@fuchsia-infra.iam.gserviceaccount.com>
>
> TBR=thatguy@google.com,viktard@google.com,quiche@google.com,neelsa@google.com,fmil@google.com,auto-submit@fuchsia-infra.iam.gserviceaccount.com
>
> Change-Id: I5aeb98308a881f6b24b8d038d4bb5eec7f620ee8
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Reviewed-on: https://fuchsia-review.googlesource.com/c/fuchsia/+/511693
> Reviewed-by: Alice Neels <neelsa@google.com>
> Reviewed-by: Filip Filmar <fmil@google.com>
> Reviewed-by: Yegor Pomortsev <ypomortsev@google.com>
> Reviewed-by: Gabe Schine <thatguy@google.com>
> Fuchsia-Auto-Submit: Filip Filmar <fmil@google.com>
> Commit-Queue: Auto-Submit <auto-submit@fuchsia-infra.iam.gserviceaccount.com>
> API-Review: Alice Neels <neelsa@google.com>

Change-Id: I2c4d7f0d9c1e07cdce64632f670bf3a57b416e6b
Reviewed-on: https://fuchsia-review.googlesource.com/c/fuchsia/+/512034
Fuchsia-Auto-Submit: Filip Filmar <fmil@google.com>
Reviewed-by: Mukesh Agrawal <quiche@google.com>
Reviewed-by: Gabe Schine <thatguy@google.com>
Reviewed-by: Alice Neels <neelsa@google.com>
Commit-Queue: Auto-Submit <auto-submit@fuchsia-infra.iam.gserviceaccount.com>
API-Review: Alice Neels <neelsa@google.com>