85934 – [WebSocket] Must ignore incoming message in CLOSING state

RESOLVED FIXED 85934

[WebSocket] Must ignore incoming message in CLOSING state

https://bugs.webkit.org/show_bug.cgi?id=85934

Summary [WebSocket] Must ignore incoming message in CLOSING state

Takashi Toyoshima

Reported 2012-05-08 17:34:46 PDT

Current implementation allow to receive incoming message in CLOSING state. It means onmessage envent might happen after user calls close(). This behavior is right in old API spec; http://www.w3.org/TR/2011/WD-websockets-20110929/ But, latest spec is changed as http://www.w3.org/TR/2011/CR-websockets-20111208/ FYI - discussion to make this change; https://www.w3.org/Bugs/Public/show_bug.cgi?id=14474 - change; http://html5.org/tools/web-apps-tracker?from=6791&to=6792

Attachments
Patch (10.12 KB, patch) 2013-03-27 11:17 PDT, Lamarque V. Souza	no flags	Details Formatted Diff Diff
Archive of layout-test-results from gce-cr-linux-05 for chromium-linux-x86_64 (1.01 MB, application/zip) 2013-03-27 12:36 PDT, WebKit Review Bot	no flags	Details
Patch (10.12 KB, patch) 2013-03-27 13:13 PDT, Lamarque V. Souza	no flags	Details Formatted Diff Diff
Archive of layout-test-results from webkit-ews-12 for mac-mountainlion-wk2 (1.38 MB, application/zip) 2013-03-27 14:55 PDT, Build Bot	no flags	Details
Archive of layout-test-results from webkit-ews-07 for mac-future (744.39 KB, application/zip) 2013-03-28 14:50 PDT, Build Bot	no flags	Details
Archive of layout-test-results from webkit-ews-03 for mac-future (643.63 KB, application/zip) 2013-03-28 15:24 PDT, Build Bot	no flags	Details
Patch (10.66 KB, patch) 2013-03-28 20:48 PDT, Lamarque V. Souza	no flags	Details Formatted Diff Diff
Patch (6.68 KB, patch) 2013-04-02 15:42 PDT, Lamarque V. Souza	no flags	Details Formatted Diff Diff
Archive of layout-test-results from gce-cr-linux-02 for chromium-linux-x86_64 (1.04 MB, application/zip) 2013-04-02 17:06 PDT, WebKit Review Bot	no flags	Details
Patch (8.48 KB, patch) 2013-04-05 15:07 PDT, Lamarque V. Souza	tkent: review+ rniwa: commit-queue-	Details Formatted Diff Diff
Patch (8.43 KB, patch) 2013-04-08 11:55 PDT, Lamarque V. Souza	commit-queue: commit-queue-	Details Formatted Diff Diff
Patch (8.45 KB, patch) 2013-04-08 16:02 PDT, Lamarque V. Souza	commit-queue: commit-queue-	Details Formatted Diff Diff
Patch (8.41 KB, patch) 2013-04-08 20:35 PDT, Lamarque V. Souza	commit-queue: commit-queue-	Details Formatted Diff Diff
Patch (8.33 KB, patch) 2013-04-09 05:36 PDT, Lamarque V. Souza	no flags	Details Formatted Diff Diff
Show Obsolete (4) View All Add attachment proposed patch, testcase, etc.

Lamarque V. Souza

Comment 1 2013-03-27 11:17:38 PDT

Created attachment 195363 [details] Patch Proposed patch

WebKit Review Bot

Comment 2 2013-03-27 12:36:04 PDT

Comment on attachment 195363 [details] Patch Attachment 195363 [details] did not pass chromium-ews (chromium-xvfb): Output: http://webkit-commit-queue.appspot.com/results/17314421 New failing tests: http/tests/websocket/tests/hybi/client-close.html

WebKit Review Bot

Comment 3 2013-03-27 12:36:07 PDT

Created attachment 195380 [details] Archive of layout-test-results from gce-cr-linux-05 for chromium-linux-x86_64 The attached test failures were seen while running run-webkit-tests on the chromium-ews. Bot: gce-cr-linux-05 Port: chromium-linux-x86_64 Platform: Linux-3.3.8-gcg-201212281604-x86_64-with-GCEL-10.04-gcel_10.04

Lamarque V. Souza

Comment 4 2013-03-27 13:13:36 PDT

Created attachment 195383 [details] Patch Rebaseline client-close.html because of change in the description field.

Build Bot

Comment 5 2013-03-27 14:55:48 PDT

Comment on attachment 195383 [details] Patch Attachment 195383 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-commit-queue.appspot.com/results/17312553 New failing tests: http/tests/websocket/tests/hybi/client-close.html

Build Bot

Comment 6 2013-03-27 14:55:49 PDT

Created attachment 195402 [details] Archive of layout-test-results from webkit-ews-12 for mac-mountainlion-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: webkit-ews-12 Port: mac-mountainlion-wk2 Platform: Mac OS X 10.8.2

Build Bot

Comment 7 2013-03-28 14:50:03 PDT

Comment on attachment 195383 [details] Patch Attachment 195383 [details] did not pass mac-ews (mac): Output: http://webkit-commit-queue.appspot.com/results/17313356

Build Bot

Comment 8 2013-03-28 14:50:05 PDT

Created attachment 195650 [details] Archive of layout-test-results from webkit-ews-07 for mac-future The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: webkit-ews-07 Port: mac-future Platform: Mac OS X 10.8.2

Build Bot

Comment 9 2013-03-28 15:24:50 PDT

Comment on attachment 195383 [details] Patch Attachment 195383 [details] did not pass mac-ews (mac): Output: http://webkit-commit-queue.appspot.com/results/17309556 New failing tests: http/tests/websocket/tests/hybi/client-close.html

Build Bot

Comment 10 2013-03-28 15:24:53 PDT

Created attachment 195657 [details] Archive of layout-test-results from webkit-ews-03 for mac-future The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: webkit-ews-03 Port: mac-future Platform: Mac OS X 10.8.2

Lamarque V. Souza

Comment 11 2013-03-28 20:48:46 PDT

Created attachment 195693 [details] Patch Split close-client.html into two unit tests.

Takashi Toyoshima

Comment 12 2013-03-29 09:40:26 PDT

Comment on attachment 195693 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=195693&action=review Hi Lamarque, Thank you for sending a patch to fix this. Actually, I'm not an authoritative WebKit reviewer, but I'd say some comments on this patch as a WebKit committer working on WebSocket. > LayoutTests/http/tests/websocket/tests/hybi/client-close-ignore-incoming.html:1 > +<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML//EN"> Old tests still uses this document type, but we adopt HTML5 doctype <!DOCTYPE html> for new tests. It's better to fix other files which you touched in this change. > LayoutTests/http/tests/websocket/tests/hybi/client-close-ignore-incoming.html:4 > +<script src="../../../../js-test-resources/js-test-pre.js"></script> For the same reason, please don't use relative path, but absolute path here and the end of this html. You may want to see useragent-in-openinghandshake.html as a test which is introduced recently. > LayoutTests/http/tests/websocket/tests/hybi/client-close-ignore-incoming.html:10 > +description("WebSocket: Test client-initiated close. After WebSocket.close() any message from server must be discarded by WebSocket stack according to the new WebSocket API, see http://webkit.org/b/85934."); I think it is not usual to describe webkit bug here. If you want to say something on this test, please write them in ChangeLog. > LayoutTests/http/tests/websocket/tests/hybi/client-close_wsh.py:14 > This test intends to check a client sending close frame itself. So, this change make it meaningless. Please read comments in this test carefully.

Lamarque V. Souza

Comment 13 2013-04-02 13:09:40 PDT

Comment on attachment 195693 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=195693&action=review >> LayoutTests/http/tests/websocket/tests/hybi/client-close-ignore-incoming.html:10 >> +description("WebSocket: Test client-initiated close. After WebSocket.close() any message from server must be discarded by WebSocket stack according to the new WebSocket API, see http://webkit.org/b/85934."); > > I think it is not usual to describe webkit bug here. > If you want to say something on this test, please write them in ChangeLog. Well, I have added the link to the bug that motivated changes before and the other reviewers did not complain about it. The description also says what is the expected behavior for this unit test. In this particular case it is important because we expect a message to be ignored. >> LayoutTests/http/tests/websocket/tests/hybi/client-close_wsh.py:14 >> > > This test intends to check a client sending close frame itself. So, this change make it meaningless. > Please read comments in this test carefully. Yeah, I was in doubt if this change was correct. I think this unit test can be discarded then. After this change there is no way to make the line "msgutil.send_message(request, 'close_frame[:2]=%r' % close_frame[:2])" work since the client will ignore the message. The new unit test I created (client-close-ignore-incomming.html) covers the new behavior. I am also worried about this change. A lot of websocket scripts out there will have to be updated to avoid the same regressions I have found in our unit tests.

Lamarque V. Souza

Comment 14 2013-04-02 15:42:38 PDT

Created attachment 196245 [details] Patch Fix issues reported.

WebKit Review Bot

Comment 15 2013-04-02 17:06:52 PDT

Comment on attachment 196245 [details] Patch Attachment 196245 [details] did not pass chromium-ews (chromium-xvfb): Output: http://webkit-commit-queue.appspot.com/results/17366537 New failing tests: http/tests/websocket/tests/hybi/client-close.html

WebKit Review Bot

Comment 16 2013-04-02 17:06:57 PDT

Created attachment 196255 [details] Archive of layout-test-results from gce-cr-linux-02 for chromium-linux-x86_64 The attached test failures were seen while running run-webkit-tests on the chromium-ews. Bot: gce-cr-linux-02 Port: chromium-linux-x86_64 Platform: Linux-3.3.8-gcg-201212281604-x86_64-with-GCEL-10.04-gcel_10.04

Takashi Toyoshima

Comment 17 2013-04-03 04:16:39 PDT

Comment on attachment 195693 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=195693&action=review >>> LayoutTests/http/tests/websocket/tests/hybi/client-close_wsh.py:14 >>> >> >> This test intends to check a client sending close frame itself. So, this change make it meaningless. >> Please read comments in this test carefully. > > Yeah, I was in doubt if this change was correct. I think this unit test can be discarded then. After this change there is no way to make the line "msgutil.send_message(request, 'close_frame[:2]=%r' % close_frame[:2])" work since the client will ignore the message. The new unit test I created (client-close-ignore-incomming.html) covers the new behavior. I am also worried about this change. A lot of websocket scripts out there will have to be updated to avoid the same regressions I have found in our unit tests. How about sending a close frame response iff close_frame[:2] is expected data. You can check CloseEvent.wasClean to know if the close_frame contains expected data. Or you can embed arbitrary data to reason field of close frame which can be read as CloseEvent.reason in browser. See also close-code-and-reason_wsh.py.

Lamarque V. Souza

Comment 18 2013-04-05 15:07:17 PDT

Created attachment 196687 [details] Patch Use Takashi's suggestion to send message embedded in close frame.

Lamarque V. Souza

Comment 19 2013-04-05 15:08:53 PDT

(In reply to comment #17) > How about sending a close frame response iff close_frame[:2] is expected data. You can check CloseEvent.wasClean to know if the close_frame contains expected data. > Or you can embed arbitrary data to reason field of close frame which can be read as CloseEvent.reason in browser. > See also close-code-and-reason_wsh.py. It works. Thanks for the tip :-)

Takashi Toyoshima

Comment 20 2013-04-05 16:23:25 PDT

Cool. Thanks! This change looks good. Please ask ap@webkit.org for the final review to commit this. After that, I'll try to push this change to Blink, too.

Kent Tamura

Comment 21 2013-04-07 18:21:24 PDT

Comment on attachment 196687 [details] Patch ok

WebKit Commit Bot

Comment 22 2013-04-07 21:35:06 PDT

The commit-queue encountered the following flaky tests while processing attachment 196687 [details]: svg/batik/paints/patternPreserveAspectRatioA.svg bug 114139 (author: zimmermann@kde.org) svg/as-image/img-relative-height.html bug 114140 (author: zimmermann@kde.org) The commit-queue is continuing to process your patch.

WebKit Commit Bot

Comment 23 2013-04-07 21:36:32 PDT

Comment on attachment 196687 [details] Patch Rejecting attachment 196687 [details] from commit-queue. Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-commit-queue.appspot.com', '--bot-id=webkit-cq-03', 'land-attachment', '--force-clean', '--non-interactive', '--parent-command=commit-queue', 196687, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit Last 500 characters of output: fs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc ... Currently at 147890 = dce53a8d75be88d4b224a35bcb48af24022fccf2 r147891 = d4d91ae23cf49fdb88e5502039deeff04627efeb r147892 = 4e3f96534501c6afed9a27a90e39f5df36b51763 r147893 = 11e52cf9f60d507c230704947ae0c2db8e17da9b Done rebuilding .git/svn/refs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc First, rewinding head to replay your work on top of it... Fast-forwarded master to refs/remotes/origin/master. Full output: http://webkit-commit-queue.appspot.com/results/17562005

Bruno Abinader (history only)

Comment 24 2013-04-08 06:31:00 PDT

Comment on attachment 196687 [details] Patch Seems cq bot had some issues, let us try again...

WebKit Commit Bot

Comment 25 2013-04-08 07:10:31 PDT

Comment on attachment 196687 [details] Patch Rejecting attachment 196687 [details] from commit-queue. Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-commit-queue.appspot.com', '--bot-id=webkit-cq-01', 'land-attachment', '--force-clean', '--non-interactive', '--parent-command=commit-queue', 196687, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit Last 500 characters of output: 331ac3fe M LayoutTests :040000 040000 24d9b3d66502bb5ba11668a063affddcac162f42 9f1b470b1b5cf49471fd367583018bd6eb6da966 M Source Current branch master is up to date. ERROR: Not all changes have been committed into SVN, however the committed ones (if any) seem to be successfully integrated into the working tree. Please see the above messages for details. Failed to run "['git', 'svn', 'dcommit']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit Updating OpenSource Current branch master is up to date. Full output: http://webkit-commit-queue.appspot.com/results/17491111

Ryosuke Niwa

Comment 26 2013-04-08 11:44:11 PDT

Comment on attachment 196687 [details] Patch Please re-upload the patch with webkit-patch upload. git diff wouldn't apply.

Lamarque V. Souza

Comment 27 2013-04-08 11:55:00 PDT

Created attachment 196881 [details] Patch Uploading patch again for landing.

WebKit Commit Bot

Comment 28 2013-04-08 13:43:57 PDT

Comment on attachment 196881 [details] Patch Rejecting attachment 196881 [details] from commit-queue. Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-commit-queue.appspot.com', '--bot-id=webkit-cq-03', 'land-attachment', '--force-clean', '--non-interactive', '--parent-command=commit-queue', 196881, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit Last 500 characters of output: /git.webkit.org/WebKit 14ddb34..da4c535 master -> origin/master Partial-rebuilding .git/svn/refs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc ... Currently at 147946 = 14ddb34f2dd43fc51a356679ea878cc05dc02c9c r147947 = da4c535f8abc55a094169db9f19f6c67fd8c89c7 Done rebuilding .git/svn/refs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc First, rewinding head to replay your work on top of it... Fast-forwarded master to refs/remotes/origin/master. Full output: http://webkit-commit-queue.appspot.com/results/17575044

Lamarque V. Souza

Comment 29 2013-04-08 16:02:57 PDT

Created attachment 196966 [details] Patch Uploading patch for landing. I ran git-fsck on my tree it fixed several dangling commits, let's see if it works now.

WebKit Commit Bot

Comment 30 2013-04-08 17:40:48 PDT

Comment on attachment 196966 [details] Patch Rejecting attachment 196966 [details] from commit-queue. Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-commit-queue.appspot.com', '--bot-id=webkit-cq-01', 'land-attachment', '--force-clean', '--non-interactive', '--parent-command=commit-queue', 196966, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit Last 500 characters of output: 495ba2d4a83c234980e8d72e9f1e90440e8 r147964 = 1a247a3ac5dc5f789ee359644edb1c415508ca63 r147965 = ee10e45215b3cbe9629cd0c679f26503590ff3c1 r147966 = 924692f6afc55819809ce99ddbeb7d8758b0caba r147967 = 4ca681abae66e2ce349a6589c55e230212cf5fb4 r147968 = cf28c9c62e25be7b1e23fd9e53550278c6c7b520 Done rebuilding .git/svn/refs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc First, rewinding head to replay your work on top of it... Fast-forwarded master to refs/remotes/origin/master. Full output: http://webkit-commit-queue.appspot.com/results/17636010

Takashi Toyoshima

Comment 31 2013-04-08 18:22:27 PDT

I guess you created this branch from a local trunk which has wrong hash value because of unexpected local changes. Can you take a clean checkout and create a new branch. Then apply this patch to it. After that, webkit-patch will work hopefully.

Lamarque V. Souza

Comment 32 2013-04-08 18:46:10 PDT

(In reply to comment #31) > I guess you created this branch from a local trunk which has wrong hash value because of unexpected local changes. > Can you take a clean checkout and create a new branch. Then apply this patch to it. After that, webkit-patch will work hopefully. Hi, I have never created a branch in my tree and I use git, not subversion. A clean checkout is going to take two or three days due to my slow Internet connection :-/

Takashi Toyoshima

Comment 33 2013-04-08 19:36:29 PDT

Usually we create a local branch for each bug, then make changes in the branch. If you work at a local trunk, please revert all local commits firstly. After that, make it up to date by synching with remote trunk. It must be equivalent of a clean checkout.

Lamarque V. Souza

Comment 34 2013-04-08 20:35:26 PDT

Created attachment 196985 [details] Patch Uploading patch for landing.

WebKit Commit Bot

Comment 35 2013-04-08 21:20:19 PDT

Comment on attachment 196985 [details] Patch Rejecting attachment 196985 [details] from commit-queue. Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-commit-queue.appspot.com', '--bot-id=webkit-cq-02', 'land-attachment', '--force-clean', '--non-interactive', '--parent-command=commit-queue', 196985, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit Last 500 characters of output: 8bae380b M LayoutTests :040000 040000 159beb0a331e0fec27542bd290529f911c6ffcdc 772fa7104dcdff27ec119070e3be30443252aca6 M Source Current branch master is up to date. ERROR: Not all changes have been committed into SVN, however the committed ones (if any) seem to be successfully integrated into the working tree. Please see the above messages for details. Failed to run "['git', 'svn', 'dcommit']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit Updating OpenSource Current branch master is up to date. Full output: http://webkit-commit-queue.appspot.com/results/17696014

Kent Tamura

Comment 36 2013-04-08 21:56:41 PDT

Comment on attachment 196985 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=196985&action=review > LayoutTests/ChangeLog:7 > + Need a short description (OOPS!). > + Need the bug URL (OOPS!). Please remove these lines.

Lamarque V. Souza

Comment 37 2013-04-09 05:36:15 PDT

Created attachment 197034 [details] Patch Fixed problem in LayoutTests/ChangeLog.

WebKit Commit Bot

Comment 38 2013-04-09 06:42:45 PDT

The commit-queue encountered the following flaky tests while processing attachment 197034 [details]: svg/dom/SVGScriptElement/script-load-and-error-events.svg bug 114279 (authors: fmalita@chromium.org, yurys@chromium.org, and zimmermann@kde.org) svg/animations/smil-leak-elements.svg bug 114280 (authors: fmalita@chromium.org and timothy_horton@apple.com) svg/animations/smil-leak-dynamically-added-element-instances.svg bug 114281 (authors: fmalita@chromium.org and timothy_horton@apple.com) transitions/color-transition-rounding.html bug 114182 (author: simon.fraser@apple.com) transitions/cubic-bezier-overflow-svg-length.html bug 114183 (author: peter@chromium.org) transitions/interrupt-zero-duration.html bug 114184 (authors: cmarrin@apple.com, rniwa@webkit.org, and simon.fraser@apple.com) transitions/multiple-background-transitions.html bug 114185 (author: simon.fraser@apple.com) transitions/cubic-bezier-overflow-color.html bug 114186 (author: peter@chromium.org) transitions/multiple-shadow-transitions.html bug 114187 (author: simon.fraser@apple.com) transitions/mismatched-shadow-transitions.html bug 114188 (author: simon.fraser@apple.com) transitions/color-transition-all.html bug 114189 (authors: ossy@webkit.org and simon.fraser@apple.com) transitions/cubic-bezier-overflow-shadow.html bug 114191 (author: peter@chromium.org) transitions/min-max-width-height-transitions.html bug 114192 (author: simon.fraser@apple.com) transitions/cancel-transition.html bug 114193 (authors: ojan@chromium.org, rniwa@webkit.org, and simon.fraser@apple.com) transitions/border-radius-transition.html bug 114194 (author: simon.fraser@apple.com) transitions/flex-transitions.html bug 114195 (author: tony@chromium.org) transitions/mixed-type.html bug 114196 (author: mikelawther@chromium.org) transitions/multiple-mask-transitions.html bug 114197 (author: simon.fraser@apple.com) transitions/color-transition-premultiplied.html bug 114198 (author: simon.fraser@apple.com) transitions/mismatched-shadow-styles.html bug 114199 (author: simon.fraser@apple.com) transitions/mask-transitions.html bug 114200 (authors: ojan@chromium.org, oliver@apple.com, and simon.fraser@apple.com) transitions/cubic-bezier-overflow-length.html bug 114201 (author: peter@chromium.org) transitions/multiple-background-size-transitions.html bug 114202 (authors: mitz@webkit.org and simon.fraser@apple.com) transitions/clip-transition.html bug 114203 (authors: dglazkov@chromium.org and simon.fraser@apple.com) transitions/cubic-bezier-overflow-transform.html bug 114204 (author: peter@chromium.org) transitions/interrupted-accelerated-transition.html bug 56242 (authors: rniwa@webkit.org, simon.fraser@apple.com, and tonyg@chromium.org) transitions/background-transitions.html bug 114206 (author: simon.fraser@apple.com) http/tests/security/mixedContent/redirect-https-to-http-iframe-in-main-frame.html bug 114208 (authors: abarth@webkit.org and rniwa@webkit.org) fast/loader/javascript-url-in-object.html bug 114210 (authors: rniwa@webkit.org and sam@webkit.org) platform/mac/editing/deleting/deletionUI-single-instance.html bug 114181 (author: rniwa@webkit.org) The commit-queue is continuing to process your patch.

WebKit Commit Bot

Comment 39 2013-04-09 06:43:43 PDT

Comment on attachment 197034 [details] Patch Clearing flags on attachment: 197034 Committed r148019: <http://trac.webkit.org/changeset/148019>

Lamarque V. Souza

Comment 40 2013-04-09 09:06:39 PDT

(In reply to comment #36) > (From update of attachment 196985 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=196985&action=review > > > LayoutTests/ChangeLog:7 > > + Need a short description (OOPS!). > > + Need the bug URL (OOPS!). > > Please remove these lines. Thanks for the tip. I am wondering why the style bot did not catch that problem.

Alexey Proskuryakov

Comment 41 2013-04-10 23:24:35 PDT

This got landed in <http://trac.webkit.org/changeset/148019>. Commit queue didn't mark the bug resolved because there was an older patch with r+ that wasn't obsolete.

Note You need to log in before you can comment on or make changes to this bug.

Status RESOLVED

Resolution FIXED

Priority P2

Severity Normal

Classification Unclassified

Version 528+ (Nightly build)

Hardware All

OS All

Product WebKit

Component WebCore Misc.

Assignee

Takashi Toyoshima

Reported

2012-05-08 17:34 PDT

Modified

2013-04-10 23:24 PDT History

CC List

10 users Show

URL

Keywords

Depends on

Blocks