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

Issue 12891005: Revert 189990 - re-re-land of "Refactor libjingle and libpeerconn..." (Closed)

Created:
7 years, 9 months ago by tommi (sloooow) - chröme
Modified:
7 years, 9 months ago
CC:
chromium-reviews
Visibility:
Public.

Description

Revert 189990 - re-re-land of "Refactor libjingle and libpeerconn..." The previous attempt failed because libjingle hadn't been rolled yet. Still waiting on trybots a day later. > > > > Refactor libjingle and libpeerconnection targets. > > > > I'm moving most of the source files back to the libjingle target since the current splitting doesn't work for all targets that currently rely on libjingle. The webrtc core libs are still only in the libpeerconnection target, and require the enable_webrtc gyp variable to be be set. > > > > > > > > I'm also adding a gyp variable to the libjingle target to support dependency injection as is supported for libpeerconnection. > > > > > > > > TBR=darin > > > > > > > > Review URL: https://codereview.chromium.org/12457027 > > > > > > TBR=tommi@chromium.org > > > Review URL: https://codereview.chromium.org/12662028 TBR=tommi@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=189995

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+96 lines, -74 lines) Patch
M trunk/src/chrome/chrome_browser.gypi View 1 chunk +5 lines, -0 lines 0 comments Download
M trunk/src/content/content_tests.gypi View 1 chunk +1 line, -0 lines 0 comments Download
M trunk/src/third_party/libjingle/libjingle.gyp View 6 chunks +90 lines, -74 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
tommi (sloooow) - chröme
7 years, 9 months ago (2013-03-23 08:19:57 UTC) #1
tommi (sloooow) - chröme
7 years, 9 months ago (2013-03-23 08:35:58 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 manually as r189995.

Powered by Google App Engine
This is Rietveld 408576698