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

Issue 13928020: Update test plugins to be fat. (Closed)

Created:
7 years, 8 months ago by Avi (use Gerrit)
Modified:
7 years, 8 months ago
CC:
chromium-reviews, Aaron Boodman, chromium-apps-reviews_chromium.org
Visibility:
Public.

Description

Update test plugins to be fat. BUG=229514 TEST=ExtensionBrowserTest.Plugin* tests pass in 64 bit Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=193245 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=193424

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, --1 lines) Patch
M chrome/test/data/extensions/uitest/plugins/README View 1 chunk +10 lines, -1 line 0 comments Download
M chrome/test/data/extensions/uitest/plugins/plugin.plugin/Contents/MacOS/TestNetscapePlugIn View 0 chunks +-1 lines, --1 lines 0 comments Download
M chrome/test/data/extensions/uitest/plugins_private/plugin.plugin/Contents/MacOS/TestNetscapePlugIn View 0 chunks +-1 lines, --1 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
Avi (use Gerrit)
Matt: review Mark: FYI
7 years, 8 months ago (2013-04-09 19:08:24 UTC) #1
Mark Mentovai
LGTM, although I’m not sure why this is a checked-in binary instead of something that ...
7 years, 8 months ago (2013-04-09 19:14:51 UTC) #2
Matt Perry
lgtm
7 years, 8 months ago (2013-04-09 19:28:18 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/avi@chromium.org/13928020/1
7 years, 8 months ago (2013-04-09 19:29:05 UTC) #4
commit-bot: I haz the power
Change committed as 193245
7 years, 8 months ago (2013-04-09 22:33:12 UTC) #5
Avi (use Gerrit)
7 years, 8 months ago (2013-04-10 18:38:49 UTC) #6
Message was sent while issue was closed.
Committed patchset #1 manually as r193424 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698