| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL changes the behavior to:
- connection error handler doesn't reenter ongoing sync calls.
- connection error handler is delayed until all queued asynchronous
messages are processed.
BUG=577699
Review URL: https://codereview.chromium.org/1781573004
Cr-Commit-Position: refs/heads/master@{#380259}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL introduces the correct re-entrancy behavior: when a sync call is waiting
for response, allow incoming sync requests on the same thread to re-enter, async
messages are queued until the sync call completes.
The following will be in future CLs:
- Support sync calls with associated interfaces.
The original CL is at https://codereview.chromium.org/1713203002/
BUG=577699
Review URL: https://codereview.chromium.org/1723673002
Cr-Commit-Position: refs/heads/master@{#376892}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
id:100001 of https://codereview.chromium.org/1713203002/ )
Reason for revert:
This change is triggering a use-of-uninitialized-value error on the Linux and Chrome OS MSan bots:
https://build.chromium.org/p/chromium.memory.fyi/builders/Linux%20ChromeOS%20MSan%20Tests/builds/7464
Original issue's description:
> Mojo C++ bindings: support sync methods - part 2
>
> This CL introduces the correct re-entrancy behavior: when a sync call is waiting
> for response, allow incoming sync requests on the same thread to re-enter, async
> messages are queued until the sync call completes.
>
> The following will be in future CLs:
> - Support sync calls with associated interfaces.
>
> BUG=577699
>
> Committed: https://crrev.com/083f46638041ba7d7620e47a7f9edf06982a9340
> Cr-Commit-Position: refs/heads/master@{#376751}
TBR=jam@chromium.org,yzshen@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=577699
Review URL: https://codereview.chromium.org/1720093002
Cr-Commit-Position: refs/heads/master@{#376793}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL introduces the correct re-entrancy behavior: when a sync call is waiting
for response, allow incoming sync requests on the same thread to re-enter, async
messages are queued until the sync call completes.
The following will be in future CLs:
- Support sync calls with associated interfaces.
BUG=577699
Review URL: https://codereview.chromium.org/1713203002
Cr-Commit-Position: refs/heads/master@{#376751}
|
|
This CL:
- adds support for [Sync] attribute;
- generates C++ sync method signatures;
- does the simplest thing to block and wait for results of sync calls. (Allows
any messages of the same interface ptr to re-enter; disallows any messages of
other message pipes to re-enter.)
The following will be in future CLs:
- Change the re-entrancy behavior.
- Support sync calls with associated interfaces.
BUG=577699
Review URL: https://codereview.chromium.org/1701423002
Cr-Commit-Position: refs/heads/master@{#376023}
|