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

Issue 16298004: Stop running worker layout tests as browser tests (Closed)

Created:
7 years, 6 months ago by jochen (gone - plz use gerrit)
Modified:
7 years, 6 months ago
Reviewers:
jam, Dmitry Titov
CC:
chromium-reviews, joi+watch-content_chromium.org, darin-cc_chromium.org, jam
Visibility:
Public.

Description

Patch Set 1 #

Patch Set 2 : updates #

Patch Set 3 : rebase #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -219 lines) Patch
M build/android/pylib/gtest/filter/content_browsertests_disabled View 1 2 1 chunk +0 lines, -14 lines 0 comments Download
M content/browser/worker_host/test/worker_browsertest.cc View 1 2 1 chunk +1 line, -205 lines 0 comments Download

Messages

Total messages: 10 (0 generated)
jochen (gone - plz use gerrit)
7 years, 6 months ago (2013-06-06 20:40:43 UTC) #1
jam
lgtm, please mark all the associated bugs in this file as WontFix.
7 years, 6 months ago (2013-06-07 00:51:52 UTC) #2
Dmitry Titov
Do we run these tests already in content_shell? Seems they should be first enabled there, ...
7 years, 6 months ago (2013-06-07 01:53:04 UTC) #3
jochen (gone - plz use gerrit)
On 2013/06/07 01:53:04, Dmitry Titov wrote: > Do we run these tests already in content_shell? ...
7 years, 6 months ago (2013-06-07 10:04:26 UTC) #4
Dmitry Titov
On 2013/06/07 10:04:26, jochen wrote: > On 2013/06/07 01:53:04, Dmitry Titov wrote: > > Do ...
7 years, 6 months ago (2013-06-07 17:14:59 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/jochen@chromium.org/16298004/3001
7 years, 6 months ago (2013-06-07 20:21:08 UTC) #6
commit-bot: I haz the power
Failed to apply patch for build/android/pylib/gtest/filter/content_browsertests_disabled: While running patch -p1 --forward --force --no-backup-if-mismatch; patching file ...
7 years, 6 months ago (2013-06-07 20:21:10 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/jochen@chromium.org/16298004/14001
7 years, 6 months ago (2013-06-08 10:13:46 UTC) #8
commit-bot: I haz the power
Retried try job too often on win7_aura for step(s) browser_tests http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win7_aura&number=47844
7 years, 6 months ago (2013-06-08 11:23:03 UTC) #9
jochen (gone - plz use gerrit)
7 years, 6 months ago (2013-06-10 08:34:45 UTC) #10
Message was sent while issue was closed.
Committed patchset #3 manually as r205164 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698