New issue
Advanced search Search tips

Issue 614410 link

Starred by 2 users
Status: Available
Owner: ----
Cc:
rdevlin....@chromium.org
lazyboy@chromium.org
rockot@google.com
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug

Blocked on:
issue 697613
Blocking:
issue 692120



Sign in to add a comment

Convert Extension Request Messages to Mojo

Project Member Reported by rdevlin....@chromium.org, May 24 2016 
Mojo is the new hotness.  We should convert extension request messages (the IPCs that correspond to making extension API calls) to mojo.  In the long run, it would actually be better to have a mojo message for each different API, which would save on (de)serialization, but that's much longer term.
Project Member

Comment 1 by bugdroid1@chromium.org, May 24 2016 

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

commit a6458a0c45099522f647b63337f94ea4e98c9f94
Author: rdevlin.cronin <rdevlin.cronin@chromium.org>
Date: Tue May 24 21:33:59 2016

[Mojo] Add [Native] type traits for base::Values

Mojo struct traits aren't quite ready for something like base::Value,
but adding [native] traits will allow base::DictionaryValue and
base::ListValue to be passed in mojo messages.

BUG=614410

Review-Url: https://codereview.chromium.org/2013433002
Cr-Commit-Position: refs/heads/master@{#395701}

[modify] https://crrev.com/a6458a0c45099522f647b63337f94ea4e98c9f94/mojo/common/common_custom_types.mojom
[modify] https://crrev.com/a6458a0c45099522f647b63337f94ea4e98c9f94/mojo/common/common_custom_types.typemap
[modify] https://crrev.com/a6458a0c45099522f647b63337f94ea4e98c9f94/mojo/common/common_custom_types_unittest.cc
[modify] https://crrev.com/a6458a0c45099522f647b63337f94ea4e98c9f94/mojo/common/test_common_custom_types.mojom

Comment 2 by rdevlin....@chromium.org, Oct 21 2016

Cc: rdevlin....@chromium.org
Owner: catmulli...@chromium.org
catmullings@ is going to pursue migrating extensions IPCs to mojo.

Comment 3 by rdevlin....@chromium.org, Feb 14 2017

Blocking: 692120

Comment 4 by catmulli...@chromium.org, Mar 3 2017

Blockedon: 697613

Comment 5 by catmulli...@chromium.org, Feb 8 2018

Owner: ----

Comment 6 by asargent@chromium.org, May 15 2018

Cc: -asargent@chromium.org

Comment 7 by roc...@chromium.org, Oct 17

Cc: -roc...@chromium.org rockot@google.com

Comment 8 by benhenry@google.com, Jan 11

Status: Available (was: Started)
This issue has been marked as started, but has no owner. Making available.
►

Sign in to add a comment