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

Issue 1084423003: Merge 193840 "Service Worker: Add a test for error messages from..." (Closed)

Created:
5 years, 8 months ago by falken
Modified:
5 years, 8 months ago
Reviewers:
falken
CC:
blink-reviews, michaeln, jsbell+serviceworker_chromium.org, kenjibaheux+watch_chromium.org, tzik, serviceworker-reviews, nhiroki, kinuko+serviceworker, horo+watch_chromium.org
Project:
blink
Visibility:
Public.

Description

Merge 193840 "Service Worker: Add a test for error messages from..." > Service Worker: Add a test for error messages from register() > > This tests existing behavior as well as some added in > https://codereview.chromium.org/1063823005 > > BUG=448003, 436928, 444296 > > Review URL: https://codereview.chromium.org/1058323004 TBR=falken@chromium.org Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=193932

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+10 lines, -4 lines) Patch
A + LayoutTests/http/tests/serviceworker/chromium/register-error-messages.html View 0 chunks +-1 lines, --1 lines 0 comments Download
M LayoutTests/http/tests/serviceworker/registration.html View 1 chunk +12 lines, -3 lines 0 comments Download
A + LayoutTests/http/tests/serviceworker/resources/mime-type-worker.php View 0 chunks +-1 lines, --1 lines 0 comments Download
D LayoutTests/http/tests/serviceworker/resources/plain-text-worker.php View 1 chunk +0 lines, -3 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
falken
5 years, 8 months ago (2015-04-17 10:00:25 UTC) #1
falken
5 years, 8 months ago (2015-04-17 10:00:38 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as r193932.

Powered by Google App Engine
This is Rietveld 408576698