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

Issue 8622007: Fix testSaveDangerousFile (Closed)

Created:
9 years, 1 month ago by Nirnimesh
Modified:
9 years, 1 month ago
CC:
chromium-reviews, John Grabowski, anantha, dyu1, Paweł Hajdan Jr., dennis_jeffrey
Visibility:
Public.

Description

Fix testSaveDangerousFile Use dangerous.com on windows. BUG=104886 TEST=pyauto test testSaveDangerousFile R=rdsmith@chromium.org Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=111201

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, --1 lines) Patch
A chrome/test/data/downloads/dangerous/dangerous.com View 0 chunks +-1 lines, --1 lines 0 comments Download
M chrome/test/functional/downloads.py View 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Nirnimesh
9 years, 1 month ago (2011-11-22 01:45:36 UTC) #1
Randy Smith (Not in Mondays)
I'd love to understand why .jar isn't working on Windows. I'm a big believer in ...
9 years, 1 month ago (2011-11-22 02:12:08 UTC) #2
Nirnimesh
9 years, 1 month ago (2011-11-22 08:29:01 UTC) #3
On 2011/11/22 02:12:08, rdsmith wrote:
> I'd love to understand why .jar isn't working on Windows.  I'm a big believer
in
> "if you don't understand it, it's dangerous".  Having said that, LGTM.

That was puzzling to me too and I could not repro it manually; but through
automation I'd get 0bytes reliably. :(

Powered by Google App Engine
This is Rietveld 408576698