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

Issue 799479 link

Starred by 2 users

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Rename public/interfaces to public/mojom

Project Member Reported by roc...@chromium.org, Jan 5 2018

Issue description

This is in the context of Mojo APIs as well as any service or public component APIs across the code base.

There is some agreement that "mojom" is a better name than "interfaces" for several reasons:

 - more accurate; not all mojom are interface definitions
 - consistent with all other public subdirs which are names of languages
 - less typing!

The downside is that the ship has sailed pretty far out to sea on this issue, so renaming everything will be lots of churn. Still worth doing at low priority IMHO, and we should encourage use of "mojom" moving forward.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Sep 18

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

commit b595cb9f5e2726ae028954fa7f6de7c2c2b182ac
Author: Amos Lim <eui-sang.lim@samsung.com>
Date: Tue Sep 18 23:18:13 2018

Rename components/web_cache/public/interfaces to mojom

As per updated naming conventions, rename interfaces to mojom

Bug: 799479, 806965
Change-Id: Ie539ca95b7e221d4568f0b03b736f354a882d2e6
Reviewed-on: https://chromium-review.googlesource.com/1229726
Reviewed-by: Nico Weber <thakis@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Ken Rockot <rockot@chromium.org>
Commit-Queue: Amos Lim <eui-sang.lim@samsung.com>
Cr-Commit-Position: refs/heads/master@{#592250}
[modify] https://crrev.com/b595cb9f5e2726ae028954fa7f6de7c2c2b182ac/components/web_cache/browser/BUILD.gn
[modify] https://crrev.com/b595cb9f5e2726ae028954fa7f6de7c2c2b182ac/components/web_cache/browser/web_cache_manager.h
[rename] https://crrev.com/b595cb9f5e2726ae028954fa7f6de7c2c2b182ac/components/web_cache/public/mojom/BUILD.gn
[rename] https://crrev.com/b595cb9f5e2726ae028954fa7f6de7c2c2b182ac/components/web_cache/public/mojom/OWNERS
[rename] https://crrev.com/b595cb9f5e2726ae028954fa7f6de7c2c2b182ac/components/web_cache/public/mojom/web_cache.mojom
[modify] https://crrev.com/b595cb9f5e2726ae028954fa7f6de7c2c2b182ac/components/web_cache/renderer/BUILD.gn
[modify] https://crrev.com/b595cb9f5e2726ae028954fa7f6de7c2c2b182ac/components/web_cache/renderer/web_cache_impl.h

Sign in to add a comment