[webkit-changes] [191622] trunk

2015-10-27 Thread calvaris 
Title: [191622] trunk








Revision 191622
Author calva...@igalia.com
Date 2015-10-27 04:36:56 -0700 (Tue, 27 Oct 2015)


Log Message
[Streams API] Add close method to writable stream
https://bugs.webkit.org/show_bug.cgi?id=150560

Reviewed by Darin Adler.

Source/WebCore:

Added the close method which requires three additional writable stream internal functions plus a queuing
function to retrieve a value from the queue according to the spec.

Current test set suffices. Expectations were updated accordingly.

* Modules/streams/StreamInternals.js:
(peekQueueValue):
* Modules/streams/WritableStream.js:
(close):
* Modules/streams/WritableStreamInternals.js:
(callOrScheduleWritableStreamAdvanceQueue):
(writableStreamAdvanceQueue):
(closeWritableStream): Added as per spec.

LayoutTests:

New expectations after adding close method to writable stream.

Two tests were also flagged because of bug 147933 that prevents them from running properly in certain
conditions. So far this only happened in Mac, hence they were flagged only for it.

* platform/mac/TestExpectations:
* streams/reference-implementation/bad-underlying-sinks-expected.txt:
* streams/reference-implementation/brand-checks-expected.txt:
* streams/reference-implementation/pipe-to-expected.txt:
* streams/reference-implementation/writable-stream-abort-expected.txt:
* streams/reference-implementation/writable-stream-expected.txt:

Modified Paths

trunk/LayoutTests/ChangeLog
trunk/LayoutTests/platform/mac/TestExpectations
trunk/LayoutTests/streams/reference-implementation/bad-underlying-sinks-expected.txt
trunk/LayoutTests/streams/reference-implementation/brand-checks-expected.txt
trunk/LayoutTests/streams/reference-implementation/pipe-to-expected.txt
trunk/LayoutTests/streams/reference-implementation/writable-stream-abort-expected.txt
trunk/LayoutTests/streams/reference-implementation/writable-stream-expected.txt
trunk/Source/WebCore/ChangeLog
trunk/Source/WebCore/Modules/streams/StreamInternals.js
trunk/Source/WebCore/Modules/streams/WritableStream.js
trunk/Source/WebCore/Modules/streams/WritableStreamInternals.js




Diff

Modified: trunk/LayoutTests/ChangeLog (191621 => 191622)

--- trunk/LayoutTests/ChangeLog	2015-10-27 07:00:57 UTC (rev 191621)
+++ trunk/LayoutTests/ChangeLog	2015-10-27 11:36:56 UTC (rev 191622)
@@ -1,3 +1,22 @@
+2015-10-27  Xabier Rodriguez Calvar  
+
+[Streams API] Add close method to writable stream
+https://bugs.webkit.org/show_bug.cgi?id=150560
+
+Reviewed by Darin Adler.
+
+New expectations after adding close method to writable stream.
+
+Two tests were also flagged because of bug 147933 that prevents them from running properly in certain
+conditions. So far this only happened in Mac, hence they were flagged only for it.
+
+* platform/mac/TestExpectations:
+* streams/reference-implementation/bad-underlying-sinks-expected.txt:
+* streams/reference-implementation/brand-checks-expected.txt:
+* streams/reference-implementation/pipe-to-expected.txt:
+* streams/reference-implementation/writable-stream-abort-expected.txt:
+* streams/reference-implementation/writable-stream-expected.txt:
+
 2015-10-26  Ryan Haddad  
 
 Removing duplicated test expectations and removing debug flag from flaky fast/canvas/webgl tests


Modified: trunk/LayoutTests/platform/mac/TestExpectations (191621 => 191622)

--- trunk/LayoutTests/platform/mac/TestExpectations	2015-10-27 07:00:57 UTC (rev 191621)
+++ trunk/LayoutTests/platform/mac/TestExpectations	2015-10-27 11:36:56 UTC (rev 191622)
@@ -1348,3 +1348,7 @@
 
 # Imported Blink tests which have not been investigated.
 imported/blink/compositing/video/video-controls-layer-creation-squashing.html [ Pass ImageOnlyFailure ]
+
+# Promises callbacks issues create problems in the writable stream tests
+webkit.org/b/147933 streams/reference-implementation/count-queuing-strategy.html [ Pass Failure ]
+webkit.org/b/147933 streams/reference-implementation/writable-stream-abort.html [ Pass Failure ]


Modified: trunk/LayoutTests/streams/reference-implementation/bad-underlying-sinks-expected.txt (191621 => 191622)

--- trunk/LayoutTests/streams/reference-implementation/bad-underlying-sinks-expected.txt	2015-10-27 07:00:57 UTC (rev 191621)
+++ trunk/LayoutTests/streams/reference-implementation/bad-underlying-sinks-expected.txt	2015-10-27 11:36:56 UTC (rev 191622)
@@ -5,6 +5,6 @@
 FAIL Underlying sink: throwing write method write not implemented
 PASS Underlying sink: throwing abort getter 
 PASS Underlying sink: throwing abort method 
-FAIL Underlying sink: throwing close getter close not implemented
-FAIL Underlying sink: throwing close method close not implemented
+PASS Underlying sink: throwing close getter 
+PASS Underlying sink: throwing close method 
 


Modified: trunk/LayoutTests/streams/reference-implementation/brand-checks-expected.txt (191621 => 191622)

---

[webkit-changes] [191624] branches/safari-601-branch/Source

2015-10-27 Thread bshafiei 
Title: [191624] branches/safari-601-branch/Source








Revision 191624
Author bshaf...@apple.com
Date 2015-10-27 10:27:53 -0700 (Tue, 27 Oct 2015)


Log Message
Versioning.

Modified Paths

branches/safari-601-branch/Source/_javascript_Core/Configurations/Version.xcconfig
branches/safari-601-branch/Source/WebCore/Configurations/Version.xcconfig
branches/safari-601-branch/Source/WebInspectorUI/Configurations/Version.xcconfig
branches/safari-601-branch/Source/WebKit/mac/Configurations/Version.xcconfig
branches/safari-601-branch/Source/WebKit2/Configurations/Version.xcconfig




Diff

Modified: branches/safari-601-branch/Source/_javascript_Core/Configurations/Version.xcconfig (191623 => 191624)

--- branches/safari-601-branch/Source/_javascript_Core/Configurations/Version.xcconfig	2015-10-27 16:38:10 UTC (rev 191623)
+++ branches/safari-601-branch/Source/_javascript_Core/Configurations/Version.xcconfig	2015-10-27 17:27:53 UTC (rev 191624)
@@ -23,7 +23,7 @@
 
 MAJOR_VERSION = 601;
 MINOR_VERSION = 3;
-TINY_VERSION = 6;
+TINY_VERSION = 7;
 MICRO_VERSION = 0;
 NANO_VERSION = 0;
 FULL_VERSION = $(MAJOR_VERSION).$(MINOR_VERSION).$(TINY_VERSION);


Modified: branches/safari-601-branch/Source/WebCore/Configurations/Version.xcconfig (191623 => 191624)

--- branches/safari-601-branch/Source/WebCore/Configurations/Version.xcconfig	2015-10-27 16:38:10 UTC (rev 191623)
+++ branches/safari-601-branch/Source/WebCore/Configurations/Version.xcconfig	2015-10-27 17:27:53 UTC (rev 191624)
@@ -23,7 +23,7 @@
 
 MAJOR_VERSION = 601;
 MINOR_VERSION = 3;
-TINY_VERSION = 6;
+TINY_VERSION = 7;
 MICRO_VERSION = 0;
 NANO_VERSION = 0;
 FULL_VERSION = $(MAJOR_VERSION).$(MINOR_VERSION).$(TINY_VERSION);


Modified: branches/safari-601-branch/Source/WebInspectorUI/Configurations/Version.xcconfig (191623 => 191624)

--- branches/safari-601-branch/Source/WebInspectorUI/Configurations/Version.xcconfig	2015-10-27 16:38:10 UTC (rev 191623)
+++ branches/safari-601-branch/Source/WebInspectorUI/Configurations/Version.xcconfig	2015-10-27 17:27:53 UTC (rev 191624)
@@ -1,6 +1,6 @@
 MAJOR_VERSION = 601;
 MINOR_VERSION = 3;
-TINY_VERSION = 6;
+TINY_VERSION = 7;
 MICRO_VERSION = 0;
 NANO_VERSION = 0;
 FULL_VERSION = $(MAJOR_VERSION).$(MINOR_VERSION).$(TINY_VERSION);


Modified: branches/safari-601-branch/Source/WebKit/mac/Configurations/Version.xcconfig (191623 => 191624)

--- branches/safari-601-branch/Source/WebKit/mac/Configurations/Version.xcconfig	2015-10-27 16:38:10 UTC (rev 191623)
+++ branches/safari-601-branch/Source/WebKit/mac/Configurations/Version.xcconfig	2015-10-27 17:27:53 UTC (rev 191624)
@@ -23,7 +23,7 @@
 
 MAJOR_VERSION = 601;
 MINOR_VERSION = 3;
-TINY_VERSION = 6;
+TINY_VERSION = 7;
 MICRO_VERSION = 0;
 NANO_VERSION = 0;
 FULL_VERSION = $(MAJOR_VERSION).$(MINOR_VERSION).$(TINY_VERSION);


Modified: branches/safari-601-branch/Source/WebKit2/Configurations/Version.xcconfig (191623 => 191624)

--- branches/safari-601-branch/Source/WebKit2/Configurations/Version.xcconfig	2015-10-27 16:38:10 UTC (rev 191623)
+++ branches/safari-601-branch/Source/WebKit2/Configurations/Version.xcconfig	2015-10-27 17:27:53 UTC (rev 191624)
@@ -23,7 +23,7 @@
 
 MAJOR_VERSION = 601;
 MINOR_VERSION = 3;
-TINY_VERSION = 6;
+TINY_VERSION = 7;
 MICRO_VERSION = 0;
 NANO_VERSION = 0;
 FULL_VERSION = $(MAJOR_VERSION).$(MINOR_VERSION).$(TINY_VERSION);






___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

[webkit-changes] [191625] trunk

2015-10-27 Thread msaboff 
Title: [191625] trunk








Revision 191625
Author msab...@apple.com
Date 2015-10-27 10:48:51 -0700 (Tue, 27 Oct 2015)


Log Message
REGRESSION (r191360): Crash: com.apple.WebKit.WebContent at com.apple._javascript_Core: JSC::FTL:: + 386
https://bugs.webkit.org/show_bug.cgi?id=150580

Reviewed by Mark Lam.

Source/_javascript_Core:

Changed code to box 32 bit integers and booleans arguments when generating the call instead of boxing
them in the shuffler.

The ASSERT in CallFrameShuffler::extendFrameIfNeeded is wrong when called from CallFrameShuffler::spill(),
as we could be making space to spill a register so that we have a spare that we can use for the new
frame's base pointer.

* ftl/FTLJSTailCall.cpp:
(JSC::FTL::DFG::recoveryFor): Added RELEASE_ASSERT to check that we never see unboxed 32 bit
arguments stored in the stack.
* ftl/FTLLowerDFGToLLVM.cpp:
(JSC::FTL::DFG::LowerDFGToLLVM::exitValueForTailCall):
* jit/CallFrameShuffler.cpp:
(JSC::CallFrameShuffler::extendFrameIfNeeded): Removed unneeded ASSERT.

LayoutTests:

New regression test.

* js/regress-150580-expected.txt: Added.
* js/regress-150580.html: Added.
* js/script-tests/regress-150580.js: Added.
(addEmUp):
(sumVector):
(test):

Modified Paths

trunk/LayoutTests/ChangeLog
trunk/Source/_javascript_Core/ChangeLog
trunk/Source/_javascript_Core/ftl/FTLJSTailCall.cpp
trunk/Source/_javascript_Core/ftl/FTLLowerDFGToLLVM.cpp
trunk/Source/_javascript_Core/jit/CallFrameShuffler.cpp


Added Paths

trunk/LayoutTests/js/regress-150580-expected.txt
trunk/LayoutTests/js/regress-150580.html
trunk/LayoutTests/js/script-tests/regress-150580.js




Diff

Modified: trunk/LayoutTests/ChangeLog (191624 => 191625)

--- trunk/LayoutTests/ChangeLog	2015-10-27 17:27:53 UTC (rev 191624)
+++ trunk/LayoutTests/ChangeLog	2015-10-27 17:48:51 UTC (rev 191625)
@@ -1,3 +1,19 @@
+2015-10-27  Michael Saboff  
+
+REGRESSION (r191360): Crash: com.apple.WebKit.WebContent at com.apple._javascript_Core: JSC::FTL:: + 386
+https://bugs.webkit.org/show_bug.cgi?id=150580
+
+Reviewed by Mark Lam.
+
+New regression test.
+
+* js/regress-150580-expected.txt: Added.
+* js/regress-150580.html: Added.
+* js/script-tests/regress-150580.js: Added.
+(addEmUp):
+(sumVector):
+(test):
+
 2015-10-20  Zalan Bujtas  
 
 Subpixel layout: Convert RenderTable* and AutoTableLayout to use LayoutUnit.


Added: trunk/LayoutTests/js/regress-150580-expected.txt (0 => 191625)

--- trunk/LayoutTests/js/regress-150580-expected.txt	(rev 0)
+++ trunk/LayoutTests/js/regress-150580-expected.txt	2015-10-27 17:48:51 UTC (rev 191625)
@@ -0,0 +1,10 @@
+Regression test for https://webkit.org/b/150580. - This test should not crash.
+
+On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
+
+
+PASS Properly handled tail calls with 32 bit integer arguments when all registers are used.
+PASS successfullyParsed is true
+
+TEST COMPLETE
+


Added: trunk/LayoutTests/js/regress-150580.html (0 => 191625)

--- trunk/LayoutTests/js/regress-150580.html	(rev 0)
+++ trunk/LayoutTests/js/regress-150580.html	2015-10-27 17:48:51 UTC (rev 191625)
@@ -0,0 +1,10 @@
+
+
+

[webkit-changes] [191626] trunk

2015-10-27 Thread commit-queue 
Title: [191626] trunk








Revision 191626
Author commit-qu...@webkit.org
Date 2015-10-27 10:53:56 -0700 (Tue, 27 Oct 2015)


Log Message
Do not sanitize user input for input[type=url]
https://bugs.webkit.org/show_bug.cgi?id=150346


Patch by Keith Rollin  on 2015-10-27
Reviewed by Darin Adler.

Source/WebCore:

Do not sanitize user input in text-based input fields that support
the Selection API, in order to not break _javascript_ code that expects
element.value to match what's on the screen.

Test: fast/forms/input-user-input-sanitization.html

* html/TextFieldInputType.cpp:
(WebCore::TextFieldInputType::subtreeHasChanged):

LayoutTests:

Test the sanitization of text-based input fields when the user enters
text.

* fast/forms/input-user-input-sanitization-expected.txt: Added.
* fast/forms/input-user-input-sanitization.html: Added.

Modified Paths

trunk/LayoutTests/ChangeLog
trunk/Source/WebCore/ChangeLog
trunk/Source/WebCore/html/HTMLInputElement.cpp
trunk/Source/WebCore/html/TextFieldInputType.cpp


Added Paths

trunk/LayoutTests/fast/forms/input-user-input-sanitization-expected.txt
trunk/LayoutTests/fast/forms/input-user-input-sanitization.html




Diff

Modified: trunk/LayoutTests/ChangeLog (191625 => 191626)

--- trunk/LayoutTests/ChangeLog	2015-10-27 17:48:51 UTC (rev 191625)
+++ trunk/LayoutTests/ChangeLog	2015-10-27 17:53:56 UTC (rev 191626)
@@ -1,3 +1,17 @@
+2015-10-27  Keith Rollin  
+
+Do not sanitize user input for input[type=url]
+https://bugs.webkit.org/show_bug.cgi?id=150346
+
+
+Reviewed by Darin Adler.
+
+Test the sanitization of text-based input fields when the user enters
+text.
+
+* fast/forms/input-user-input-sanitization-expected.txt: Added.
+* fast/forms/input-user-input-sanitization.html: Added.
+
 2015-10-27  Michael Saboff  
 
 REGRESSION (r191360): Crash: com.apple.WebKit.WebContent at com.apple._javascript_Core: JSC::FTL:: + 386


Added: trunk/LayoutTests/fast/forms/input-user-input-sanitization-expected.txt (0 => 191626)

--- trunk/LayoutTests/fast/forms/input-user-input-sanitization-expected.txt	(rev 0)
+++ trunk/LayoutTests/fast/forms/input-user-input-sanitization-expected.txt	2015-10-27 17:53:56 UTC (rev 191626)
@@ -0,0 +1,15 @@
+Check whether or not sanitization is performed on user input in text-input elements.
+
+On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
+
+
+PASS focusAndType("email", "   foo...@example.com   ").value is "foo...@example.com"
+PASS focusAndType("password", "   foobar   ").value is "   foobar   "
+PASS focusAndType("search", "   foobar   ").value is "   foobar   "
+PASS focusAndType("telephone", "   123-456-7890   ").value is "   123-456-7890   "
+PASS focusAndType("text", "   foobar   ").value is "   foobar   "
+PASS focusAndType("url", "   https://foobar.example.com   ").value is "   https://foobar.example.com   "
+PASS successfullyParsed is true
+
+TEST COMPLETE
+ 


Added: trunk/LayoutTests/fast/forms/input-user-input-sanitization.html (0 => 191626)

--- trunk/LayoutTests/fast/forms/input-user-input-sanitization.html	(rev 0)
+++ trunk/LayoutTests/fast/forms/input-user-input-sanitization.html	2015-10-27 17:53:56 UTC (rev 191626)
@@ -0,0 +1,47 @@
+
+
+
+
+
+
+
+

+
+
+
+
+
+
+

+
+