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

Issue 216793008: Revert 260175 "Fixup mojo dylibs to support mojo in content" (Closed)

Created:
6 years, 9 months ago by DaveMoore
Modified:
6 years, 9 months ago
Reviewers:
DaveMoore
CC:
chromium-reviews
Visibility:
Public.

Description

Revert 260175 "Fixup mojo dylibs to support mojo in content" sky is about to check something in that will interact poorly with this and it'll be easier to apply the necessary changes after that. > Fixup mojo dylibs to support mojo in content > > Second try after revert of > https://codereview.chromium.org/211503005/ > > The isolates failed because libmojo_system.dylib wasn't getting copied. > > BUG= > TBR=mark@chromium.org > > Review URL: https://codereview.chromium.org/217003003 TBR=davemoore@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=260181

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+27 lines, -77 lines) Patch
M trunk/src/build/common.gypi View 1 chunk +1 line, -1 line 0 comments Download
M trunk/src/chrome/chrome.isolate View 1 chunk +0 lines, -1 line 0 comments Download
M trunk/src/chrome/chrome_dll_bundle.gypi View 2 chunks +1 line, -30 lines 0 comments Download
M trunk/src/chrome/tools/build/mac/dump_product_syms View 1 chunk +0 lines, -1 line 0 comments Download
M trunk/src/content/content_shell.gypi View 1 chunk +1 line, -30 lines 0 comments Download
M trunk/src/mojo/mojo_public.gypi View 3 chunks +24 lines, -14 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
DaveMoore
6 years, 9 months ago (2014-03-28 17:19:24 UTC) #1
DaveMoore
6 years, 9 months ago (2014-03-28 17:19:31 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 manually as r260181.

Powered by Google App Engine
This is Rietveld 408576698