AppProcessHasPermission() does more than just a check and return a bool: it kills the content process if it doesn't have the permission that's being queried. The naming should reflect that. In bug 776832 we went with "assertPermission", so I'm proposing AssertAppProcessPermission().

Comment on attachment 663796 [details] [diff] [review] Update C++ permission-checking API to match that of the message-manager one > /** > * Return true iff any of the PBrowsers loaded in this content process > * has the specified capability. > */ Can you update the comment here, too? (It's not clear to me that the comment above applies here, since this one also has an explicit comment.) > bool >-AppProcessHasPermission(mozilla::dom::PContentParent* aActor, >- const char* aPermission); >+AssertAppProcessPermission(mozilla::dom::PContentParent* aActor, >+ const char* aPermission);

Sure.