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

Issue 8570024: Enable content component (again!). (Closed)

Created:
9 years, 1 month ago by Dirk Pranke
Modified:
9 years, 1 month ago
Reviewers:
jam
CC:
chromium-reviews, michaeln, cbentzel+watch_chromium.org, jam, dpranke-watch+content_chromium.org, joi+watch-content_chromium.org, darin-cc_chromium.org, tommi (sloooow) - chröme
Visibility:
Public.

Description

Enable content component (again!). This version fixes a problem with the windows incremental linking build. We can't incrementally link chrome when content is being built as a DLL because chrome links in webkit_glue and webkit_glue depends on symbols defined in content. We can remove this when we fix glue. R=jam@chromium.org BUG=90442, 98755 . Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=110360

Patch Set 1 #

Patch Set 2 : compile content statically if we are linking chrome.dll incrementally #

Patch Set 3 : rebase to HEAD (r110244) #

Patch Set 4 : export BrokerDispatcherWrapper, WorkerProcessHost::TerminateWorker #

Patch Set 5 : export ShowDesktopNotificationHostMsgParams #

Patch Set 6 : tweak audio exports #

Unified diffs Side-by-side diffs Delta from patch set Stats (+76 lines, -21 lines) Patch
M build/common.gypi View 1 2 1 chunk +3 lines, -0 lines 0 comments Download
M chrome/chrome_exe.gypi View 2 chunks +4 lines, -0 lines 0 comments Download
M chrome/chrome_installer_util.gypi View 1 3 chunks +17 lines, -2 lines 0 comments Download
M chrome/chrome_tests.gypi View 1 2 3 4 5 2 chunks +20 lines, -0 lines 0 comments Download
M chrome/nacl.gypi View 1 2 1 chunk +1 line, -0 lines 0 comments Download
M content/browser/net/url_request_abort_on_end_job.h View 2 chunks +1 line, -2 lines 0 comments Download
M content/browser/worker_host/worker_process_host.h View 1 2 3 2 chunks +2 lines, -1 line 0 comments Download
M content/common/content_export.h View 1 chunk +1 line, -1 line 0 comments Download
M content/content.gyp View 1 2 2 chunks +8 lines, -4 lines 0 comments Download
M content/public/common/show_desktop_notification_params.h View 1 2 3 4 1 chunk +2 lines, -1 line 0 comments Download
M content/renderer/media/audio_input_device.h View 1 2 3 4 5 2 chunks +2 lines, -2 lines 0 comments Download
M content/renderer/media/webrtc_audio_device_impl.h View 1 2 3 4 5 2 chunks +2 lines, -1 line 0 comments Download
M content/renderer/pepper_plugin_delegate_impl.h View 1 2 3 1 chunk +1 line, -1 line 0 comments Download
M webkit/appcache/appcache_export.h View 1 chunk +1 line, -2 lines 0 comments Download
M webkit/appcache/webkit_appcache.gypi View 1 2 3 4 5 3 chunks +11 lines, -4 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
Dirk Pranke
9 years, 1 month ago (2011-11-16 03:04:26 UTC) #1
jam
Can you explain, at least in the cl description, why content has to be built ...
9 years, 1 month ago (2011-11-16 16:51:45 UTC) #2
tommi (sloooow) - chröme
Won't this mean that those that use shared+incremental linking can unknowingly break the shared build? ...
9 years, 1 month ago (2011-11-16 17:01:27 UTC) #3
Dirk Pranke
On Wed, Nov 16, 2011 at 8:51 AM, <jam@chromium.org> wrote: > Can you explain, at ...
9 years, 1 month ago (2011-11-16 17:58:35 UTC) #4
jam
9 years, 1 month ago (2011-11-16 20:43:51 UTC) #5
lgtm

I still don't understand why content dll breaks incremental linking, but I defer
to you.

Powered by Google App Engine
This is Rietveld 408576698