Commit ccc6d585 authored by Gayane Petrosyan's avatar Gayane Petrosyan Committed by Chromium LUCI CQ

[SH-Clank] Cleanup tab observer after iframe is deteected.

cleanup() should have also been called after line:79 when iframe is
detected.

Bug: 1157974
Change-Id: I505fa12a82a17da4f3a2ae6dcd239c0f0613be59
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2588644Reviewed-by: default avatarKyle Milka <kmilka@chromium.org>
Commit-Queue: Gayane Petrosyan <gayane@chromium.org>
Cr-Commit-Position: refs/heads/master@{#836733}
parent be236438
...@@ -70,6 +70,8 @@ public class LinkToTextCoordinator extends EmptyTabObserver { ...@@ -70,6 +70,8 @@ public class LinkToTextCoordinator extends EmptyTabObserver {
mContext.getResources().getDimensionPixelSize(R.dimen.y_offset)); mContext.getResources().getDimensionPixelSize(R.dimen.y_offset));
toast.show(); toast.show();
} }
// After generation results are communicated to users, cleanup to remove tab listener.
cleanup();
} }
public void requestSelector() { public void requestSelector() {
...@@ -85,7 +87,6 @@ public class LinkToTextCoordinator extends EmptyTabObserver { ...@@ -85,7 +87,6 @@ public class LinkToTextCoordinator extends EmptyTabObserver {
@Override @Override
public void call(String selector) { public void call(String selector) {
onSelectorReady(selector); onSelectorReady(selector);
cleanup();
} }
}); });
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment