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

Issue 10916010: Relanding "views: Add a gyp file to build webview.". Try 2. (Closed)

Created:
8 years, 3 months ago by tfarina
Modified:
8 years, 3 months ago
CC:
chromium-reviews
Visibility:
Public.

Description

Relanding "views: Add a gyp file to build webview.". Try 2. This reverts commit d2287c4694d749d9e0c4095d7b004beee7b27e86. BUG=105557 TBR=ben@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=154621

Patch Set 1 #

Patch Set 2 : does this fix win trybot? #

Patch Set 3 : exports #

Patch Set 4 : no export #

Patch Set 5 : no need to export views_delegate #

Patch Set 6 : does this fix win trybot?? #

Patch Set 7 : webview_export.h #

Patch Set 8 : revert the hacks #

Patch Set 9 : WEBVIEW_EXPORT #

Patch Set 10 : do not compile webview.cc in both views and webview targets #

Patch Set 11 : add skia dep? #

Patch Set 12 : more fixes #

Patch Set 13 : webview depens on web_dialogs #

Patch Set 14 : chromeos fixes #

Unified diffs Side-by-side diffs Delta from patch set Stats (+88 lines, -15 lines) Patch
M ash/ash.gyp View 1 2 3 4 5 6 7 8 9 1 chunk +1 line, -0 lines 0 comments Download
M build/all.gyp View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/chrome_browser.gypi View 1 2 3 4 5 6 7 8 9 10 11 12 13 2 chunks +3 lines, -0 lines 0 comments Download
M chrome/chrome_browser_chromeos.gypi View 1 2 3 4 5 6 7 8 9 10 11 12 13 1 chunk +1 line, -0 lines 0 comments Download
M content/content_shell.gypi View 1 2 3 4 5 6 7 8 9 1 chunk +1 line, -0 lines 0 comments Download
M ui/views/controls/webview/web_dialog_view.h View 1 2 3 4 5 6 7 8 9 10 11 2 chunks +5 lines, -5 lines 0 comments Download
M ui/views/controls/webview/webview.h View 1 2 3 4 5 6 7 8 2 chunks +5 lines, -4 lines 0 comments Download
A ui/views/controls/webview/webview.gyp View 1 2 3 4 5 6 7 8 9 10 11 12 1 chunk +36 lines, -0 lines 0 comments Download
A ui/views/controls/webview/webview_export.h View 1 2 3 4 5 6 1 chunk +32 lines, -0 lines 0 comments Download
M ui/views/views.gyp View 1 2 3 4 5 6 7 8 9 10 11 12 4 chunks +2 lines, -5 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
tfarina
Hi Ben, could you review this to me? The errors that patch set 6 should ...
8 years, 3 months ago (2012-08-31 16:00:49 UTC) #1
tfarina
On 2012/08/31 16:00:49, tfarina wrote: > Hi Ben, could you review this to me? > ...
8 years, 3 months ago (2012-08-31 17:12:43 UTC) #2
tfarina
Issues fixed! Landing...
8 years, 3 months ago (2012-09-01 18:47:27 UTC) #3
Ben Goodger (Google)
8 years, 3 months ago (2012-09-04 15:26:49 UTC) #4
lgtm

Powered by Google App Engine
This is Rietveld 408576698