Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(88)

Issue 1954263003: Monet: Roll mojo_sdk to 3762c8c0b74b48b00786a42fca8f7db7f761935f. (Closed)

Created:
4 years, 7 months ago by viettrungluu
Modified:
4 years, 7 months ago
Reviewers:
vardhan, jamesr
CC:
mojo-reviews_chromium.org
Base URL:
https://github.com/domokit/monet.git@master
Target Ref:
refs/heads/master
Project:
monet
Visibility:
Public.

Description

Monet: Roll mojo_sdk to 3762c8c0b74b48b00786a42fca8f7db7f761935f. This is the latest mojo_sdk, and should be up to date with mojo.git at b7dbc87691e8f9ddb85e2ae367293dace441776d. Also update: * //mojo/{application,common,services/{public,tracing}} (from mojo.git) * //third_party/mojo/MOJO_VERSION * //mojo/data_pipe_utils/BUILD.gn * various things in //mojo/services/network, to use new mojo::ConnectToService() and avoid the old ApplicationImpl::ConnectToApplication(), etc. (All things mojo from mojo.git at b7dbc87691e8f9ddb85e2ae367293dace441776d, except for the fix for //mojo/data_pipe_utils/BUILD.gn which landed at //12c8df4e201fdcd096f2ab4608008cd35f072d05.) TBR=jamesr@chromium.org,vardhan@google.com Committed: https://github.com/domokit/monet4bd8ec848bf53694f1fcb31d5303461aa0e2f472

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+59 lines, -49 lines) Patch
M DEPS View 1 chunk +1 line, -1 line 0 comments Download
M mojo/application/application_runner_chromium.cc View 1 chunk +1 line, -1 line 0 comments Download
M mojo/application/content_handler_factory.cc View 1 chunk +0 lines, -1 line 0 comments Download
M mojo/common/BUILD.gn View 1 chunk +1 line, -0 lines 0 comments Download
D mojo/common/dart/.packages View 1 chunk +0 lines, -2 lines 0 comments Download
D mojo/common/dart/packages/common View 1 chunk +0 lines, -1 line 0 comments Download
M mojo/common/tracing_impl.h View 1 chunk +2 lines, -8 lines 0 comments Download
M mojo/common/tracing_impl.cc View 3 chunks +11 lines, -8 lines 0 comments Download
M mojo/data_pipe_utils/BUILD.gn View 2 chunks +3 lines, -1 line 0 comments Download
M mojo/services/network/host_resolver_apptest.cc View 2 chunks +3 lines, -4 lines 0 comments Download
M mojo/services/network/http_server_apptest.cc View 2 chunks +3 lines, -4 lines 0 comments Download
M mojo/services/network/udp_socket_apptest.cc View 2 chunks +3 lines, -4 lines 0 comments Download
M mojo/services/network/url_loader_interceptor_apptest.cc View 2 chunks +3 lines, -4 lines 0 comments Download
M mojo/services/public/build/config/BUILD.gn View 1 chunk +4 lines, -9 lines 0 comments Download
M mojo/services/tracing/interfaces/BUILD.gn View 1 chunk +3 lines, -0 lines 0 comments Download
A mojo/services/tracing/interfaces/trace_provider_registry.mojom View 1 chunk +17 lines, -0 lines 0 comments Download
M mojo/services/tracing/interfaces/tracing.mojom View 1 chunk +3 lines, -0 lines 0 comments Download
M third_party/mojo/MOJO_VERSION View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 3 (1 generated)
viettrungluu
Committed patchset #1 (id:1) manually as 4bd8ec848bf53694f1fcb31d5303461aa0e2f472 (presubmit successful).
4 years, 7 months ago (2016-05-06 21:31:44 UTC) #2
viettrungluu
4 years, 7 months ago (2016-05-06 21:32:47 UTC) #3
Message was sent while issue was closed.
TBRed, since the changes are very routine:
jamesr -> general stuff
vardhan -> ConnectToApplication()/ConnectToService() conversions (see CL
description)

Powered by Google App Engine
This is Rietveld 408576698