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

Issue 1407273010: Reland of Change mojo_public_bindings_unittests to use chromium environment. (Closed)

Created:
5 years, 1 month ago by nasko
Modified:
5 years, 1 month ago
Reviewers:
yzshen1, sky
CC:
chromium-reviews, 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@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Reland of Change mojo_public_bindings_unittests to use chromium environment. (patchset #1 id:1 of https://codereview.chromium.org/1430913005/ ) Reason for revert: Another mojo patch was causing compile errors too. Trying to undo the revert and see if this patch is clean. Original issue's description: > Revert of Change mojo_public_bindings_unittests to use chromium environment. (patchset #5 id:70001 of https://codereview.chromium.org/1406073004/ ) > > Reason for revert: > Broke compile on ChromeOS GN bot: > > http://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%20GN%20%28dbg%29/builds/11875/ > > [4097/4100] STAMP obj/chrome/test/browser_tests_js_webui.stamp > 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: > > Change mojo_public_bindings_unittests to use chromium environment. > > > > This is needed by the associated interface implementation, which uses > > base::MessageLoop and related classes. It won't work with the standalone > > environment, and we want to stop using the standalone environment anyway. > > > > BUG=546067 > > > > Committed: https://crrev.com/70bfaf3f892dcfe48e980c01e22422226816f717 > > Cr-Commit-Position: refs/heads/master@{#358146} > > TBR=sky@chromium.org,yzshen@chromium.org > NOPRESUBMIT=true > NOTREECHECKS=true > NOTRY=true > BUG=546067 > > Committed: https://crrev.com/9f14d56493895bf874c7780a7d5e7c2c51ff0f77 > Cr-Commit-Position: refs/heads/master@{#358155} TBR=sky@chromium.org,yzshen@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=546067 Committed: https://crrev.com/e33507ce46666657829b8fc87daa196797b1a774 Cr-Commit-Position: refs/heads/master@{#358189}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+63 lines, -145 lines) Patch
M third_party/mojo/mojo_edk_tests.gyp View 1 chunk +2 lines, -1 line 0 comments Download
M third_party/mojo/src/mojo/public/cpp/bindings/tests/BUILD.gn View 1 chunk +2 lines, -1 line 0 comments Download
M third_party/mojo/src/mojo/public/cpp/bindings/tests/array_unittest.cc View 2 chunks +1 line, -8 lines 0 comments Download
M third_party/mojo/src/mojo/public/cpp/bindings/tests/binding_callback_unittest.cc View 3 chunks +4 lines, -4 lines 0 comments Download
M third_party/mojo/src/mojo/public/cpp/bindings/tests/binding_unittest.cc View 2 chunks +7 lines, -8 lines 0 comments Download
M third_party/mojo/src/mojo/public/cpp/bindings/tests/connector_unittest.cc View 5 chunks +6 lines, -7 lines 0 comments Download
M third_party/mojo/src/mojo/public/cpp/bindings/tests/equals_unittest.cc View 2 chunks +2 lines, -8 lines 0 comments Download
M third_party/mojo/src/mojo/public/cpp/bindings/tests/handle_passing_unittest.cc View 2 chunks +5 lines, -4 lines 0 comments Download
M third_party/mojo/src/mojo/public/cpp/bindings/tests/interface_ptr_unittest.cc View 5 chunks +7 lines, -9 lines 0 comments Download
M third_party/mojo/src/mojo/public/cpp/bindings/tests/map_unittest.cc View 2 chunks +1 line, -8 lines 0 comments Download
M third_party/mojo/src/mojo/public/cpp/bindings/tests/request_response_unittest.cc View 2 chunks +4 lines, -4 lines 0 comments Download
M third_party/mojo/src/mojo/public/cpp/bindings/tests/router_unittest.cc View 3 chunks +4 lines, -5 lines 0 comments Download
M third_party/mojo/src/mojo/public/cpp/bindings/tests/sample_service_unittest.cc View 2 chunks +1 line, -12 lines 0 comments Download
M third_party/mojo/src/mojo/public/cpp/bindings/tests/serialization_warning_unittest.cc View 2 chunks +0 lines, -2 lines 0 comments Download
M third_party/mojo/src/mojo/public/cpp/bindings/tests/struct_unittest.cc View 2 chunks +1 line, -8 lines 0 comments Download
M third_party/mojo/src/mojo/public/cpp/bindings/tests/type_conversion_unittest.cc View 1 chunk +0 lines, -1 line 0 comments Download
M third_party/mojo/src/mojo/public/cpp/bindings/tests/union_unittest.cc View 37 chunks +6 lines, -43 lines 0 comments Download
M third_party/mojo/src/mojo/public/cpp/bindings/tests/validation_unittest.cc View 4 chunks +7 lines, -11 lines 0 comments Download
M third_party/mojo/src/mojo/public/mojo_sdk.gni View 1 chunk +3 lines, -1 line 0 comments Download

Messages

Total messages: 5 (0 generated)
nasko
Created Reland of Change mojo_public_bindings_unittests to use chromium environment.
5 years, 1 month ago (2015-11-05 23:58:18 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1407273010/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1407273010/1
5 years, 1 month ago (2015-11-05 23:59:45 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1407273010/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1407273010/1
5 years, 1 month ago (2015-11-06 00:05:13 UTC) #3
commit-bot: I haz the power
Committed patchset #1 (id:1)
5 years, 1 month ago (2015-11-06 00:09:38 UTC) #4
commit-bot: I haz the power
5 years, 1 month ago (2015-11-06 00:10:31 UTC) #5
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/e33507ce46666657829b8fc87daa196797b1a774
Cr-Commit-Position: refs/heads/master@{#358189}

Powered by Google App Engine
This is Rietveld 408576698