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

Issue 13825012: url: Copy 'googleurl_unittests' target from 'url_unittests'. Third attempt. (Closed)

Created:
7 years, 8 months ago by tfarina
Modified:
7 years, 7 months ago
Reviewers:
stuartmorgan, brettw
CC:
chromium-reviews, Nico, lliabraa
Visibility:
Public.

Description

url: Copy 'googleurl_unittests' target from 'url_unittests'. Third attempt. To not break the buildbot, this creates a fake target that copies url_unittests to googleurl_unittests as temporary scaffolding. BUG=229660 R=brettw@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=196507

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+87 lines, -59 lines) Patch
M build/all.gyp View 11 chunks +18 lines, -18 lines 0 comments Download
M build/temp_gyp/googleurl.gyp View 1 chunk +0 lines, -34 lines 0 comments Download
M chrome/chrome_tests.gypi View 1 chunk +7 lines, -7 lines 0 comments Download
M url/url.gyp View 1 chunk +62 lines, -0 lines 0 comments Download

Messages

Total messages: 10 (0 generated)
tfarina
Bots seems to be happy with this now. Thanks Nico for the hard work on ...
7 years, 8 months ago (2013-04-24 02:23:08 UTC) #1
tfarina
Brett, ping?
7 years, 8 months ago (2013-04-25 11:26:03 UTC) #2
brettw
lgtm
7 years, 8 months ago (2013-04-25 18:01:06 UTC) #3
tfarina
Committed patchset #1 manually as r196507 (presubmit successful).
7 years, 8 months ago (2013-04-25 21:42:01 UTC) #4
stuartmorgan
Please don't add iOS-only special-casing without having anyone familiar with iOS on the review. Anyone ...
7 years, 7 months ago (2013-05-02 17:05:10 UTC) #5
tfarina
On 2013/05/02 17:05:10, stuartmorgan wrote: > Please don't add iOS-only special-casing without having anyone familiar ...
7 years, 7 months ago (2013-05-02 17:45:59 UTC) #6
iannucci1
@stuartmorgan, do you know how we could get this working correctly (or someone else with ...
7 years, 7 months ago (2013-05-02 22:31:00 UTC) #7
iannucci1
On 2013/05/02 22:31:00, iannucci1 wrote: > @stuartmorgan, do you know how we could get this ...
7 years, 7 months ago (2013-05-02 22:32:02 UTC) #8
stuartmorgan
On 2013/05/02 17:45:59, tfarina wrote: > ios_dbg_simulator and ios_rel_device are green. Doesn't that cover iOS ...
7 years, 7 months ago (2013-05-03 06:58:40 UTC) #9
stuartmorgan
7 years, 7 months ago (2013-05-03 06:59:13 UTC) #10
Message was sent while issue was closed.
On 2013/05/02 22:32:02, iannucci1 wrote:
> On 2013/05/02 22:31:00, iannucci1 wrote:
> > @stuartmorgan, do you know how we could get this working correctly (or
someone
> > else with a ready knowledge of the ios who could help out)?
> 
> s/the ios/the ios build/

Either copy the whole .app (we build it as an app for a reason), or just have
the bots run url_unittests instead.

Powered by Google App Engine
This is Rietveld 408576698