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

Issue 7497024: Add a test for the workaround for bug 89967. (Closed)

Created:
9 years, 5 months ago by Mihai Parparita -not on Chrome
Modified:
9 years, 5 months ago
Reviewers:
abarth-chromium
CC:
chromium-reviews, Aaron Boodman, Erik does not do reviews, Paweł Hajdan Jr.
Visibility:
Public.

Description

Add a test for the workaround for bug 89967. BUG=89967 TEST=no R=abarth@chromium.org Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=94234

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+60 lines, -0 lines) Patch
M chrome/browser/extensions/window_open_apitest.cc View 1 chunk +4 lines, -0 lines 0 comments Download
A chrome/test/data/extensions/api_test/window_open/opener/check-opener.html View 1 chunk +3 lines, -0 lines 0 comments Download
A chrome/test/data/extensions/api_test/window_open/opener/manifest.json View 1 chunk +7 lines, -0 lines 0 comments Download
A chrome/test/data/extensions/api_test/window_open/opener/set-name.html View 1 chunk +4 lines, -0 lines 0 comments Download
A chrome/test/data/extensions/api_test/window_open/opener/test.html View 1 chunk +42 lines, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Mihai Parparita -not on Chrome
9 years, 5 months ago (2011-07-27 00:58:26 UTC) #1
abarth-chromium
9 years, 5 months ago (2011-07-27 01:53:14 UTC) #2
LGTM.  Thanks!

Powered by Google App Engine
This is Rietveld 408576698