Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove useBrowserInterfaceBroker from idl files and JS generate template #21187

Merged
merged 1 commit into from Feb 4, 2020

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Jan 15, 2020

This CL removes the useBrowserInterfaceBroker parameter in
mojo.idl, mojo_interface_interceptor.idl and JS generate template
since all the interfaces in the following call sites are acquired
through BrowserInterfaceBroker.

Bug: 995556, 994843
Change-Id: I1b50496084eaec315539d75c40a39ceff8c8aa43
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1963910
Commit-Queue: Miyoung Shin <myid.shin@igalia.com>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Reviewed-by: Oksana Zhuravlova <oksamyt@chromium.org>
Cr-Commit-Position: refs/heads/master@{#738142}

Copy link
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The review process for this patch is being conducted in the Chromium project.

@chromium-wpt-export-bot chromium-wpt-export-bot force-pushed the chromium-export-cl-1963910 branch 2 times, most recently from dea2342 to 08debd8 Compare January 17, 2020 11:34
@gsnedders gsnedders closed this Jan 24, 2020
@gsnedders gsnedders deleted the chromium-export-cl-1963910 branch January 24, 2020 18:05
@gsnedders gsnedders restored the chromium-export-cl-1963910 branch January 24, 2020 18:52
@Hexcles Hexcles reopened this Jan 24, 2020
This CL removes the useBrowserInterfaceBroker parameter in
mojo.idl, mojo_interface_interceptor.idl and JS generate template
since all the interfaces in the following call sites are acquired
through BrowserInterfaceBroker.

Bug: 995556, 994843
Change-Id: I1b50496084eaec315539d75c40a39ceff8c8aa43
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1963910
Commit-Queue: Miyoung Shin <myid.shin@igalia.com>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Reviewed-by: Oksana Zhuravlova <oksamyt@chromium.org>
Cr-Commit-Position: refs/heads/master@{#738142}
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants