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

Issue 1427393002: Revert of Allow Chrome to bind an Application request from mojo_runner. (Closed)

Created:
5 years, 1 month ago by nasko
Modified:
5 years, 1 month ago
CC:
chromium-reviews, rjkroege, qsr+mojo_chromium.org, viettrungluu+watch_chromium.org, yzshen+watch_chromium.org, abarth-chromium, Aaron Boodman, darin (slow to review), ben+mojo_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@callback
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Revert of Allow Chrome to bind an Application request from mojo_runner. (patchset #7 id:100001 of https://codereview.chromium.org/1423063004/ ) Reason for revert: Reverting, as it broke the ChromeOS GN bots. Reverting the other Mojo change (https://codereview.chromium.org/1406073004) did not help, so reverting this one as well. [2774/2777] LINK ./unit_tests FAILED: ../../third_party/llvm-build/Release+Asserts/bin/clang++ -pie -Wl,--fatal-warnings -fPIC -Wl,-z,noexecstack -Wl,-z,now -Wl,-z,relro -Wl,-z,defs -B../../third_party/binutils/Linux_x64/Release/bin -fuse-ld=gold -Wl,--icf=all -pthread -m64 -Wl,-rpath=\$ORIGIN/ -Wl,-rpath-link= -Wl,--disable-new-dtags -Wl,--export-dynamic -o ./chrome -Wl,--start-group @./chrome.rsp ./libbase.so ./libbase_i18n.so ./libicui18n.so ./libicuuc.so ./libprefs.so ./libui_base.so ./libskia.so ./libplatform.so ./libevents_base.so ./libgfx.so ./libgeometry.so ./liburl.so ./libnet.so ./libprotobuf_lite.so ./libcrcrypto.so ./libboringssl.so ./libnative_theme.so ./libkeyed_service_core.so ./libsql.so ./libwebdata_common.so ./libsessions.so ./libprotocol.so ./libcontent.so ./libmojo_system_impl.so ./libgpu.so ./libsync.so ./libcloud_policy_proto_generated_compile.so ./libpolicy_component.so ./libproto.so ./libmojo_environment_impl.so ./libipc.so ./libcommon.so ./libgin.so ./libv8.so ./libppapi_shared.so ./libaccessibility.so ./libgfx_ipc.so ./libgl_wrapper.so ./libmessage_center.so ./libblink_common.so ./libblink_web.so ./libblink_platform.so ./libgles2_c_lib.so ./libmedia.so ./libdevice_event_log.so ./libdevice_core.so ./libchromeos.so ./libdbus.so ./liburl_matcher.so ./libbluetooth.so ./libkeyboard_with_content.so ./libprinting.so ./libkeyed_service_content.so ./libonc.so ./libshared_memory_support.so ./libgcm.so ./libstorage_common.so ./libui_base_ime.so ./libstorage_browser.so ./libchrome_apps.so ./libcc.so ./libppapi_host.so ./libaura.so ./libcompositor.so ./libevents.so ./libviews.so ./libash.so ./libkeyboard.so ./libx11_events_platform.so ./libdevices.so ./libgfx_x11.so ./libwebview.so ./libgesture_detection.so ./libsnapshot.so ./libbrowser_ui_views.so ./libnet_with_v8.so ./libsurface.so ./libweb_dialogs.so ./libppapi_proxy.so ./libwallpaper.so ./libash_with_content.so ./liblogin.so ./libcore.so ./libuser_manager.so ./libownership.so ./libproxy_config.so ./libapp_list.so ./libui_chromeos.so ./libdisplay.so ./libfile_manager.so ./libwm.so ./libcaptive_portal.so ./libdomain_reliability.so ./libchromium_sqlite3.so ./libstartup_tracing.so ./libuser_prefs.so ./libshell_dialogs.so ./libmidi.so ./libmojo_common_lib.so ./libmessage_pump.so ./libgles2_implementation.so ./libgles2_utils.so ./libtranslator.so -Wl,--end-group -ldl -lpangocairo-1.0 -lpango-1.0 -lcairo -lgobject-2.0 -lglib-2.0 -lX11 -lXcomposite -lXcursor -lXdamage -lXext -lXfixes -lXi -lXrender -lXtst -ldbus-1 -lpthread -lrt -lnss3 -lnssutil3 -lsmime3 -lplds4 -lplc4 -lnspr4 -lexpat -lXrandr -lm -lfontconfig -lfreetype ../../third_party/mojo/src/mojo/edk/embedder/embedder.cc:132:error: undefined reference to 'mojo::edk::internal::g_core' ../../third_party/mojo/src/mojo/edk/embedder/entrypoints.cc:33:error: undefined reference to 'mojo::edk::internal::g_core' ../../third_party/mojo/src/mojo/edk/embedder/entrypoints.cc:39:error: undefined reference to 'mojo::edk::internal::g_core' ../../third_party/mojo/src/mojo/edk/embedder/entrypoints.cc:48:error: undefined reference to 'mojo::edk::internal::g_core' ../../third_party/mojo/src/mojo/edk/embedder/test_embedder.cc:63:error: undefined reference to 'mojo::edk::internal::g_platform_support' ../../third_party/mojo/src/mojo/edk/embedder/test_embedder.cc:66:error: undefined reference to 'mojo::edk::internal::g_platform_support' ../../third_party/mojo/src/mojo/edk/embedder/test_embedder.cc:67:error: undefined reference to 'mojo::edk::internal::g_platform_support' clang:error: linker command failed with exit code 1 (use -v to see invocation) FAILED: ../../third_party/llvm-build/Release+Asserts/bin/clang++ -Wl,--fatal-warnings -fPIC -Wl,-z,noexecstack -Wl,-z,now -Wl,-z,relro -Wl,-z,defs -B../../third_party/binutils/Linux_x64/Release/bin -fuse-ld=gold -Wl,--icf=all -pthread -m64 -Wl,-rpath=\$ORIGIN/ -Wl,-rpath-link= -Wl,--disable-new-dtags -Wl,--export-dynamic -o ./interactive_ui_tests -Wl,--start-group @./interactive_ui_tests.rsp ./libnet.so ./libskia.so ./libsync.so ./libviews.so ./libui_touch_selection.so ./libchromeos.so ./libcontent.so ./libmojo_system_impl.so ./libgpu.so ./libblink_common.so ./libblink_web.so ./libblink_platform.so ./libgles2_c_lib.so ./liburl.so ./libv8.so ./libcc_blink.so ./libbase.so ./libbase_i18n.so ./libicui18n.so ./libicuuc.so ./libcrcrypto.so ./libprotobuf_lite.so ./libboringssl.so ./libnet_with_v8.so ./libstorage_common.so ./libui_base.so ./libplatform.so ./libevents_base.so ./libgfx.so ./libgeometry.so ./libui_base_ime.so ./libevents.so ./libgesture_detection.so ./libdevices.so ./libgfx_x11.so ./libgl_wrapper.so ./libx11_window.so ./libmojo_environment_impl.so ./libscheduler.so ./libcc.so ./libcc_surfaces.so ./libgles2_implementation.so ./libgl_in_process_context.so ./libgles2_utils.so ./libmedia.so ./libmojo.so ./libmojo_common_lib.so ./libmessage_pump.so ./libppapi_host.so ./libppapi_proxy.so ./libipc.so ./libppapi_shared.so ./libstorage_browser.so ./libcompositor.so ./libsurface.so ./libaura.so ./libwm.so ./libgin.so ./libprefs.so ./libnative_theme.so ./libkeyed_service_core.so ./libsql.so ./libwebdata_common.so ./libsessions.so ./libprotocol.so ./libcloud_policy_proto_generated_compile.so ./libpolicy_component.so ./libproto.so ./libcommon.so ./libaccessibility.so ./libgfx_ipc.so ./libmessage_center.so ./libdevice_event_log.so ./libdevice_core.so ./libdbus.so ./liburl_matcher.so ./libbluetooth.so ./libkeyboard_with_content.so ./libprinting.so ./libkeyed_service_content.so ./libonc.so ./libshared_memory_support.so ./libgcm.so ./libchrome_apps.so ./libash.so ./libkeyboard.so ./libx11_events_platform.so ./libwebview.so ./libsnapshot.so ./libbrowser_ui_views.so ./libweb_dialogs.so ./libwallpaper.so ./libash_with_content.so ./liblogin.so ./libcore.so ./libuser_manager.so ./libownership.so ./libproxy_config.so ./libapp_list.so ./libui_chromeos.so ./libdisplay.so ./libfile_manager.so ./libcaptive_portal.so ./libdomain_reliability.so ./libchromium_sqlite3.so ./libstartup_tracing.so ./libuser_prefs.so ./libshell_dialogs.so ./libmidi.so ./libtranslator.so -Wl,--end-group -ldl -lXtst -lgmodule-2.0 -lgobject-2.0 -lgthread-2.0 -lrt -lglib-2.0 -lnss3 -lnssutil3 -lsmime3 -lplds4 -lplc4 -lnspr4 -lfontconfig -lX11 -lXcomposite -lXcursor -lXdamage -lXext -lXfixes -lXi -lXrender -lexpat -lm -ldbus-1 -lpthread -lXrandr -lfreetype ../../third_party/mojo/src/mojo/edk/embedder/embedder.cc:132:error: undefined reference to 'mojo::edk::internal::g_core' ../../third_party/mojo/src/mojo/edk/embedder/entrypoints.cc:33:error: undefined reference to 'mojo::edk::internal::g_core' ../../third_party/mojo/src/mojo/edk/embedder/entrypoints.cc:39:error: undefined reference to 'mojo::edk::internal::g_core' ../../third_party/mojo/src/mojo/edk/embedder/entrypoints.cc:48:error: undefined reference to 'mojo::edk::internal::g_core' ../../third_party/mojo/src/mojo/edk/embedder/test_embedder.cc:63:error: undefined reference to 'mojo::edk::internal::g_platform_support' ../../third_party/mojo/src/mojo/edk/embedder/test_embedder.cc:66:error: undefined reference to 'mojo::edk::internal::g_platform_support' ../../third_party/mojo/src/mojo/edk/embedder/test_embedder.cc:67:error: undefined reference to 'mojo::edk::internal::g_platform_support' clang:error: linker command failed with exit code 1 (use -v to see invocation) FAILED: ../../third_party/llvm-build/Release+Asserts/bin/clang++ -Wl,--fatal-warnings -fPIC -Wl,-z,noexecstack -Wl,-z,now -Wl,-z,relro -Wl,-z,defs -B../../third_party/binutils/Linux_x64/Release/bin -fuse-ld=gold -Wl,--icf=all -pthread -m64 -Wl,-rpath=\$ORIGIN/ -Wl,-rpath-link= -Wl,--disable-new-dtags -Wl,--export-dynamic -o ./unit_tests -Wl,--start-group @./unit_tests.rsp ./libnet.so ./libskia.so ./libgl_wrapper.so ./libv8.so ./libwm.so ./libdevices.so ./libdbus.so ./libviews.so ./libbase.so ./libbase_i18n.so ./libicui18n.so ./libicuuc.so ./libprefs.so ./libui_base.so ./libplatform.so ./libevents_base.so ./libgfx.so ./libgeometry.so ./liburl.so ./libprotobuf_lite.so ./libcrcrypto.so ./libboringssl.so ./libnative_theme.so ./libkeyed_service_core.so ./libsql.so ./libwebdata_common.so ./libsessions.so ./libprotocol.so ./libcontent.so ./libmojo_system_impl.so ./libgpu.so ./libsync.so ./libcloud_policy_proto_generated_compile.so ./libpolicy_component.so ./libproto.so ./libmojo_environment_impl.so ./libipc.so ./libcommon.so ./libgin.so ./libppapi_shared.so ./libaccessibility.so ./libgfx_ipc.so ./libmessage_center.so ./libblink_common.so ./libblink_web.so ./libblink_platform.so ./libgles2_c_lib.so ./libmedia.so ./libdevice_event_log.so ./libdevice_core.so ./libchromeos.so ./liburl_matcher.so ./libbluetooth.so ./libkeyboard_with_content.so ./libprinting.so ./libkeyed_service_content.so ./libonc.so ./libshared_memory_support.so ./libgcm.so ./libstorage_common.so ./libui_base_ime.so ./libstorage_browser.so ./libchrome_apps.so ./libcc.so ./libppapi_host.so ./libaura.so ./libcompositor.so ./libevents.so ./libash.so ./libkeyboard.so ./libx11_events_platform.so ./libgfx_x11.so ./libwebview.so ./libgesture_detection.so ./libsnapshot.so ./libbrowser_ui_views.so ./libnet_with_v8.so ./libsurface.so ./libweb_dialogs.so ./libppapi_proxy.so ./libwallpaper.so ./libash_with_content.so ./liblogin.so ./libcore.so ./libuser_manager.so ./libownership.so ./libproxy_config.so ./libapp_list.so ./libui_chromeos.so ./libdisplay.so ./libfile_manager.so ./libcaptive_portal.so ./libdomain_reliability.so ./libchromium_sqlite3.so ./libstartup_tracing.so ./libuser_prefs.so ./libshell_dialogs.so ./libmidi.so ./libmojo_common_lib.so ./libmessage_pump.so ./libgles2_implementation.so ./libgles2_utils.so ./libtranslator.so ./libcc_blink.so ./libx11_window.so ./libscheduler.so ./libcc_surfaces.so ./libgl_in_process_context.so ./libmojo.so -Wl,--end-group -ldl -ldbus-1 -lpthread -lrt -lX11 -lXcomposite -lXcursor -lXdamage -lXext -lXfixes -lXi -lXrender -lXtst -lnss3 -lnssutil3 -lsmime3 -lplds4 -lplc4 -lnspr4 -lexpat -lXrandr -lm -lfontconfig -lgmodule-2.0 -lgobject-2.0 -lgthread-2.0 -lglib-2.0 -lfreetype ../../third_party/mojo/src/mojo/edk/embedder/embedder.cc:132:error: undefined reference to 'mojo::edk::internal::g_core' ../../third_party/mojo/src/mojo/edk/embedder/entrypoints.cc:33:error: undefined reference to 'mojo::edk::internal::g_core' ../../third_party/mojo/src/mojo/edk/embedder/entrypoints.cc:39:error: undefined reference to 'mojo::edk::internal::g_core' ../../third_party/mojo/src/mojo/edk/embedder/entrypoints.cc:48:error: undefined reference to 'mojo::edk::internal::g_core' ../../third_party/mojo/src/mojo/edk/embedder/test_embedder.cc:63:error: undefined reference to 'mojo::edk::internal::g_platform_support' ../../third_party/mojo/src/mojo/edk/embedder/test_embedder.cc:66:error: undefined reference to 'mojo::edk::internal::g_platform_support' ../../third_party/mojo/src/mojo/edk/embedder/test_embedder.cc:67:error: undefined reference to 'mojo::edk::internal::g_platform_support' clang:error: linker command failed with exit code 1 (use -v to see invocation) ninja: build stopped: subcommand failed. Original issue's description: > Allow Chrome to bind an Application request from mojo_runner. > > When chrome.exe is run from mojo_runner, block on startup until the InterfaceRequest<Application> is bound, and a message loop is run until the runner's shell sends Initialize(). > > R=sky@chromium.org > http://crbug.com/548694 > > Committed: https://crrev.com/2b4a62d4282df3fe69696925a6b84fdca1df2d0f > Cr-Commit-Position: refs/heads/master@{#358145} TBR=sky@chromium.org,ben@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Committed: https://crrev.com/9580fa43b47d303541f08fffc156cd524b4dba6d Cr-Commit-Position: refs/heads/master@{#358172}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+14 lines, -154 lines) Patch
M chrome/browser/BUILD.gn View 1 chunk +1 line, -10 lines 0 comments Download
M chrome/browser/DEPS View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/browser/chrome_browser_main.h View 2 chunks +0 lines, -5 lines 0 comments Download
M chrome/browser/chrome_browser_main.cc View 3 chunks +0 lines, -12 lines 0 comments Download
D chrome/browser/mojo_runner_state.h View 1 chunk +0 lines, -41 lines 0 comments Download
D chrome/browser/mojo_runner_state.cc View 1 chunk +0 lines, -39 lines 0 comments Download
D chrome/browser/mojo_runner_util.h View 1 chunk +0 lines, -11 lines 0 comments Download
D chrome/browser/mojo_runner_util.cc View 1 chunk +0 lines, -12 lines 0 comments Download
M chrome/chrome_browser.gypi View 1 chunk +0 lines, -2 lines 0 comments Download
M components/mus/example/main/main_application_delegate.cc View 1 chunk +0 lines, -1 line 0 comments Download
M mojo/application/public/cpp/application_impl.h View 1 chunk +0 lines, -4 lines 0 comments Download
M mojo/application/public/cpp/lib/application_impl.cc View 2 chunks +5 lines, -10 lines 0 comments Download
M mojo/runner/BUILD.gn View 2 chunks +8 lines, -6 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
nasko
Created Revert of Allow Chrome to bind an Application request from mojo_runner.
5 years, 1 month ago (2015-11-05 23:24:12 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1427393002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1427393002/1
5 years, 1 month ago (2015-11-05 23:25:28 UTC) #2
commit-bot: I haz the power
Committed patchset #1 (id:1)
5 years, 1 month ago (2015-11-05 23:27:54 UTC) #3
commit-bot: I haz the power
5 years, 1 month ago (2015-11-05 23:29:10 UTC) #4
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/9580fa43b47d303541f08fffc156cd524b4dba6d
Cr-Commit-Position: refs/heads/master@{#358172}

Powered by Google App Engine
This is Rietveld 408576698