date:20160829

[webkit-changes] [205174] tags/Safari-603.1.3.0.1

2016-08-29 Thread bshafiei

Title: [205174] tags/Safari-603.1.3.0.1








Revision 205174
Author bshaf...@apple.com
Date 2016-08-29 23:10:18 -0700 (Mon, 29 Aug 2016)


Log Message
Merged r204901.  rdar://problem/28062188

Modified Paths

tags/Safari-603.1.3.0.1/LayoutTests/ChangeLog
tags/Safari-603.1.3.0.1/LayoutTests/TestExpectations
tags/Safari-603.1.3.0.1/Source/_javascript_Core/ChangeLog
tags/Safari-603.1.3.0.1/Source/_javascript_Core/bytecode/ObjectAllocationProfile.h
tags/Safari-603.1.3.0.1/Source/_javascript_Core/jit/JITOperations.cpp




Diff

Modified: tags/Safari-603.1.3.0.1/LayoutTests/ChangeLog (205173 => 205174)

--- tags/Safari-603.1.3.0.1/LayoutTests/ChangeLog	2016-08-30 06:07:44 UTC (rev 205173)
+++ tags/Safari-603.1.3.0.1/LayoutTests/ChangeLog	2016-08-30 06:10:18 UTC (rev 205174)
@@ -1,3 +1,16 @@
+2016-08-29  Babak Shafiei  
+
+Merge r204901.
+
+2016-08-23  Filip Pizlo  
+
+js/regress/put-by-id-transition-with-indexing-header.html and svg/carto.net/window.svg fail in debug after r204854
+https://bugs.webkit.org/show_bug.cgi?id=161115
+
+Reviewed by Keith Miller.
+
+* TestExpectations:
+
 2016-08-25  Babak Shafiei  
 
 Merge r204986.


Modified: tags/Safari-603.1.3.0.1/LayoutTests/TestExpectations (205173 => 205174)

--- tags/Safari-603.1.3.0.1/LayoutTests/TestExpectations	2016-08-30 06:07:44 UTC (rev 205173)
+++ tags/Safari-603.1.3.0.1/LayoutTests/TestExpectations	2016-08-30 06:10:18 UTC (rev 205174)
@@ -1234,4 +1234,3 @@
 webkit.org/b/159755 fast/text/emoji-gender-fe0f-9.html [ ImageOnlyFailure ]
 webkit.org/b/159755 fast/text/emoji-gender.html [ ImageOnlyFailure ]
 
-[Debug] webkit.org/b/161114 svg/carto.net/window.svg [ Skip ]


Modified: tags/Safari-603.1.3.0.1/Source/_javascript_Core/ChangeLog (205173 => 205174)

--- tags/Safari-603.1.3.0.1/Source/_javascript_Core/ChangeLog	2016-08-30 06:07:44 UTC (rev 205173)
+++ tags/Safari-603.1.3.0.1/Source/_javascript_Core/ChangeLog	2016-08-30 06:10:18 UTC (rev 205174)
@@ -1,5 +1,22 @@
 2016-08-29  Babak Shafiei  
 
+Merge r204901.
+
+2016-08-23  Filip Pizlo  
+
+js/regress/put-by-id-transition-with-indexing-header.html and svg/carto.net/window.svg fail in debug after r204854
+https://bugs.webkit.org/show_bug.cgi?id=161115
+
+Reviewed by Keith Miller.
+
+There were two small goofs.
+
+* bytecode/ObjectAllocationProfile.h:
+(JSC::ObjectAllocationProfile::isNull): The new policy is that the allocator can be null. So now the way you tell if the profile is null is by checking the structure.
+* jit/JITOperations.cpp: This was using DeferGC, which is now definitely wrong. It forces the GC to happen when the structure and butterfly are mismatched. It's better for the GC to happen before we put the butterfly in the object.
+
+2016-08-29  Babak Shafiei  
+
 Merge r204897.
 
 2016-08-24  Filip Pizlo  


Modified: tags/Safari-603.1.3.0.1/Source/_javascript_Core/bytecode/ObjectAllocationProfile.h (205173 => 205174)

--- tags/Safari-603.1.3.0.1/Source/_javascript_Core/bytecode/ObjectAllocationProfile.h	2016-08-30 06:07:44 UTC (rev 205173)
+++ tags/Safari-603.1.3.0.1/Source/_javascript_Core/bytecode/ObjectAllocationProfile.h	2016-08-30 06:10:18 UTC (rev 205174)
@@ -45,7 +45,7 @@
 {
 }
 
-bool isNull() { return !m_allocator; }
+bool isNull() { return !m_structure; }
 
 void initialize(VM& vm, JSCell* owner, JSObject* prototype, unsigned inferredInlineCapacity)
 {


Modified: tags/Safari-603.1.3.0.1/Source/_javascript_Core/jit/JITOperations.cpp (205173 => 205174)

--- tags/Safari-603.1.3.0.1/Source/_javascript_Core/jit/JITOperations.cpp	2016-08-30 06:07:44 UTC (rev 205173)
+++ tags/Safari-603.1.3.0.1/Source/_javascript_Core/jit/JITOperations.cpp	2016-08-30 06:10:18 UTC (rev 205174)
@@ -2119,7 +2119,6 @@
 NativeCallFrameTracer tracer(&vm, exec);
 
 ASSERT(!object->structure()->outOfLineCapacity());
-DeferGC deferGC(vm.heap);
 Butterfly* result = object->growOutOfLineStorage(vm, 0, initialOutOfLineCapacity);
 object->setButterflyWithoutChangingStructure(vm, result);
 return reinterpret_cast(result);
@@ -2130,7 +2129,6 @@
 VM& vm = exec->vm();
 NativeCallFrameTracer tracer(&vm, exec);
 
-DeferGC deferGC(vm.heap);
 Butterfly* result = object->growOutOfLineStorage(vm, object->structure()->outOfLineCapacity(), newSize);
 object->setButterflyWithoutChangingStructure(vm, result);
 return reinterpret_cast(result);






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

[webkit-changes] [205173] tags/Safari-603.1.3.0.1/Source/JavaScriptCore

2016-08-29 Thread bshafiei

Title: [205173] tags/Safari-603.1.3.0.1/Source/_javascript_Core








Revision 205173
Author bshaf...@apple.com
Date 2016-08-29 23:07:44 -0700 (Mon, 29 Aug 2016)


Log Message
Merged r204897.  rdar://problem/28062188

Modified Paths

tags/Safari-603.1.3.0.1/Source/_javascript_Core/ChangeLog
tags/Safari-603.1.3.0.1/Source/_javascript_Core/jit/AssemblyHelpers.h




Diff

Modified: tags/Safari-603.1.3.0.1/Source/_javascript_Core/ChangeLog (205172 => 205173)

--- tags/Safari-603.1.3.0.1/Source/_javascript_Core/ChangeLog	2016-08-30 05:56:25 UTC (rev 205172)
+++ tags/Safari-603.1.3.0.1/Source/_javascript_Core/ChangeLog	2016-08-30 06:07:44 UTC (rev 205173)
@@ -1,3 +1,23 @@
+2016-08-29  Babak Shafiei  
+
+Merge r204897.
+
+2016-08-24  Filip Pizlo  
+
+AssemblyHelpers::emitAllocateWithNonNullAllocator() crashes in the FTL on ARM64
+https://bugs.webkit.org/show_bug.cgi?id=161138
+rdar://problem/27985868
+
+Reviewed by Saam Barati.
+
+The FTL expects that this method can be used with scratch registers disallowed, but it
+uses addPtr(Addr, Reg).
+
+The solution is to only use addPtr(Addr, Reg) on x86.
+
+* jit/AssemblyHelpers.h:
+(JSC::AssemblyHelpers::emitAllocateWithNonNullAllocator):
+
 2016-08-23  Ryan Haddad  
 
 Rebaseline builtins-generator-tests after r204854.


Modified: tags/Safari-603.1.3.0.1/Source/_javascript_Core/jit/AssemblyHelpers.h (205172 => 205173)

--- tags/Safari-603.1.3.0.1/Source/_javascript_Core/jit/AssemblyHelpers.h	2016-08-30 05:56:25 UTC (rev 205172)
+++ tags/Safari-603.1.3.0.1/Source/_javascript_Core/jit/AssemblyHelpers.h	2016-08-30 06:07:44 UTC (rev 205173)
@@ -1432,7 +1432,13 @@
 }
 negPtr(resultGPR);
 store32(scratchGPR, Address(allocatorGPR, MarkedAllocator::offsetOfFreeList() + OBJECT_OFFSETOF(FreeList, remaining)));
-addPtr(Address(allocatorGPR, MarkedAllocator::offsetOfFreeList() + OBJECT_OFFSETOF(FreeList, payloadEnd)), resultGPR);
+Address payloadEndAddr = Address(allocatorGPR, MarkedAllocator::offsetOfFreeList() + OBJECT_OFFSETOF(FreeList, payloadEnd));
+if (isX86())
+addPtr(payloadEndAddr, resultGPR);
+else {
+loadPtr(payloadEndAddr, scratchGPR);
+addPtr(scratchGPR, resultGPR);
+}
 
 done = jump();
 






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

[webkit-changes] [205172] trunk/Source/WebCore

2016-08-29 Thread achristensen

Title: [205172] trunk/Source/WebCore








Revision 205172
Author achristen...@apple.com
Date 2016-08-29 22:56:25 -0700 (Mon, 29 Aug 2016)


Log Message
Fix Windows build after r205161

* bindings/js/JSPerformanceTimingCustom.cpp:

Modified Paths

trunk/Source/WebCore/ChangeLog
trunk/Source/WebCore/bindings/js/JSPerformanceTimingCustom.cpp




Diff

Modified: trunk/Source/WebCore/ChangeLog (205171 => 205172)

--- trunk/Source/WebCore/ChangeLog	2016-08-30 05:24:13 UTC (rev 205171)
+++ trunk/Source/WebCore/ChangeLog	2016-08-30 05:56:25 UTC (rev 205172)
@@ -1,3 +1,9 @@
+2016-08-29  Alex Christensen  
+
+Fix Windows build after r205161
+
+* bindings/js/JSPerformanceTimingCustom.cpp:
+
 2016-08-29  Chris Dumez  
 
 We should throw a SecurityError when denying setting a cross-origin Location property


Modified: trunk/Source/WebCore/bindings/js/JSPerformanceTimingCustom.cpp (205171 => 205172)

--- trunk/Source/WebCore/bindings/js/JSPerformanceTimingCustom.cpp	2016-08-30 05:24:13 UTC (rev 205171)
+++ trunk/Source/WebCore/bindings/js/JSPerformanceTimingCustom.cpp	2016-08-30 05:56:25 UTC (rev 205172)
@@ -26,6 +26,7 @@
 #include "config.h"
 #include "JSPerformanceTiming.h"
 
+#include "DOMWrapperWorld.h"
 #include 
 #include 
 #include 






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

[webkit-changes] [205171] trunk

2016-08-29 Thread cdumez

Title: [205171] trunk








Revision 205171
Author cdu...@apple.com
Date 2016-08-29 22:24:13 -0700 (Mon, 29 Aug 2016)


Log Message
We should throw a SecurityError when denying setting a cross-origin Location property
https://bugs.webkit.org/show_bug.cgi?id=161368

Reviewed by Ryosuke Niwa.

Source/WebCore:

We should throw a SecurityError when denying setting a cross-origin
Location property:
- https://html.spec.whatwg.org/#location-set
- https://html.spec.whatwg.org/#crossoriginset-(-o,-p,-v,-receiver-)
- https://html.spec.whatwg.org/#location-getownproperty

Firefox and Chrome already throw. We currently ignore and log an error
message.

No new tests, updated existing tests.

* bindings/js/JSLocationCustom.cpp:
(WebCore::JSLocation::putDelegate):

LayoutTests:

Update / Rebaseline existing tests to reflect behavior change.

* http/tests/security/cross-frame-access-location-put-expected.txt:
* http/tests/security/location-cross-origin-expected.txt:
* http/tests/security/location-cross-origin.html:
* http/tests/security/xss-DENIED-assign-location-hash-expected.txt:
* http/tests/security/xss-DENIED-assign-location-host-expected.txt:
* http/tests/security/xss-DENIED-assign-location-hostname-expected.txt:
* http/tests/security/xss-DENIED-assign-location-nonstandardProperty-expected.txt:
* http/tests/security/xss-DENIED-assign-location-pathname-expected.txt:
* http/tests/security/xss-DENIED-assign-location-protocol-expected.txt:
* http/tests/security/xss-DENIED-assign-location-reload-expected.txt:
* http/tests/security/xss-DENIED-assign-location-search-expected.txt:

Modified Paths

trunk/LayoutTests/ChangeLog
trunk/LayoutTests/http/tests/security/cross-frame-access-location-put-expected.txt
trunk/LayoutTests/http/tests/security/location-cross-origin-expected.txt
trunk/LayoutTests/http/tests/security/location-cross-origin.html
trunk/LayoutTests/http/tests/security/xss-DENIED-assign-location-hash-expected.txt
trunk/LayoutTests/http/tests/security/xss-DENIED-assign-location-host-expected.txt
trunk/LayoutTests/http/tests/security/xss-DENIED-assign-location-hostname-expected.txt
trunk/LayoutTests/http/tests/security/xss-DENIED-assign-location-nonstandardProperty-expected.txt
trunk/LayoutTests/http/tests/security/xss-DENIED-assign-location-pathname-expected.txt
trunk/LayoutTests/http/tests/security/xss-DENIED-assign-location-protocol-expected.txt
trunk/LayoutTests/http/tests/security/xss-DENIED-assign-location-reload-expected.txt
trunk/LayoutTests/http/tests/security/xss-DENIED-assign-location-search-expected.txt
trunk/Source/WebCore/ChangeLog
trunk/Source/WebCore/bindings/js/JSLocationCustom.cpp




Diff

Modified: trunk/LayoutTests/ChangeLog (205170 => 205171)

--- trunk/LayoutTests/ChangeLog	2016-08-30 05:18:48 UTC (rev 205170)
+++ trunk/LayoutTests/ChangeLog	2016-08-30 05:24:13 UTC (rev 205171)
@@ -1,3 +1,24 @@
+2016-08-29  Chris Dumez  
+
+We should throw a SecurityError when denying setting a cross-origin Location property
+https://bugs.webkit.org/show_bug.cgi?id=161368
+
+Reviewed by Ryosuke Niwa.
+
+Update / Rebaseline existing tests to reflect behavior change.
+
+* http/tests/security/cross-frame-access-location-put-expected.txt:
+* http/tests/security/location-cross-origin-expected.txt:
+* http/tests/security/location-cross-origin.html:
+* http/tests/security/xss-DENIED-assign-location-hash-expected.txt:
+* http/tests/security/xss-DENIED-assign-location-host-expected.txt:
+* http/tests/security/xss-DENIED-assign-location-hostname-expected.txt:
+* http/tests/security/xss-DENIED-assign-location-nonstandardProperty-expected.txt:
+* http/tests/security/xss-DENIED-assign-location-pathname-expected.txt:
+* http/tests/security/xss-DENIED-assign-location-protocol-expected.txt:
+* http/tests/security/xss-DENIED-assign-location-reload-expected.txt:
+* http/tests/security/xss-DENIED-assign-location-search-expected.txt:
+
 2016-08-29  Gyuyoung Kim  
 
 [EFL] Gardening on Aug 30


Modified: trunk/LayoutTests/http/tests/security/cross-frame-access-location-put-expected.txt (205170 => 205171)

--- trunk/LayoutTests/http/tests/security/cross-frame-access-location-put-expected.txt	2016-08-30 05:18:48 UTC (rev 205170)
+++ trunk/LayoutTests/http/tests/security/cross-frame-access-location-put-expected.txt	2016-08-30 05:24:13 UTC (rev 205171)
@@ -1,8 +1,8 @@
-CONSOLE MESSAGE: line 29: Blocked a frame with origin "http://127.0.0.1:8000" from accessing a frame with origin "http://localhost:8000". Protocols, domains, and ports must match.
-CONSOLE MESSAGE: line 29: Blocked a frame with origin "http://127.0.0.1:8000" from accessing a frame with origin "http://localhost:8000". Protocols, domains, and ports must match.
-CONSOLE MESSAGE: line 29: Blocked a frame with origin "http://127.0.0.1:8000" from accessing a frame with origin "http://localhost:8000". Protocols, domains, and ports must match.
-CONSO

[webkit-changes] [205170] trunk/LayoutTests

2016-08-29 Thread gyuyoung . kim

Title: [205170] trunk/LayoutTests








Revision 205170
Author gyuyoung@webkit.org
Date 2016-08-29 22:18:48 -0700 (Mon, 29 Aug 2016)


Log Message
[EFL] Gardening on Aug 30

Unreviewed gardening. Mark unexpecting passing tests to PASS.

* platform/efl/TestExpectations:

Modified Paths

trunk/LayoutTests/ChangeLog
trunk/LayoutTests/platform/efl/TestExpectations




Diff

Modified: trunk/LayoutTests/ChangeLog (205169 => 205170)

--- trunk/LayoutTests/ChangeLog	2016-08-30 04:53:17 UTC (rev 205169)
+++ trunk/LayoutTests/ChangeLog	2016-08-30 05:18:48 UTC (rev 205170)
@@ -1,3 +1,11 @@
+2016-08-29  Gyuyoung Kim  
+
+[EFL] Gardening on Aug 30
+
+Unreviewed gardening. Mark unexpecting passing tests to PASS.
+
+* platform/efl/TestExpectations:
+
 2016-08-29  Chris Dumez  
 
 Unskip several web-platform-tests that are now passing


Modified: trunk/LayoutTests/platform/efl/TestExpectations (205169 => 205170)

--- trunk/LayoutTests/platform/efl/TestExpectations	2016-08-30 04:53:17 UTC (rev 205169)
+++ trunk/LayoutTests/platform/efl/TestExpectations	2016-08-30 05:18:48 UTC (rev 205170)
@@ -835,7 +835,7 @@
 webkit.org/b/103926 media/track/opera/track/track-element/cloneNode.html [ Skip ]
 webkit.org/b/103926 media/track/opera/track/track-element/src-clear-cues.html [ Skip ]
 webkit.org/b/103926 media/track/opera/track/webvtt/parsing/001.html [ Skip ]
-webkit.org/b/103926 media/track/opera/track/webvtt/parsing-cue-data/tests/entities.html [ Skip ] 
+webkit.org/b/103926 media/track/opera/track/webvtt/parsing-cue-data/tests/entities.html [ Skip ]
 webkit.org/b/103926 media/track/opera/track/webvtt/parsing-cue-data/tests/tags.html [ Skip ]
 webkit.org/b/103926 media/track/opera/track/webvtt/rendering/adhoc/12345_timestamps.html [ Skip ]
 webkit.org/b/103926 media/track/opera/track/webvtt/parsing-cue-data/tests/timestamps.html [ Skip ]
@@ -1280,6 +1280,14 @@
 imported/w3c/web-platform-tests/html/semantics/embedded-content/the-img-element/sizes/parse-a-sizes-attribute.html [ Pass ]
 imported/w3c/web-platform-tests/html/semantics/embedded-content/the-video-element/video_initially_paused.html [ Pass ]
 imported/w3c/web-platform-tests/html/semantics/grouping-content/the-ol-element/grouping-ol.html [ Pass ]
+imported/w3c/web-platform-tests/IndexedDB/idbcursor_iterating.htm [ Pass ]
+imported/w3c/web-platform-tests/IndexedDB/idbindex-multientry-big.htm [ Pass ]
+imported/w3c/web-platform-tests/dom/nodes/Document-characterSet-normalization.html [ Pass ]
+imported/w3c/web-platform-tests/html/browsers/browsing-the-web/unloading-documents/001.html [ Pass ]
+imported/w3c/web-platform-tests/html/browsers/browsing-the-web/unloading-documents/002.html [ Pass ]
+imported/w3c/web-platform-tests/html/browsers/browsing-the-web/unloading-documents/003.html [ Pass ]
+imported/w3c/web-platform-tests/html/browsers/browsing-the-web/unloading-documents/005.html [ Pass ]
+imported/w3c/web-platform-tests/html/semantics/document-metadata/styling/LinkStyle.html [ Pass ]
 css3/conditional/w3c/at-supports-010.html [ Pass ]
 css3/conditional/w3c/at-supports-014.html [ Pass ]
 fast/borders/border-painting-dashed.html [ Pass ]
@@ -1355,6 +1363,37 @@
 
 webkit.org/b/148695 fast/shadow-dom [ Pass ]
 
+Bug(EFL) fast/text/emoji-gender-2-3.html [ Pass ]
+Bug(EFL) fast/text/emoji-gender-2-4.html [ Pass ]
+Bug(EFL) fast/text/emoji-gender-2-5.html [ Pass ]
+Bug(EFL) fast/text/emoji-gender-2-6.html [ Pass ]
+Bug(EFL) fast/text/emoji-gender-2-7.html [ Pass ]
+Bug(EFL) fast/text/emoji-gender-2-8.html [ Pass ]
+Bug(EFL) fast/text/emoji-gender-2-9.html [ Pass ]
+Bug(EFL) fast/text/emoji-gender-3.html [ Pass ]
+Bug(EFL) fast/text/emoji-gender-4.html [ Pass ]
+Bug(EFL) fast/text/emoji-gender-5.html [ Pass ]
+Bug(EFL) fast/text/emoji-gender-6.html [ Pass ]
+Bug(EFL) fast/text/emoji-gender-8.html [ Pass ]
+Bug(EFL) fast/text/emoji-gender-9.html [ Pass ]
+Bug(EFL) fast/text/emoji-gender-fe0f-3.html [ Pass ]
+Bug(EFL) fast/text/emoji-gender-fe0f-4.html [ Pass ]
+Bug(EFL) fast/text/emoji-gender-fe0f-5.html [ Pass ]
+Bug(EFL) fast/text/emoji-gender-fe0f-6.html [ Pass ]
+Bug(EFL) fast/text/emoji-gender-fe0f-7.html [ Pass ]
+Bug(EFL) fast/text/emoji-gender-fe0f-8.html [ Pass ]
+Bug(EFL) fast/text/emoji-gender-fe0f-9.html [ Pass ]
+
+Bug(EFL) fast/scrolling/rtl-scrollbars-alternate-body-dir-attr-does-not-update-scrollbar-placement-2.html [ Pass ]
+Bug(EFL) fast/scrolling/rtl-scrollbars-alternate-iframe-body-dir-attr-does-not-update-scrollbar-placement.html [ Pass ]
+Bug(EFL) fast/scrolling/rtl-scrollbars-elementFromPoint-static.html [ Pass ]
+Bug(EFL) fast/scrolling/rtl-scrollbars-listbox-scroll.html [ Pass ]
+Bug(EFL) fast/scrolling/rtl-scrollbars-listbox-select-left.html [ Pass ]
+Bug(EFL) fast/scrolling/rtl-scrollbars-listbox-select-right.html [ Pass ]
+Bug(EFL) fast/scrolling/rtl-scrollbars-listbox.html [ Pass ]
+Bug(EFL) fast/scrolling/rtl-scrollbars-overflow-elementFromPoint.html [ Pass ]
+Bug(EFL) fast/scrolling/rtl-scrollbars-positioned

[webkit-changes] [205169] trunk/LayoutTests

2016-08-29 Thread cdumez

Title: [205169] trunk/LayoutTests








Revision 205169
Author cdu...@apple.com
Date 2016-08-29 21:53:17 -0700 (Mon, 29 Aug 2016)


Log Message
Unskip several web-platform-tests that are now passing
https://bugs.webkit.org/show_bug.cgi?id=161367

Reviewed by Geoffrey Garen.

LayoutTests/imported/w3c:

Rebaseline those tests and update window-security.sub.html as per:
https://github.com/w3c/web-platform-tests/pull/3607

* web-platform-tests/html/browsers/history/the-location-interface/security_location_0.sub-expected.txt:
* web-platform-tests/html/browsers/the-window-object/security-window/window-security.sub-expected.txt:
* web-platform-tests/html/browsers/the-window-object/security-window/window-security.sub.html:
* web-platform-tests/html/browsers/windows/nested-browsing-contexts/frameElement.sub-expected.txt:

LayoutTests:

Unskip several web-platform-tests that are now passing.

* TestExpectations:

Modified Paths

trunk/LayoutTests/ChangeLog
trunk/LayoutTests/TestExpectations
trunk/LayoutTests/imported/w3c/ChangeLog
trunk/LayoutTests/imported/w3c/web-platform-tests/html/browsers/history/the-location-interface/security_location_0.sub-expected.txt
trunk/LayoutTests/imported/w3c/web-platform-tests/html/browsers/the-window-object/security-window/window-security.sub-expected.txt
trunk/LayoutTests/imported/w3c/web-platform-tests/html/browsers/the-window-object/security-window/window-security.sub.html
trunk/LayoutTests/imported/w3c/web-platform-tests/html/browsers/windows/nested-browsing-contexts/frameElement.sub-expected.txt




Diff

Modified: trunk/LayoutTests/ChangeLog (205168 => 205169)

--- trunk/LayoutTests/ChangeLog	2016-08-30 02:07:12 UTC (rev 205168)
+++ trunk/LayoutTests/ChangeLog	2016-08-30 04:53:17 UTC (rev 205169)
@@ -1,3 +1,14 @@
+2016-08-29  Chris Dumez  
+
+Unskip several web-platform-tests that are now passing
+https://bugs.webkit.org/show_bug.cgi?id=161367
+
+Reviewed by Geoffrey Garen.
+
+Unskip several web-platform-tests that are now passing.
+
+* TestExpectations:
+
 2016-08-29  Joseph Pecoraro  
 
 REGRESSION(r202568): Web Inspector: Expanding Array Prototype in Console shows no properties


Modified: trunk/LayoutTests/TestExpectations (205168 => 205169)

--- trunk/LayoutTests/TestExpectations	2016-08-30 02:07:12 UTC (rev 205168)
+++ trunk/LayoutTests/TestExpectations	2016-08-30 04:53:17 UTC (rev 205169)
@@ -475,6 +475,7 @@
 # WPT tests that fail after doing full test repository reimport and need further investigation
 imported/w3c/web-platform-tests/XMLHttpRequest/send-entity-body-document.htm [ Pass Failure Timeout ]
 imported/w3c/web-platform-tests/dom/nodes/Document-createElement-namespace.html [ Pass Failure ]
+imported/w3c/web-platform-tests/html/browsers/windows/nested-browsing-contexts/frameElement.sub.html [ Pass Failure ]
 imported/w3c/web-platform-tests/html/browsers/windows/targeting-cross-origin-nested-browsing-contexts.sub.html [ Pass Failure ]
 
 # W3C XMLHttpRequest tests
@@ -1211,12 +1212,6 @@
 
 editing/deleting/delete-emoji.html [ Slow ]
 
-# W3C tests what are trying to access www1.localhost.
-webkit.org/b/161142 imported/w3c/web-platform-tests/html/browsers/history/the-location-interface/security_location_0.sub.htm [ Skip ]
-webkit.org/b/161142 imported/w3c/web-platform-tests/html/browsers/the-window-object/security-window/window-security.sub.html [ Skip ]
-webkit.org/b/161142 imported/w3c/web-platform-tests/html/browsers/windows/nested-browsing-contexts/frameElement.sub.html [ Skip ]
-webkit.org/b/161142 imported/w3c/web-platform-tests/html/semantics/embedded-content/the-iframe-element/cross_origin_parentage.html [ Skip ]
-
 # New emoji requires system support.
 webkit.org/b/159755 fast/text/emoji-num-glyphs.html [ Failure ]
 webkit.org/b/159755 fast/text/emoji-gender-2-3.html [ ImageOnlyFailure ]


Modified: trunk/LayoutTests/imported/w3c/ChangeLog (205168 => 205169)

--- trunk/LayoutTests/imported/w3c/ChangeLog	2016-08-30 02:07:12 UTC (rev 205168)
+++ trunk/LayoutTests/imported/w3c/ChangeLog	2016-08-30 04:53:17 UTC (rev 205169)
@@ -1,5 +1,20 @@
 2016-08-29  Chris Dumez  
 
+Unskip several web-platform-tests that are now passing
+https://bugs.webkit.org/show_bug.cgi?id=161367
+
+Reviewed by Geoffrey Garen.
+
+Rebaseline those tests and update window-security.sub.html as per:
+https://github.com/w3c/web-platform-tests/pull/3607
+
+* web-platform-tests/html/browsers/history/the-location-interface/security_location_0.sub-expected.txt:
+* web-platform-tests/html/browsers/the-window-object/security-window/window-security.sub-expected.txt:
+* web-platform-tests/html/browsers/the-window-object/security-window/window-security.sub.html:
+* web-platform-tests/html/browsers/windows/nested-browsing-contexts/frameElement.sub-expected.txt:
+
+2016-08-29  Chris Dumez  
+
 document.createEvent("popstateevent") should create a PopStateEvent
 ht

[webkit-changes] [205168] trunk

2016-08-29 Thread commit-queue

Title: [205168] trunk








Revision 205168
Author commit-qu...@webkit.org
Date 2016-08-29 19:07:12 -0700 (Mon, 29 Aug 2016)


Log Message
REGRESSION(r202568): Web Inspector: Expanding Array Prototype in Console shows no properties
https://bugs.webkit.org/show_bug.cgi?id=161263


Patch by Joseph Pecoraro  on 2016-08-29
Reviewed by Matt Baker.

Source/_javascript_Core:

* inspector/InjectedScriptSource.js:
(InjectedScript.prototype._propertyDescriptors):
Previously we only took the "numeric index fast path" if an object was
array like with length > 100. When we dropped the length check we
ended up breaking our display of Array prototype, because [].__proto__
is an array instance. Get it back by just doing a check of length > 0.
We may want to address this differently in the future by knowing if
we are getting properties for a prototype or not.

LayoutTests:

* inspector/model/remote-object-get-properties-expected.txt:
* inspector/model/remote-object-get-properties.html:
Include tests for an Array and an Array's proto (which is an Array).

Modified Paths

trunk/LayoutTests/ChangeLog
trunk/LayoutTests/inspector/model/remote-object-get-properties-expected.txt
trunk/LayoutTests/inspector/model/remote-object-get-properties.html
trunk/Source/_javascript_Core/ChangeLog
trunk/Source/_javascript_Core/inspector/InjectedScriptSource.js




Diff

Modified: trunk/LayoutTests/ChangeLog (205167 => 205168)

--- trunk/LayoutTests/ChangeLog	2016-08-30 01:56:37 UTC (rev 205167)
+++ trunk/LayoutTests/ChangeLog	2016-08-30 02:07:12 UTC (rev 205168)
@@ -1,3 +1,15 @@
+2016-08-29  Joseph Pecoraro  
+
+REGRESSION(r202568): Web Inspector: Expanding Array Prototype in Console shows no properties
+https://bugs.webkit.org/show_bug.cgi?id=161263
+
+
+Reviewed by Matt Baker.
+
+* inspector/model/remote-object-get-properties-expected.txt:
+* inspector/model/remote-object-get-properties.html:
+Include tests for an Array and an Array's proto (which is an Array).
+
 2016-08-29  Dean Jackson  
 
 JSON.stringify returns empty when used with performance.timing.


Modified: trunk/LayoutTests/inspector/model/remote-object-get-properties-expected.txt (205167 => 205168)

--- trunk/LayoutTests/inspector/model/remote-object-get-properties-expected.txt	2016-08-30 01:56:37 UTC (rev 205167)
+++ trunk/LayoutTests/inspector/model/remote-object-get-properties-expected.txt	2016-08-30 02:07:12 UTC (rev 205168)
@@ -32,6 +32,195 @@
 -
 
 -
+_expression_: window.simpleArray
+type: object
+subtype: array
+description: Array
+
+OWN PROPERTIES:
+0
+1
+2
+__proto__
+
+DISPLAYABLE PROPERTIES:
+0
+1
+2
+__proto__
+
+ALL PROPERTIES:
+0
+1
+2
+length
+toString
+values
+toLocaleString
+concat
+fill
+join
+pop
+push
+reverse
+shift
+slice
+sort
+splice
+unshift
+every
+forEach
+some
+indexOf
+lastIndexOf
+filter
+reduce
+reduceRight
+map
+entries
+keys
+find
+findIndex
+includes
+copyWithin
+constructor
+Symbol(Symbol.iterator)
+Symbol(Symbol.unscopables)
+valueOf
+hasOwnProperty
+propertyIsEnumerable
+isPrototypeOf
+__defineGetter__
+__defineSetter__
+__lookupGetter__
+__lookupSetter__
+__proto__
+-
+
+-
+_expression_: window.arrayProto
+type: object
+subtype: array
+description: Array
+
+OWN PROPERTIES:
+length
+toString
+values
+toLocaleString
+concat
+fill
+join
+pop
+push
+reverse
+shift
+slice
+sort
+splice
+unshift
+every
+forEach
+some
+indexOf
+lastIndexOf
+filter
+reduce
+reduceRight
+map
+entries
+keys
+find
+findIndex
+includes
+copyWithin
+constructor
+Symbol(Symbol.iterator)
+Symbol(Symbol.unscopables)
+__proto__
+
+DISPLAYABLE PROPERTIES:
+length
+toString
+values
+toLocaleString
+concat
+fill
+join
+pop
+push
+reverse
+shift
+slice
+sort
+splice
+unshift
+every
+forEach
+some
+indexOf
+lastIndexOf
+filter
+reduce
+reduceRight
+map
+entries
+keys
+find
+findIndex
+includes
+copyWithin
+constructor
+Symbol(Symbol.iterator)
+Symbol(Symbol.unscopables)
+__proto__
+
+ALL PROPERTIES:
+length
+toString
+values
+toLocaleString
+concat
+fill
+join
+pop
+push
+reverse
+shift
+slice
+sort
+splice
+unshift
+every
+forEach
+some
+indexOf
+lastIndexOf
+filter
+reduce
+reduceRight
+map
+entries
+keys
+find
+findIndex

[webkit-changes] [205167] trunk/Source/WebKit2

2016-08-29 Thread gyuyoung . kim

Title: [205167] trunk/Source/WebKit2








Revision 205167
Author gyuyoung@webkit.org
Date 2016-08-29 18:56:37 -0700 (Mon, 29 Aug 2016)


Log Message
Unreviewed EFL/GTK build fix since r205150

Patch by Gyuyoung Kim  on 2016-08-29

* CMakeLists.txt: Add Shared/FrameInfoData.cpp.

Modified Paths

trunk/Source/WebKit2/CMakeLists.txt
trunk/Source/WebKit2/ChangeLog




Diff

Modified: trunk/Source/WebKit2/CMakeLists.txt (205166 => 205167)

--- trunk/Source/WebKit2/CMakeLists.txt	2016-08-30 01:48:10 UTC (rev 205166)
+++ trunk/Source/WebKit2/CMakeLists.txt	2016-08-30 01:56:37 UTC (rev 205167)
@@ -240,6 +240,7 @@
 Shared/ContextMenuContextData.cpp
 Shared/EditorState.cpp
 Shared/FontInfo.cpp
+Shared/FrameInfoData.cpp
 Shared/LayerTreeContext.cpp
 Shared/LoadParameters.cpp
 Shared/NavigationActionData.cpp


Modified: trunk/Source/WebKit2/ChangeLog (205166 => 205167)

--- trunk/Source/WebKit2/ChangeLog	2016-08-30 01:48:10 UTC (rev 205166)
+++ trunk/Source/WebKit2/ChangeLog	2016-08-30 01:56:37 UTC (rev 205167)
@@ -1,3 +1,9 @@
+2016-08-29  Gyuyoung Kim  
+
+Unreviewed EFL/GTK build fix since r205150
+
+* CMakeLists.txt: Add Shared/FrameInfoData.cpp.
+
 2016-08-29  Anders Carlsson  
 
 Get rid of ChildProcessProxy::fromConnection






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

[webkit-changes] [205166] trunk/Source/JavaScriptCore

2016-08-29 Thread commit-queue

Title: [205166] trunk/Source/_javascript_Core








Revision 205166
Author commit-qu...@webkit.org
Date 2016-08-29 18:48:10 -0700 (Mon, 29 Aug 2016)


Log Message
[JSC] Clean up FTL Capabilities for CompareEq
https://bugs.webkit.org/show_bug.cgi?id=161353

Patch by Benjamin Poulain  on 2016-08-29
Reviewed by Geoffrey Garen.

It looks like we already have code for every case.
This patch removes the tests from FTLCapabilities
and move the generic case last as usual.

* ftl/FTLCapabilities.cpp:
(JSC::FTL::canCompile):
* ftl/FTLLowerDFGToB3.cpp:
(JSC::FTL::DFG::LowerDFGToB3::compileCompareEq):

Modified Paths

trunk/Source/_javascript_Core/ChangeLog
trunk/Source/_javascript_Core/ftl/FTLCapabilities.cpp
trunk/Source/_javascript_Core/ftl/FTLLowerDFGToB3.cpp




Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (205165 => 205166)

--- trunk/Source/_javascript_Core/ChangeLog	2016-08-30 01:16:42 UTC (rev 205165)
+++ trunk/Source/_javascript_Core/ChangeLog	2016-08-30 01:48:10 UTC (rev 205166)
@@ -1,3 +1,19 @@
+2016-08-29  Benjamin Poulain  
+
+[JSC] Clean up FTL Capabilities for CompareEq
+https://bugs.webkit.org/show_bug.cgi?id=161353
+
+Reviewed by Geoffrey Garen.
+
+It looks like we already have code for every case.
+This patch removes the tests from FTLCapabilities
+and move the generic case last as usual.
+
+* ftl/FTLCapabilities.cpp:
+(JSC::FTL::canCompile):
+* ftl/FTLLowerDFGToB3.cpp:
+(JSC::FTL::DFG::LowerDFGToB3::compileCompareEq):
+
 2016-08-29  Keith Miller  
 
 Fix toStringName for Proxies and add support for normal instances


Modified: trunk/Source/_javascript_Core/ftl/FTLCapabilities.cpp (205165 => 205166)

--- trunk/Source/_javascript_Core/ftl/FTLCapabilities.cpp	2016-08-30 01:16:42 UTC (rev 205165)
+++ trunk/Source/_javascript_Core/ftl/FTLCapabilities.cpp	2016-08-30 01:48:10 UTC (rev 205166)
@@ -245,6 +245,7 @@
 case ResolveScope:
 case GetDynamicVar:
 case PutDynamicVar:
+case CompareEq:
 case CompareEqPtr:
 case CompareStrictEq:
 // These are OK.
@@ -363,32 +364,6 @@
 return CannotCompile;
 }
 break;
-case CompareEq:
-if (node->isBinaryUseKind(Int32Use))
-break;
-if (node->isBinaryUseKind(Int52RepUse))
-break;
-if (node->isBinaryUseKind(DoubleRepUse))
-break;
-if (node->isBinaryUseKind(StringIdentUse))
-break;
-if (node->isBinaryUseKind(StringUse))
-break;
-if (node->isBinaryUseKind(SymbolUse))
-break;
-if (node->isBinaryUseKind(ObjectUse))
-break;
-if (node->isBinaryUseKind(UntypedUse))
-break;
-if (node->isBinaryUseKind(BooleanUse))
-break;
-if (node->isBinaryUseKind(ObjectUse, ObjectOrOtherUse))
-break;
-if (node->isBinaryUseKind(ObjectOrOtherUse, ObjectUse))
-break;
-if (node->child1().useKind() == OtherUse || node->child2().useKind() == OtherUse)
-break;
-return CannotCompile;
 case CompareLess:
 case CompareLessEq:
 case CompareGreater:


Modified: trunk/Source/_javascript_Core/ftl/FTLLowerDFGToB3.cpp (205165 => 205166)

--- trunk/Source/_javascript_Core/ftl/FTLLowerDFGToB3.cpp	2016-08-30 01:16:42 UTC (rev 205165)
+++ trunk/Source/_javascript_Core/ftl/FTLLowerDFGToB3.cpp	2016-08-30 01:48:10 UTC (rev 205166)
@@ -4998,15 +4998,6 @@
 compareEqObjectOrOtherToObject(m_node->child1(), m_node->child2());
 return;
 }
-
-if (m_node->isBinaryUseKind(UntypedUse)) {
-nonSpeculativeCompare(
-[&] (LValue left, LValue right) {
-return m_out.equal(left, right);
-},
-operationCompareEq);
-return;
-}
 
 if (m_node->child1().useKind() == OtherUse) {
 ASSERT(!m_interpreter.needsTypeCheck(m_node->child1(), SpecOther));
@@ -5020,7 +5011,12 @@
 return;
 }
 
-DFG_CRASH(m_graph, m_node, "Bad use kinds");
+DFG_ASSERT(m_graph, m_node, m_node->isBinaryUseKind(UntypedUse));
+nonSpeculativeCompare(
+[&] (LValue left, LValue right) {
+return m_out.equal(left, right);
+},
+operationCompareEq);
 }
 
 void compileCompareStrictEq()






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

[webkit-changes] [205165] tags/Safari-603.1.3.0.1/Source

2016-08-29 Thread bshafiei

Title: [205165] tags/Safari-603.1.3.0.1/Source








Revision 205165
Author bshaf...@apple.com
Date 2016-08-29 18:16:42 -0700 (Mon, 29 Aug 2016)


Log Message
Versioning.

Modified Paths

tags/Safari-603.1.3.0.1/Source/_javascript_Core/Configurations/Version.xcconfig
tags/Safari-603.1.3.0.1/Source/WebCore/Configurations/Version.xcconfig
tags/Safari-603.1.3.0.1/Source/WebInspectorUI/Configurations/Version.xcconfig
tags/Safari-603.1.3.0.1/Source/WebKit/mac/Configurations/Version.xcconfig
tags/Safari-603.1.3.0.1/Source/WebKit2/Configurations/Version.xcconfig




Diff

Modified: tags/Safari-603.1.3.0.1/Source/_javascript_Core/Configurations/Version.xcconfig (205164 => 205165)

--- tags/Safari-603.1.3.0.1/Source/_javascript_Core/Configurations/Version.xcconfig	2016-08-30 01:14:19 UTC (rev 205164)
+++ tags/Safari-603.1.3.0.1/Source/_javascript_Core/Configurations/Version.xcconfig	2016-08-30 01:16:42 UTC (rev 205165)
@@ -25,8 +25,8 @@
 MINOR_VERSION = 1;
 TINY_VERSION = 3;
 MICRO_VERSION = 0;
-NANO_VERSION = 0;
-FULL_VERSION = $(MAJOR_VERSION).$(MINOR_VERSION).$(TINY_VERSION);
+NANO_VERSION = 1;
+FULL_VERSION = $(MAJOR_VERSION).$(MINOR_VERSION).$(TINY_VERSION).$(MICRO_VERSION).$(NANO_VERSION);
 
 // The bundle version and short version string are set based on the current build configuration, see below.
 BUNDLE_VERSION = $(BUNDLE_VERSION_$(CONFIGURATION));


Modified: tags/Safari-603.1.3.0.1/Source/WebCore/Configurations/Version.xcconfig (205164 => 205165)

--- tags/Safari-603.1.3.0.1/Source/WebCore/Configurations/Version.xcconfig	2016-08-30 01:14:19 UTC (rev 205164)
+++ tags/Safari-603.1.3.0.1/Source/WebCore/Configurations/Version.xcconfig	2016-08-30 01:16:42 UTC (rev 205165)
@@ -25,8 +25,8 @@
 MINOR_VERSION = 1;
 TINY_VERSION = 3;
 MICRO_VERSION = 0;
-NANO_VERSION = 0;
-FULL_VERSION = $(MAJOR_VERSION).$(MINOR_VERSION).$(TINY_VERSION);
+NANO_VERSION = 1;
+FULL_VERSION = $(MAJOR_VERSION).$(MINOR_VERSION).$(TINY_VERSION).$(MICRO_VERSION).$(NANO_VERSION);
 
 // The bundle version and short version string are set based on the current build configuration, see below.
 BUNDLE_VERSION = $(BUNDLE_VERSION_$(CONFIGURATION));


Modified: tags/Safari-603.1.3.0.1/Source/WebInspectorUI/Configurations/Version.xcconfig (205164 => 205165)

--- tags/Safari-603.1.3.0.1/Source/WebInspectorUI/Configurations/Version.xcconfig	2016-08-30 01:14:19 UTC (rev 205164)
+++ tags/Safari-603.1.3.0.1/Source/WebInspectorUI/Configurations/Version.xcconfig	2016-08-30 01:16:42 UTC (rev 205165)
@@ -2,8 +2,8 @@
 MINOR_VERSION = 1;
 TINY_VERSION = 3;
 MICRO_VERSION = 0;
-NANO_VERSION = 0;
-FULL_VERSION = $(MAJOR_VERSION).$(MINOR_VERSION).$(TINY_VERSION);
+NANO_VERSION = 1;
+FULL_VERSION = $(MAJOR_VERSION).$(MINOR_VERSION).$(TINY_VERSION).$(MICRO_VERSION).$(NANO_VERSION);
 
 // The system version prefix is based on the current system version.
 SYSTEM_VERSION_PREFIX[sdk=iphone*] = 8;


Modified: tags/Safari-603.1.3.0.1/Source/WebKit/mac/Configurations/Version.xcconfig (205164 => 205165)

--- tags/Safari-603.1.3.0.1/Source/WebKit/mac/Configurations/Version.xcconfig	2016-08-30 01:14:19 UTC (rev 205164)
+++ tags/Safari-603.1.3.0.1/Source/WebKit/mac/Configurations/Version.xcconfig	2016-08-30 01:16:42 UTC (rev 205165)
@@ -25,8 +25,8 @@
 MINOR_VERSION = 1;
 TINY_VERSION = 3;
 MICRO_VERSION = 0;
-NANO_VERSION = 0;
-FULL_VERSION = $(MAJOR_VERSION).$(MINOR_VERSION).$(TINY_VERSION);
+NANO_VERSION = 1;
+FULL_VERSION = $(MAJOR_VERSION).$(MINOR_VERSION).$(TINY_VERSION).$(MICRO_VERSION).$(NANO_VERSION);
 
 // The bundle version and short version string are set based on the current build configuration, see below.
 BUNDLE_VERSION = $(BUNDLE_VERSION_$(CONFIGURATION));


Modified: tags/Safari-603.1.3.0.1/Source/WebKit2/Configurations/Version.xcconfig (205164 => 205165)

--- tags/Safari-603.1.3.0.1/Source/WebKit2/Configurations/Version.xcconfig	2016-08-30 01:14:19 UTC (rev 205164)
+++ tags/Safari-603.1.3.0.1/Source/WebKit2/Configurations/Version.xcconfig	2016-08-30 01:16:42 UTC (rev 205165)
@@ -25,8 +25,8 @@
 MINOR_VERSION = 1;
 TINY_VERSION = 3;
 MICRO_VERSION = 0;
-NANO_VERSION = 0;
-FULL_VERSION = $(MAJOR_VERSION).$(MINOR_VERSION).$(TINY_VERSION);
+NANO_VERSION = 1;
+FULL_VERSION = $(MAJOR_VERSION).$(MINOR_VERSION).$(TINY_VERSION).$(MICRO_VERSION).$(NANO_VERSION);
 
 // The bundle version and short version string are set based on the current build configuration, see below.
 BUNDLE_VERSION = $(BUNDLE_VERSION_$(CONFIGURATION));






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

[webkit-changes] [205164] tags/Safari-603.1.3.0.1/

2016-08-29 Thread bshafiei

Title: [205164] tags/Safari-603.1.3.0.1/








Revision 205164
Author bshaf...@apple.com
Date 2016-08-29 18:14:19 -0700 (Mon, 29 Aug 2016)


Log Message
New tag.

Added Paths

tags/Safari-603.1.3.0.1/




Diff




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

[webkit-changes] [205163] trunk/Source/WebCore

2016-08-29 Thread bfulgham

Title: [205163] trunk/Source/WebCore








Revision 205163
Author bfulg...@apple.com
Date 2016-08-29 17:44:21 -0700 (Mon, 29 Aug 2016)


Log Message
Avoid holding GlyphData in MathOperator
https://bugs.webkit.org/show_bug.cgi?id=161256


Reviewed by Myles C. Maxfield.

Do not cache GlyphData in MathOperator elements, because the fonts referenced in the
GlyphData may be purged during low-memory conditions. Instead, we should store either
the relevant CodePoint, or the fallback Glyph (for the current system font).

Added an initialization function for GlyphAssemblyData, since unions containing structs
do not properly call constructors, resulting in garbage font/glyph content.

No new tests. Changes are covered by existing MathML test suite.

* rendering/mathml/MathOperator.cpp:
(WebCore::MathOperator::GlyphAssemblyData::initialize): Added.
(WebCore::MathOperator::MathOperator): Initialize m_assembly/m_variant.
(WebCore::MathOperator::setSizeVariant): Only store the glyph, not the font.
(WebCore::glyphDataForCodePointOrFallbackGlyph): Added helper function.
(WebCore::MathOperator::setGlyphAssembly): Do not rely on stored GlyphData.
(WebCore::MathOperator::calculateGlyphAssemblyFallback): Remove unneeded argument. Check
if a fallback glyph is being used and remember for later.
(WebCore::MathOperator::calculateStretchyData): Do not rely on stored GlyphData.
(WebCore::MathOperator::fillWithVerticalExtensionGlyph): Ditto.
(WebCore::MathOperator::fillWithHorizontalExtensionGlyph): Ditto.
(WebCore::MathOperator::paintVerticalGlyphAssembly): Ditto.
(WebCore::MathOperator::paintHorizontalGlyphAssembly): Ditto.
(WebCore::MathOperator::paint): Ditto.
* rendering/mathml/MathOperator.h:
(WebCore::MathOperator::GlyphAssemblyData::hasExtension): Added.
(WebCore::MathOperator::GlyphAssemblyData::hasMiddle): Added.
(WebCore::MathOperator::MathOperator): Deleted.

Modified Paths

trunk/Source/WebCore/ChangeLog
trunk/Source/WebCore/rendering/mathml/MathOperator.cpp
trunk/Source/WebCore/rendering/mathml/MathOperator.h




Diff

Modified: trunk/Source/WebCore/ChangeLog (205162 => 205163)

--- trunk/Source/WebCore/ChangeLog	2016-08-30 00:31:08 UTC (rev 205162)
+++ trunk/Source/WebCore/ChangeLog	2016-08-30 00:44:21 UTC (rev 205163)
@@ -1,3 +1,39 @@
+2016-08-29  Brent Fulgham  
+
+Avoid holding GlyphData in MathOperator
+https://bugs.webkit.org/show_bug.cgi?id=161256
+
+
+Reviewed by Myles C. Maxfield.
+
+Do not cache GlyphData in MathOperator elements, because the fonts referenced in the
+GlyphData may be purged during low-memory conditions. Instead, we should store either
+the relevant CodePoint, or the fallback Glyph (for the current system font).
+
+Added an initialization function for GlyphAssemblyData, since unions containing structs
+do not properly call constructors, resulting in garbage font/glyph content.
+
+No new tests. Changes are covered by existing MathML test suite.
+
+* rendering/mathml/MathOperator.cpp:
+(WebCore::MathOperator::GlyphAssemblyData::initialize): Added.
+(WebCore::MathOperator::MathOperator): Initialize m_assembly/m_variant.
+(WebCore::MathOperator::setSizeVariant): Only store the glyph, not the font.
+(WebCore::glyphDataForCodePointOrFallbackGlyph): Added helper function.
+(WebCore::MathOperator::setGlyphAssembly): Do not rely on stored GlyphData.
+(WebCore::MathOperator::calculateGlyphAssemblyFallback): Remove unneeded argument. Check
+if a fallback glyph is being used and remember for later.
+(WebCore::MathOperator::calculateStretchyData): Do not rely on stored GlyphData.
+(WebCore::MathOperator::fillWithVerticalExtensionGlyph): Ditto.
+(WebCore::MathOperator::fillWithHorizontalExtensionGlyph): Ditto.
+(WebCore::MathOperator::paintVerticalGlyphAssembly): Ditto.
+(WebCore::MathOperator::paintHorizontalGlyphAssembly): Ditto.
+(WebCore::MathOperator::paint): Ditto.
+* rendering/mathml/MathOperator.h:
+(WebCore::MathOperator::GlyphAssemblyData::hasExtension): Added.
+(WebCore::MathOperator::GlyphAssemblyData::hasMiddle): Added.
+(WebCore::MathOperator::MathOperator): Deleted.
+
 2016-08-29  Dean Jackson  
 
 JSON.stringify returns empty when used with performance.timing.


Modified: trunk/Source/WebCore/rendering/mathml/MathOperator.cpp (205162 => 205163)

--- trunk/Source/WebCore/rendering/mathml/MathOperator.cpp	2016-08-30 00:31:08 UTC (rev 205162)
+++ trunk/Source/WebCore/rendering/mathml/MathOperator.cpp	2016-08-30 00:44:21 UTC (rev 205163)
@@ -1,5 +1,6 @@
 /*
  * Copyright (C) 2016 Igalia S.L. All rights reserved.
+ * Copyright (C) 2016 Apple Inc.  All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
@@ -85,6 +86,24 @@
 { 0x222b, 0x2320, 0x23ae, 0x2321

[webkit-changes] [205162] tags/Safari-601.7.8/

2016-08-29 Thread bshafiei

Title: [205162] tags/Safari-601.7.8/








Revision 205162
Author bshaf...@apple.com
Date 2016-08-29 17:31:08 -0700 (Mon, 29 Aug 2016)


Log Message
New tag.

Added Paths

tags/Safari-601.7.8/




Diff




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

[webkit-changes] [205161] trunk

2016-08-29 Thread dino

Title: [205161] trunk








Revision 205161
Author d...@apple.com
Date 2016-08-29 17:15:17 -0700 (Mon, 29 Aug 2016)


Log Message
JSON.stringify returns empty when used with performance.timing.
https://bugs.webkit.org/show_bug.cgi?id=151813


Reviewed by Sam Weinig.

Source/WebCore:

While we are waiting for our bindings generator to handle
"serializer", add a custom toJSON function that allows
performance.timing to be stringified.

Test: fast/dom/webtiming-performance-timing-stringify.html

* CMakeLists.txt: New file added.
* WebCore.xcodeproj/project.pbxproj: New file added.
* bindings/js/JSPerformanceTimingCustom.cpp: Added. Implements toJSON.
(WebCore::JSPerformanceTiming::toJSON):
* page/PerformanceTiming.idl: Add toJSON().

LayoutTests:

* fast/dom/Window/window-properties-performance-expected.txt:
* fast/dom/Window/window-properties-performance-resource-timing-expected.txt:
* fast/dom/webtiming-document-open-expected.txt:
* fast/dom/webtiming-navigate-within-document-expected.txt:
* fast/dom/webtiming-performance-timing-stringify-expected.txt: Added.
* fast/dom/webtiming-performance-timing-stringify.html: Added.
* http/tests/misc/webtiming-cross-origin-and-back1-expected.txt:
* http/tests/misc/webtiming-cross-origin-redirect-expected.txt:
* http/tests/misc/webtiming-no-origin-expected.txt:

Modified Paths

trunk/LayoutTests/ChangeLog
trunk/LayoutTests/fast/dom/Window/window-properties-performance-expected.txt
trunk/LayoutTests/fast/dom/Window/window-properties-performance-resource-timing-expected.txt
trunk/LayoutTests/fast/dom/webtiming-document-open-expected.txt
trunk/LayoutTests/fast/dom/webtiming-navigate-within-document-expected.txt
trunk/LayoutTests/http/tests/misc/webtiming-cross-origin-and-back1-expected.txt
trunk/LayoutTests/http/tests/misc/webtiming-cross-origin-redirect-expected.txt
trunk/LayoutTests/http/tests/misc/webtiming-no-origin-expected.txt
trunk/Source/WebCore/CMakeLists.txt
trunk/Source/WebCore/ChangeLog
trunk/Source/WebCore/WebCore.xcodeproj/project.pbxproj
trunk/Source/WebCore/page/PerformanceTiming.idl


Added Paths

trunk/LayoutTests/fast/dom/webtiming-performance-timing-stringify-expected.txt
trunk/LayoutTests/fast/dom/webtiming-performance-timing-stringify.html
trunk/Source/WebCore/bindings/js/JSPerformanceTimingCustom.cpp




Diff

Modified: trunk/LayoutTests/ChangeLog (205160 => 205161)

--- trunk/LayoutTests/ChangeLog	2016-08-29 23:54:25 UTC (rev 205160)
+++ trunk/LayoutTests/ChangeLog	2016-08-30 00:15:17 UTC (rev 205161)
@@ -1,3 +1,21 @@
+2016-08-29  Dean Jackson  
+
+JSON.stringify returns empty when used with performance.timing.
+https://bugs.webkit.org/show_bug.cgi?id=151813
+
+
+Reviewed by Sam Weinig.
+
+* fast/dom/Window/window-properties-performance-expected.txt:
+* fast/dom/Window/window-properties-performance-resource-timing-expected.txt:
+* fast/dom/webtiming-document-open-expected.txt:
+* fast/dom/webtiming-navigate-within-document-expected.txt:
+* fast/dom/webtiming-performance-timing-stringify-expected.txt: Added.
+* fast/dom/webtiming-performance-timing-stringify.html: Added.
+* http/tests/misc/webtiming-cross-origin-and-back1-expected.txt:
+* http/tests/misc/webtiming-cross-origin-redirect-expected.txt:
+* http/tests/misc/webtiming-no-origin-expected.txt:
+
 2016-08-29  Chris Dumez  
 
 Regression(r204923): It should be possible to set 'Location.href' cross origin


Modified: trunk/LayoutTests/fast/dom/Window/window-properties-performance-expected.txt (205160 => 205161)

--- trunk/LayoutTests/fast/dom/Window/window-properties-performance-expected.txt	2016-08-29 23:54:25 UTC (rev 205160)
+++ trunk/LayoutTests/fast/dom/Window/window-properties-performance-expected.txt	2016-08-30 00:15:17 UTC (rev 205161)
@@ -32,6 +32,7 @@
 window.performance.timing.responseEnd [number]
 window.performance.timing.responseStart [number]
 window.performance.timing.secureConnectionStart [number]
+window.performance.timing.toJSON [function]
 window.performance.timing.unloadEventEnd [number]
 window.performance.timing.unloadEventStart [number]
 window.performance.timing [printed above as window.performance.timing]


Modified: trunk/LayoutTests/fast/dom/Window/window-properties-performance-resource-timing-expected.txt (205160 => 205161)

--- trunk/LayoutTests/fast/dom/Window/window-properties-performance-resource-timing-expected.txt	2016-08-29 23:54:25 UTC (rev 205160)
+++ trunk/LayoutTests/fast/dom/Window/window-properties-performance-resource-timing-expected.txt	2016-08-30 00:15:17 UTC (rev 205161)
@@ -38,6 +38,7 @@
 window.performance.timing.responseEnd [number]
 window.performance.timing.responseStart [number]
 window.performance.timing.secureConnectionStart [number]
+window.performance.timing.toJSON [function]
 window.performance.timing.unloadEventEnd [number]
 window.performance.timing.unloadEventStart [number]
 window.performance.timing [printed above as window.

[webkit-changes] [205160] trunk/Tools

2016-08-29 Thread aakash_jain

Title: [205160] trunk/Tools








Revision 205160
Author aakash_j...@apple.com
Date 2016-08-29 16:54:25 -0700 (Mon, 29 Aug 2016)


Log Message
EWS patch status page should indicate bot corresponding to each status message
https://bugs.webkit.org/show_bug.cgi?id=161280

Reviewed by Alexey Proskuryakov.

* QueueStatusServer/templates/patch.html: Formatting changes.

Modified Paths

trunk/Tools/ChangeLog
trunk/Tools/QueueStatusServer/templates/patch.html




Diff

Modified: trunk/Tools/ChangeLog (205159 => 205160)

--- trunk/Tools/ChangeLog	2016-08-29 23:23:56 UTC (rev 205159)
+++ trunk/Tools/ChangeLog	2016-08-29 23:54:25 UTC (rev 205160)
@@ -1,5 +1,14 @@
 2016-08-29  Aakash Jain  
 
+EWS patch status page should indicate bot corresponding to each status message
+https://bugs.webkit.org/show_bug.cgi?id=161280
+
+Reviewed by Alexey Proskuryakov.
+
+* QueueStatusServer/templates/patch.html: Formatting changes.
+
+2016-08-29  Aakash Jain  
+
 EWS should indicate which bot processed the patch
 https://bugs.webkit.org/show_bug.cgi?id=161222
 


Modified: trunk/Tools/QueueStatusServer/templates/patch.html (205159 => 205160)

--- trunk/Tools/QueueStatusServer/templates/patch.html	2016-08-29 23:23:56 UTC (rev 205159)
+++ trunk/Tools/QueueStatusServer/templates/patch.html	2016-08-29 23:54:25 UTC (rev 205160)
@@ -12,7 +12,7 @@
   {{ queue_name }}
   {% for status in statuses %}
 
-  ({{ status.bot_id }}): 
+  {{ status.bot_id }} 
   {{ status.message|force_escape|urlize|webkit_linkify|safe }}{% if status.results_file %}
   [{{ status.key.id|results_link|safe }}]{% endif %}
   {{ status.date|timesince }} ago






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

[webkit-changes] [205159] trunk/Source/WebKit2

2016-08-29 Thread andersca

Title: [205159] trunk/Source/WebKit2








Revision 205159
Author ander...@apple.com
Date 2016-08-29 16:23:56 -0700 (Mon, 29 Aug 2016)


Log Message
Get rid of ChildProcessProxy::fromConnection
https://bugs.webkit.org/show_bug.cgi?id=161357

Reviewed by Tim Horton.

* UIProcess/ChildProcessProxy.cpp:
(WebKit::ChildProcessProxy::fromConnection): Deleted.
* UIProcess/ChildProcessProxy.h:
* UIProcess/WebProcessProxy.h:
(WebKit::WebProcessProxy::fromConnection): Deleted.

Modified Paths

trunk/Source/WebKit2/ChangeLog
trunk/Source/WebKit2/UIProcess/ChildProcessProxy.cpp
trunk/Source/WebKit2/UIProcess/ChildProcessProxy.h
trunk/Source/WebKit2/UIProcess/WebProcessProxy.h




Diff

Modified: trunk/Source/WebKit2/ChangeLog (205158 => 205159)

--- trunk/Source/WebKit2/ChangeLog	2016-08-29 23:19:14 UTC (rev 205158)
+++ trunk/Source/WebKit2/ChangeLog	2016-08-29 23:23:56 UTC (rev 205159)
@@ -1,5 +1,18 @@
 2016-08-29  Anders Carlsson  
 
+Get rid of ChildProcessProxy::fromConnection
+https://bugs.webkit.org/show_bug.cgi?id=161357
+
+Reviewed by Tim Horton.
+
+* UIProcess/ChildProcessProxy.cpp:
+(WebKit::ChildProcessProxy::fromConnection): Deleted.
+* UIProcess/ChildProcessProxy.h:
+* UIProcess/WebProcessProxy.h:
+(WebKit::WebProcessProxy::fromConnection): Deleted.
+
+2016-08-29  Anders Carlsson  
+
 Remove the last uses of WebProcessProxy::fromConnection
 https://bugs.webkit.org/show_bug.cgi?id=161355
 


Modified: trunk/Source/WebKit2/UIProcess/ChildProcessProxy.cpp (205158 => 205159)

--- trunk/Source/WebKit2/UIProcess/ChildProcessProxy.cpp	2016-08-29 23:19:14 UTC (rev 205158)
+++ trunk/Source/WebKit2/UIProcess/ChildProcessProxy.cpp	2016-08-29 23:23:56 UTC (rev 205159)
@@ -46,16 +46,6 @@
 }
 }
 
-ChildProcessProxy* ChildProcessProxy::fromConnection(IPC::Connection* connection)
-{
-ASSERT(connection);
-
-ChildProcessProxy* childProcessProxy = static_cast(connection->client());
-ASSERT(childProcessProxy->connection() == connection);
-
-return childProcessProxy;
-}
-
 void ChildProcessProxy::getLaunchOptions(ProcessLauncher::LaunchOptions& launchOptions)
 {
 if (const char* userDirectorySuffix = getenv("DIRHELPER_USER_DIR_SUFFIX"))


Modified: trunk/Source/WebKit2/UIProcess/ChildProcessProxy.h (205158 => 205159)

--- trunk/Source/WebKit2/UIProcess/ChildProcessProxy.h	2016-08-29 23:19:14 UTC (rev 205158)
+++ trunk/Source/WebKit2/UIProcess/ChildProcessProxy.h	2016-08-29 23:23:56 UTC (rev 205159)
@@ -40,9 +40,6 @@
 ChildProcessProxy();
 virtual ~ChildProcessProxy();
 
-// FIXME: This function does an unchecked upcast, and it is only used in a deprecated code path. Would like to get rid of it.
-static ChildProcessProxy* fromConnection(IPC::Connection*);
-
 void connect();
 void terminate();
 


Modified: trunk/Source/WebKit2/UIProcess/WebProcessProxy.h (205158 => 205159)

--- trunk/Source/WebKit2/UIProcess/WebProcessProxy.h	2016-08-29 23:19:14 UTC (rev 205158)
+++ trunk/Source/WebKit2/UIProcess/WebProcessProxy.h	2016-08-29 23:23:56 UTC (rev 205159)
@@ -73,11 +73,6 @@
 static Ref create(WebProcessPool&);
 ~WebProcessProxy();
 
-static WebProcessProxy* fromConnection(IPC::Connection* connection)
-{
-return static_cast(ChildProcessProxy::fromConnection(connection));
-}
-
 WebConnection* webConnection() const { return m_webConnection.get(); }
 
 WebProcessPool& processPool() { return m_processPool; }






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

[webkit-changes] [205158] tags/Safari-600.8.7.1/Source/JavaScriptCore

2016-08-29 Thread bshafiei

Title: [205158] tags/Safari-600.8.7.1/Source/_javascript_Core








Revision 205158
Author bshaf...@apple.com
Date 2016-08-29 16:19:14 -0700 (Mon, 29 Aug 2016)


Log Message
Merge r204572.  rdar://problem/28059311

Modified Paths

tags/Safari-600.8.7.1/Source/_javascript_Core/ChangeLog
tags/Safari-600.8.7.1/Source/_javascript_Core/runtime/ArgList.cpp
tags/Safari-600.8.7.1/Source/_javascript_Core/runtime/ArgList.h




Diff

Modified: tags/Safari-600.8.7.1/Source/_javascript_Core/ChangeLog (205157 => 205158)

--- tags/Safari-600.8.7.1/Source/_javascript_Core/ChangeLog	2016-08-29 23:18:25 UTC (rev 205157)
+++ tags/Safari-600.8.7.1/Source/_javascript_Core/ChangeLog	2016-08-29 23:19:14 UTC (rev 205158)
@@ -1,3 +1,35 @@
+2016-08-29  Babak Shafiei  
+
+Merge r204572.
+
+2016-08-17  Geoffrey Garen  
+
+Fixed a potential bug in MarkedArgumentBuffer.
+https://bugs.webkit.org/show_bug.cgi?id=160948
+
+
+Reviewed by Oliver Hunt.
+
+I haven't been able to produce an observable test case after some trying.
+
+* runtime/ArgList.cpp:
+(JSC::MarkedArgumentBuffer::addMarkSet): New helper function -- I broke
+this out from existing code for clarity, but the behavior is the same.
+
+(JSC::MarkedArgumentBuffer::expandCapacity): Ditto.
+
+(JSC::MarkedArgumentBuffer::slowAppend): Always addMarkSet() on the slow
+path. This is faster than the old linear scan, and I think it might
+avoid cases the old scan could miss.
+
+* runtime/ArgList.h:
+(JSC::MarkedArgumentBuffer::append): Account for the case where someone
+has called clear() or removeLast().
+
+(JSC::MarkedArgumentBuffer::mallocBase): No behavior change -- but it's
+clearer to test the buffers directly instead of inferring what they
+might be based on capacity.
+
 2015-07-27  Babak Shafiei  
 
 Roll out r182829.


Modified: tags/Safari-600.8.7.1/Source/_javascript_Core/runtime/ArgList.cpp (205157 => 205158)

--- tags/Safari-600.8.7.1/Source/_javascript_Core/runtime/ArgList.cpp	2016-08-29 23:18:25 UTC (rev 205157)
+++ tags/Safari-600.8.7.1/Source/_javascript_Core/runtime/ArgList.cpp	2016-08-29 23:19:14 UTC (rev 205158)
@@ -30,6 +30,19 @@
 
 namespace JSC {
 
+void MarkedArgumentBuffer::addMarkSet(JSValue v)
+{
+if (m_markSet)
+return;
+
+Heap* heap = Heap::heap(v);
+if (!heap)
+return;
+
+m_markSet = &heap->markListSet();
+m_markSet->add(this);
+}
+
 void ArgList::getSlice(int startIndex, ArgList& result) const
 {
 if (startIndex <= 0 || startIndex >= m_argCount) {
@@ -51,12 +64,14 @@
 }
 }
 
-void MarkedArgumentBuffer::slowAppend(JSValue v)
+void MarkedArgumentBuffer::expandCapacity()
 {
 int newCapacity = m_capacity * 4;
 EncodedJSValue* newBuffer = new EncodedJSValue[newCapacity];
-for (int i = 0; i < m_capacity; ++i)
+for (int i = 0; i < m_capacity; ++i) {
 newBuffer[i] = m_buffer[i];
+addMarkSet(JSValue::decode(m_buffer[i]));
+}
 
 if (EncodedJSValue* base = mallocBase())
 delete [] base;
@@ -63,27 +78,16 @@
 
 m_buffer = newBuffer;
 m_capacity = newCapacity;
+}
 
+void MarkedArgumentBuffer::slowAppend(JSValue v)
+{
+if (m_size >= m_capacity)
+expandCapacity();
+
 slotFor(m_size) = JSValue::encode(v);
 ++m_size;
-
-if (m_markSet)
-return;
-
-// As long as our size stays within our Vector's inline 
-// capacity, all our values are allocated on the stack, and 
-// therefore don't need explicit marking. Once our size exceeds
-// our Vector's inline capacity, though, our values move to the 
-// heap, where they do need explicit marking.
-for (int i = 0; i < m_size; ++i) {
-Heap* heap = Heap::heap(JSValue::decode(slotFor(i)));
-if (!heap)
-continue;
-
-m_markSet = &heap->markListSet();
-m_markSet->add(this);
-break;
-}
+addMarkSet(v);
 }
 
 } // namespace JSC


Modified: tags/Safari-600.8.7.1/Source/_javascript_Core/runtime/ArgList.h (205157 => 205158)

--- tags/Safari-600.8.7.1/Source/_javascript_Core/runtime/ArgList.h	2016-08-29 23:18:25 UTC (rev 205157)
+++ tags/Safari-600.8.7.1/Source/_javascript_Core/runtime/ArgList.h	2016-08-29 23:19:14 UTC (rev 205158)
@@ -78,7 +78,7 @@
 
 void append(JSValue v)
 {
-if (m_size >= m_capacity)
+if (m_size >= m_capacity || mallocBase())
 return slowAppend(v);
 
 slotFor(m_size) = JSValue::encode(v);
@@ -100,6 +100,10 @@
 static void markLists(HeapRootVisitor&, ListSet&);
 
 private:
+void expandCapacity();
+
+void addMarkSet(JSValue);
+
 JS_EXPORT_PRIVATE void slowAppend(JSValue);
 
 EncodedJSValue& slotFor(int item) const
@@ -109,7 +113,7 @@
 
 EncodedJSValue* mallocBase()
 {
-if (m_capacity == s

[webkit-changes] [205157] tags/Safari-600.8.7.1/Source

2016-08-29 Thread bshafiei

Title: [205157] tags/Safari-600.8.7.1/Source








Revision 205157
Author bshaf...@apple.com
Date 2016-08-29 16:18:25 -0700 (Mon, 29 Aug 2016)


Log Message
Versioning.

Modified Paths

tags/Safari-600.8.7.1/Source/_javascript_Core/Configurations/Version.xcconfig
tags/Safari-600.8.7.1/Source/WebCore/Configurations/Version.xcconfig
tags/Safari-600.8.7.1/Source/WebInspectorUI/Configurations/Version.xcconfig
tags/Safari-600.8.7.1/Source/WebKit/mac/Configurations/Version.xcconfig
tags/Safari-600.8.7.1/Source/WebKit2/Configurations/Version.xcconfig




Diff

Modified: tags/Safari-600.8.7.1/Source/_javascript_Core/Configurations/Version.xcconfig (205156 => 205157)

--- tags/Safari-600.8.7.1/Source/_javascript_Core/Configurations/Version.xcconfig	2016-08-29 23:10:56 UTC (rev 205156)
+++ tags/Safari-600.8.7.1/Source/_javascript_Core/Configurations/Version.xcconfig	2016-08-29 23:18:25 UTC (rev 205157)
@@ -24,9 +24,9 @@
 MAJOR_VERSION = 600;
 MINOR_VERSION = 8;
 TINY_VERSION = 7;
-MICRO_VERSION = 0;
+MICRO_VERSION = 1;
 NANO_VERSION = 0;
-FULL_VERSION = $(MAJOR_VERSION).$(MINOR_VERSION).$(TINY_VERSION);
+FULL_VERSION = $(MAJOR_VERSION).$(MINOR_VERSION).$(TINY_VERSION).$(MICRO_VERSION);
 
 // The bundle version and short version string are set based on the current build configuration, see below.
 BUNDLE_VERSION = $(BUNDLE_VERSION_$(CONFIGURATION));


Modified: tags/Safari-600.8.7.1/Source/WebCore/Configurations/Version.xcconfig (205156 => 205157)

--- tags/Safari-600.8.7.1/Source/WebCore/Configurations/Version.xcconfig	2016-08-29 23:10:56 UTC (rev 205156)
+++ tags/Safari-600.8.7.1/Source/WebCore/Configurations/Version.xcconfig	2016-08-29 23:18:25 UTC (rev 205157)
@@ -24,9 +24,9 @@
 MAJOR_VERSION = 600;
 MINOR_VERSION = 8;
 TINY_VERSION = 7;
-MICRO_VERSION = 0;
+MICRO_VERSION = 1;
 NANO_VERSION = 0;
-FULL_VERSION = $(MAJOR_VERSION).$(MINOR_VERSION).$(TINY_VERSION);
+FULL_VERSION = $(MAJOR_VERSION).$(MINOR_VERSION).$(TINY_VERSION).$(MICRO_VERSION);
 
 // The bundle version and short version string are set based on the current build configuration, see below.
 BUNDLE_VERSION = $(BUNDLE_VERSION_$(CONFIGURATION));


Modified: tags/Safari-600.8.7.1/Source/WebInspectorUI/Configurations/Version.xcconfig (205156 => 205157)

--- tags/Safari-600.8.7.1/Source/WebInspectorUI/Configurations/Version.xcconfig	2016-08-29 23:10:56 UTC (rev 205156)
+++ tags/Safari-600.8.7.1/Source/WebInspectorUI/Configurations/Version.xcconfig	2016-08-29 23:18:25 UTC (rev 205157)
@@ -1,9 +1,9 @@
 MAJOR_VERSION = 600;
 MINOR_VERSION = 8;
 TINY_VERSION = 7;
-MICRO_VERSION = 0;
+MICRO_VERSION = 1;
 NANO_VERSION = 0;
-FULL_VERSION = $(MAJOR_VERSION).$(MINOR_VERSION).$(TINY_VERSION);
+FULL_VERSION = $(MAJOR_VERSION).$(MINOR_VERSION).$(TINY_VERSION).$(MICRO_VERSION);
 
 // The system version prefix is based on the current system version.
 SYSTEM_VERSION_PREFIX = $(SYSTEM_VERSION_PREFIX_$(PLATFORM_NAME));


Modified: tags/Safari-600.8.7.1/Source/WebKit/mac/Configurations/Version.xcconfig (205156 => 205157)

--- tags/Safari-600.8.7.1/Source/WebKit/mac/Configurations/Version.xcconfig	2016-08-29 23:10:56 UTC (rev 205156)
+++ tags/Safari-600.8.7.1/Source/WebKit/mac/Configurations/Version.xcconfig	2016-08-29 23:18:25 UTC (rev 205157)
@@ -24,9 +24,9 @@
 MAJOR_VERSION = 600;
 MINOR_VERSION = 8;
 TINY_VERSION = 7;
-MICRO_VERSION = 0;
+MICRO_VERSION = 1;
 NANO_VERSION = 0;
-FULL_VERSION = $(MAJOR_VERSION).$(MINOR_VERSION).$(TINY_VERSION);
+FULL_VERSION = $(MAJOR_VERSION).$(MINOR_VERSION).$(TINY_VERSION).$(MICRO_VERSION);
 
 // The bundle version and short version string are set based on the current build configuration, see below.
 BUNDLE_VERSION = $(BUNDLE_VERSION_$(CONFIGURATION));


Modified: tags/Safari-600.8.7.1/Source/WebKit2/Configurations/Version.xcconfig (205156 => 205157)

--- tags/Safari-600.8.7.1/Source/WebKit2/Configurations/Version.xcconfig	2016-08-29 23:10:56 UTC (rev 205156)
+++ tags/Safari-600.8.7.1/Source/WebKit2/Configurations/Version.xcconfig	2016-08-29 23:18:25 UTC (rev 205157)
@@ -24,9 +24,9 @@
 MAJOR_VERSION = 600;
 MINOR_VERSION = 8;
 TINY_VERSION = 7;
-MICRO_VERSION = 0;
+MICRO_VERSION = 1;
 NANO_VERSION = 0;
-FULL_VERSION = $(MAJOR_VERSION).$(MINOR_VERSION).$(TINY_VERSION);
+FULL_VERSION = $(MAJOR_VERSION).$(MINOR_VERSION).$(TINY_VERSION).$(MICRO_VERSION);
 
 // The bundle version and short version string are set based on the current build configuration, see below.
 BUNDLE_VERSION = $(BUNDLE_VERSION_$(CONFIGURATION));






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

[webkit-changes] [205156] trunk/Source/WebKit2

2016-08-29 Thread andersca

Title: [205156] trunk/Source/WebKit2








Revision 205156
Author ander...@apple.com
Date 2016-08-29 16:10:56 -0700 (Mon, 29 Aug 2016)


Log Message
Remove the last uses of WebProcessProxy::fromConnection
https://bugs.webkit.org/show_bug.cgi?id=161355

Reviewed by Tim Horton.

* UIProcess/WebProcessPool.cpp:
(WebKit::webProcessProxyFromConnection):
(WebKit::WebProcessPool::handleMessage):
(WebKit::WebProcessPool::handleSynchronousMessage):
(WebKit::WebProcessPool::startedUsingGamepads):
(WebKit::WebProcessPool::stoppedUsingGamepads):

Modified Paths

trunk/Source/WebKit2/ChangeLog
trunk/Source/WebKit2/UIProcess/WebProcessPool.cpp




Diff

Modified: trunk/Source/WebKit2/ChangeLog (205155 => 205156)

--- trunk/Source/WebKit2/ChangeLog	2016-08-29 23:07:20 UTC (rev 205155)
+++ trunk/Source/WebKit2/ChangeLog	2016-08-29 23:10:56 UTC (rev 205156)
@@ -1,5 +1,19 @@
 2016-08-29  Anders Carlsson  
 
+Remove the last uses of WebProcessProxy::fromConnection
+https://bugs.webkit.org/show_bug.cgi?id=161355
+
+Reviewed by Tim Horton.
+
+* UIProcess/WebProcessPool.cpp:
+(WebKit::webProcessProxyFromConnection):
+(WebKit::WebProcessPool::handleMessage):
+(WebKit::WebProcessPool::handleSynchronousMessage):
+(WebKit::WebProcessPool::startedUsingGamepads):
+(WebKit::WebProcessPool::stoppedUsingGamepads):
+
+2016-08-29  Anders Carlsson  
+
 Get rid of another use of WebProcessProxy::fromConnection
 https://bugs.webkit.org/show_bug.cgi?id=161331
 


Modified: trunk/Source/WebKit2/UIProcess/WebProcessPool.cpp (205155 => 205156)

--- trunk/Source/WebKit2/UIProcess/WebProcessPool.cpp	2016-08-29 23:07:20 UTC (rev 205155)
+++ trunk/Source/WebKit2/UIProcess/WebProcessPool.cpp	2016-08-29 23:10:56 UTC (rev 205156)
@@ -1216,9 +1216,20 @@
 m_networkProcess->send(Messages::NetworkProcess::GetNetworkProcessStatistics(requestID), 0);
 }
 
+static WebProcessProxy* webProcessProxyFromConnection(IPC::Connection& connection, const Vector>& processes)
+{
+for (auto& process : processes) {
+if (process->connection() == &connection)
+return process.get();
+}
+
+// FIXME: Can this ever return null?
+return nullptr;
+}
+
 void WebProcessPool::handleMessage(IPC::Connection& connection, const String& messageName, const WebKit::UserData& messageBody)
 {
-auto* webProcessProxy = WebProcessProxy::fromConnection(&connection);
+auto* webProcessProxy = webProcessProxyFromConnection(connection, m_processes);
 if (!webProcessProxy)
 return;
 m_injectedBundleClient.didReceiveMessageFromInjectedBundle(this, messageName, webProcessProxy->transformHandlesToObjects(messageBody.object()).get());
@@ -1226,7 +1237,7 @@
 
 void WebProcessPool::handleSynchronousMessage(IPC::Connection& connection, const String& messageName, const UserData& messageBody, UserData& returnUserData)
 {
-auto* webProcessProxy = WebProcessProxy::fromConnection(&connection);
+auto* webProcessProxy = webProcessProxyFromConnection(connection, m_processes);
 if (!webProcessProxy)
 return;
 
@@ -1250,7 +1261,7 @@
 
 void WebProcessPool::startedUsingGamepads(IPC::Connection& connection)
 {
-auto* proxy = WebProcessProxy::fromConnection(&connection);
+auto* proxy = webProcessProxyFromConnection(connection, m_processes);
 if (!proxy)
 return;
 
@@ -1265,7 +1276,7 @@
 
 void WebProcessPool::stoppedUsingGamepads(IPC::Connection& connection)
 {
-auto* proxy = WebProcessProxy::fromConnection(&connection);
+auto* proxy = webProcessProxyFromConnection(connection, m_processes);
 if (!proxy)
 return;
 






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

[webkit-changes] [205155] tags/Safari-600.8.7.1/

2016-08-29 Thread bshafiei

Title: [205155] tags/Safari-600.8.7.1/








Revision 205155
Author bshaf...@apple.com
Date 2016-08-29 16:07:20 -0700 (Mon, 29 Aug 2016)


Log Message
New tag.

Added Paths

tags/Safari-600.8.7.1/




Diff




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

[webkit-changes] [205154] trunk

2016-08-29 Thread cdumez

Title: [205154] trunk








Revision 205154
Author cdu...@apple.com
Date 2016-08-29 16:04:44 -0700 (Mon, 29 Aug 2016)


Log Message
Regression(r204923): It should be possible to set 'Location.href' cross origin
https://bugs.webkit.org/show_bug.cgi?id=161343


Reviewed by Ryosuke Niwa.

Source/WebCore:

It should be possible to set 'Location.href' cross origin:
- https://html.spec.whatwg.org/#crossoriginproperties-(-o-)

Firefox and Chrome allow this but we throw a SecurityError.

We already allow setting crossOrigin.window.location which is equivalent.

No new tests, updated existing test.

* bindings/js/JSLocationCustom.cpp:
(WebCore::JSLocation::putDelegate):
Refactor the [Put] delegate so that it does not log a security error
when setting 'href' attribute, given that setting it works as expected.
This fixes a bug in shipping Safari where setting 'href' would work but
log an error message anyway.

* bindings/scripts/CodeGeneratorJS.pm:
(GenerateImplementation):
Add support for [DoNotCheckSecurityOnSetter] IDL extended attribute,
in addition to the already supported [DoNotCheckSecurity] and
[DoNotCheckSecurityOnGetter].

* page/Location.idl:
Use [DoNotCheckSecurityOnSetter] on 'href' attribute as it can be
set cross-origin. This fixes the regression introduced in r204923.

LayoutTests:

Add layout test coverage.

* http/tests/security/location-cross-origin-expected.txt:
* http/tests/security/location-cross-origin.html:
* http/tests/security/xss-DENIED-assign-location-href-_javascript_-expected.txt:

Modified Paths

trunk/LayoutTests/ChangeLog
trunk/LayoutTests/http/tests/security/location-cross-origin-expected.txt
trunk/LayoutTests/http/tests/security/location-cross-origin.html
trunk/LayoutTests/http/tests/security/xss-DENIED-assign-location-href-_javascript_-expected.txt
trunk/Source/WebCore/ChangeLog
trunk/Source/WebCore/bindings/js/JSLocationCustom.cpp
trunk/Source/WebCore/bindings/scripts/CodeGeneratorJS.pm
trunk/Source/WebCore/page/Location.idl




Diff

Modified: trunk/LayoutTests/ChangeLog (205153 => 205154)

--- trunk/LayoutTests/ChangeLog	2016-08-29 22:54:48 UTC (rev 205153)
+++ trunk/LayoutTests/ChangeLog	2016-08-29 23:04:44 UTC (rev 205154)
@@ -1,3 +1,17 @@
+2016-08-29  Chris Dumez  
+
+Regression(r204923): It should be possible to set 'Location.href' cross origin
+https://bugs.webkit.org/show_bug.cgi?id=161343
+
+
+Reviewed by Ryosuke Niwa.
+
+Add layout test coverage.
+
+* http/tests/security/location-cross-origin-expected.txt:
+* http/tests/security/location-cross-origin.html:
+* http/tests/security/xss-DENIED-assign-location-href-_javascript_-expected.txt:
+
 2016-08-29  Jiewen Tan  
 
 Unreviewed, update iOS simulator WK1 flaky tests.


Modified: trunk/LayoutTests/http/tests/security/location-cross-origin-expected.txt (205153 => 205154)

--- trunk/LayoutTests/http/tests/security/location-cross-origin-expected.txt	2016-08-29 22:54:48 UTC (rev 205153)
+++ trunk/LayoutTests/http/tests/security/location-cross-origin-expected.txt	2016-08-29 23:04:44 UTC (rev 205154)
@@ -27,6 +27,8 @@
 PASS Object.getOwnPropertyDescriptor(window.location, 'ancestorOrigins').get.call(frames[0].location) threw exception SecurityError (DOM Exception 18): Blocked a frame with origin "http://127.0.0.1:8000" from accessing a frame with origin "http://localhost:8000". Protocols, domains, and ports must match..
 PASS Object.getOwnPropertyDescriptor(window.location, 'toString').value.call(frames[0].location) threw exception SecurityError (DOM Exception 18): Blocked a frame with origin "http://127.0.0.1:8000" from accessing a frame with origin "http://localhost:8000". Protocols, domains, and ports must match..
 PASS Object.getOwnPropertyDescriptor(window.location, 'href').get.call(frames[0].location) threw exception SecurityError (DOM Exception 18): Blocked a frame with origin "http://127.0.0.1:8000" from accessing a frame with origin "http://localhost:8000". Protocols, domains, and ports must match..
+PASS frames[0].location.href = '' did not throw exception.
+PASS frames[0].location.href is "about:blank"
 PASS successfullyParsed is true
 
 TEST COMPLETE


Modified: trunk/LayoutTests/http/tests/security/location-cross-origin.html (205153 => 205154)

--- trunk/LayoutTests/http/tests/security/location-cross-origin.html	2016-08-29 22:54:48 UTC (rev 205153)
+++ trunk/LayoutTests/http/tests/security/location-cross-origin.html	2016-08-29 23:04:44 UTC (rev 205154)
@@ -34,7 +34,12 @@
 shouldThrowErrorName("Object.getOwnPropertyDescriptor(window.location, 'toString').value.call(frames[0].location)", "SecurityError");
 shouldThrowErrorName("Object.getOwnPropertyDescriptor(window.location, 'href').get.call(frames[0].location)", "SecurityError");
 
-finishJSTest();
+// Setting 'href' cross origin should be allowed.
+shouldNotThrow("frames[0].location.href = ''");
+setTimeout(function() {
+shouldBeEqualToString("frames[0].locatio

[webkit-changes] [205153] trunk/LayoutTests

2016-08-29 Thread jiewen_tan

Title: [205153] trunk/LayoutTests








Revision 205153
Author jiewen_...@apple.com
Date 2016-08-29 15:54:48 -0700 (Mon, 29 Aug 2016)


Log Message
Unreviewed, update iOS simulator WK1 flaky tests.

* platform/ios-simulator-wk1/TestExpectations:

Modified Paths

trunk/LayoutTests/ChangeLog
trunk/LayoutTests/platform/ios-simulator-wk1/TestExpectations




Diff

Modified: trunk/LayoutTests/ChangeLog (205152 => 205153)

--- trunk/LayoutTests/ChangeLog	2016-08-29 22:42:07 UTC (rev 205152)
+++ trunk/LayoutTests/ChangeLog	2016-08-29 22:54:48 UTC (rev 205153)
@@ -1,3 +1,9 @@
+2016-08-29  Jiewen Tan  
+
+Unreviewed, update iOS simulator WK1 flaky tests.
+
+* platform/ios-simulator-wk1/TestExpectations:
+
 2016-08-29  Chris Dumez  
 
 We should throw a SecurityError when denying setting a cross-origin Window property


Modified: trunk/LayoutTests/platform/ios-simulator-wk1/TestExpectations (205152 => 205153)

--- trunk/LayoutTests/platform/ios-simulator-wk1/TestExpectations	2016-08-29 22:42:07 UTC (rev 205152)
+++ trunk/LayoutTests/platform/ios-simulator-wk1/TestExpectations	2016-08-29 22:54:48 UTC (rev 205153)
@@ -156,7 +156,7 @@
 editing/deleting/delete-after-span-ws-003.html [ Failure ]
 editing/deleting/delete-all-text-in-text-field-assertion.html [ Failure ]
 editing/deleting/delete-and-undo.html [ Failure ]
-editing/deleting/delete-at-paragraph-boundaries-001.html [ Failure ]
+editing/deleting/delete-at-paragraph-boundaries-001.html [ Failure Pass ]
 editing/deleting/delete-block-contents-001.html [ Failure ]
 editing/deleting/delete-block-contents-002.html [ Failure ]
 editing/deleting/delete-br-002.html [ Failure ]
@@ -767,7 +767,6 @@
 svg/animations/polyline-set.svg [ ImageOnlyFailure ]
 svg/animations/repeating-path-animation.svg [ ImageOnlyFailure ]
 svg/animations/reset-animation-crash.html [ ImageOnlyFailure ]
-svg/animations/svglengthlist-animation-3.html [ Failure ]
 svg/batik/text/textEffect.svg [ Failure ]
 svg/css/parse-height.html [ Failure ]
 svg/css/parse-length.html [ Failure ]
@@ -850,7 +849,7 @@
 fast/forms/time/time-interactive-validation-required.html [ Failure ]
 fast/forms/validation-message-on-checkbox.html [ Failure ]
 fast/forms/validation-message-on-listbox.html [ Failure ]
-fast/forms/validation-message-on-menulist.html [ Failure ]
+fast/forms/validation-message-on-menulist.html [ Failure Crash ]
 fast/forms/validation-message-on-radio.html [ Failure ]
 fast/forms/validation-message-on-range.html [ Failure ]
 fast/forms/validation-message-on-textarea.html [ Failure ]
@@ -914,7 +913,6 @@
 
 # _javascript_s tests that fail:
 js/dom/deep-recursion-test.html [ Failure ]
-js/large-expressions.html [ Failure ]
 
 # iOS-specific tests that fail:
 platform/ios-simulator/ios/storage/domstorage/5mb-quota.html [ Failure ]
@@ -1121,10 +1119,7 @@
 http/tests/security/contentSecurityPolicy/_javascript_-url-blocked-by-default-src-star.html [ Failure ]
 http/tests/security/contentSecurityPolicy/object-src-url-allowed.html [ Failure ]
 http/tests/security/cross-origin-local-storage-wk1.html [ Failure ]
-http/tests/security/dataURL/xss-DENIED-to-data-url-in-foreign-domain-window-open.html [ Failure ]
-http/tests/security/dataURL/xss-DENIED-to-data-url-window-open.html [ Failure ]
 http/tests/security/frameNavigation/not-opener.html [ Failure ]
-http/tests/security/_javascript_URL/xss-DENIED-to-_javascript_-url-in-foreign-domain-window-open.html [ Failure ]
 http/tests/security/mixedContent/insecure-audio-video-in-main-frame.html [ Failure ]
 http/tests/security/mixedContent/insecure-css-in-main-frame.html [ Failure ]
 http/tests/security/mixedContent/insecure-iframe-in-main-frame.html [ Failure ]
@@ -1417,16 +1412,6 @@
 imported/mozilla/svg/smil/transform/rotate-angle-5.svg [ Pass ]
 imported/mozilla/svg/text/textLength-6.svg [ Pass ]
 
-# Crashes as of 08/26/2016
-fast/block/nested-renderers.html [ Crash ]
-fast/dom/HTMLObjectElement/beforeload-set-text-crash.xhtml [ Crash ]
-fast/dom/URL-attribute-reflection.html [ Crash ]
-fast/images/large-size-image-crash.html [ Crash ]
-http/tests/security/cross-origin-indexeddb-allowed.html [ Crash ]
-imported/blink/storage/indexeddb/blob-delete-objectstore-db.html [ Crash ]
-imported/blink/storage/indexeddb/blob-valid-after-deletion.html [ Crash ]
-imported/blink/storage/indexeddb/blob-valid-before-commit.html [ Crash ]
-
 # Timeout as of 08/26/2016
 fast/dom/Geolocation/window-close-crash.html [ Timeout ]
 fast/events/offsetX-offsetY.html [ Timeout ]
@@ -1510,10 +1495,14 @@
 transforms/hittest-translated-content-off-to-infinity-and-back.html [ Pass ImageOnlyFailure ]
 
 # Flaky as of 08/29/2016
+compositing/child-layer-with-subpixel-gap-needs-repaint-when-parent-moves.html [ Pass ImageOnlyFailure ]
+compositing/filters/opacity-change-on-filtered-paints-into-ancestor.html [ Pass ImageOnlyFailure ]
 compositing/masks/compositing-clip-path-mask-change.html [ Pass ImageOnlyFailure ]
 compositing/masks/solid-color-masked.html [ Pass Imag

[webkit-changes] [205152] branches/safari-601-branch/Source/JavaScriptCore

2016-08-29 Thread bshafiei

Title: [205152] branches/safari-601-branch/Source/_javascript_Core








Revision 205152
Author bshaf...@apple.com
Date 2016-08-29 15:42:07 -0700 (Mon, 29 Aug 2016)


Log Message
Merge r204572.  rdar://problem/28059311

Modified Paths

branches/safari-601-branch/Source/_javascript_Core/ChangeLog
branches/safari-601-branch/Source/_javascript_Core/runtime/ArgList.cpp
branches/safari-601-branch/Source/_javascript_Core/runtime/ArgList.h




Diff

Modified: branches/safari-601-branch/Source/_javascript_Core/ChangeLog (205151 => 205152)

--- branches/safari-601-branch/Source/_javascript_Core/ChangeLog	2016-08-29 22:10:09 UTC (rev 205151)
+++ branches/safari-601-branch/Source/_javascript_Core/ChangeLog	2016-08-29 22:42:07 UTC (rev 205152)
@@ -1,3 +1,35 @@
+2016-08-29  Babak Shafiei  
+
+Merge r204572.
+
+2016-08-17  Geoffrey Garen  
+
+Fixed a potential bug in MarkedArgumentBuffer.
+https://bugs.webkit.org/show_bug.cgi?id=160948
+
+
+Reviewed by Oliver Hunt.
+
+I haven't been able to produce an observable test case after some trying.
+
+* runtime/ArgList.cpp:
+(JSC::MarkedArgumentBuffer::addMarkSet): New helper function -- I broke
+this out from existing code for clarity, but the behavior is the same.
+
+(JSC::MarkedArgumentBuffer::expandCapacity): Ditto.
+
+(JSC::MarkedArgumentBuffer::slowAppend): Always addMarkSet() on the slow
+path. This is faster than the old linear scan, and I think it might
+avoid cases the old scan could miss.
+
+* runtime/ArgList.h:
+(JSC::MarkedArgumentBuffer::append): Account for the case where someone
+has called clear() or removeLast().
+
+(JSC::MarkedArgumentBuffer::mallocBase): No behavior change -- but it's
+clearer to test the buffers directly instead of inferring what they
+might be based on capacity.
+
 2016-05-13  Matthew Hanson  
 
 Merge r196490. rdar://problem/26270811


Modified: branches/safari-601-branch/Source/_javascript_Core/runtime/ArgList.cpp (205151 => 205152)

--- branches/safari-601-branch/Source/_javascript_Core/runtime/ArgList.cpp	2016-08-29 22:10:09 UTC (rev 205151)
+++ branches/safari-601-branch/Source/_javascript_Core/runtime/ArgList.cpp	2016-08-29 22:42:07 UTC (rev 205152)
@@ -30,6 +30,19 @@
 
 namespace JSC {
 
+void MarkedArgumentBuffer::addMarkSet(JSValue v)
+{
+if (m_markSet)
+return;
+
+Heap* heap = Heap::heap(v);
+if (!heap)
+return;
+
+m_markSet = &heap->markListSet();
+m_markSet->add(this);
+}
+
 void ArgList::getSlice(int startIndex, ArgList& result) const
 {
 if (startIndex <= 0 || startIndex >= m_argCount) {
@@ -51,13 +64,15 @@
 }
 }
 
-void MarkedArgumentBuffer::slowAppend(JSValue v)
+void MarkedArgumentBuffer::expandCapacity()
 {
 int newCapacity = (Checked(m_capacity) * 2).unsafeGet();
 size_t size = (Checked(newCapacity) * sizeof(EncodedJSValue)).unsafeGet();
 EncodedJSValue* newBuffer = static_cast(fastMalloc(size));
-for (int i = 0; i < m_capacity; ++i)
+for (int i = 0; i < m_capacity; ++i) {
 newBuffer[i] = m_buffer[i];
+addMarkSet(JSValue::decode(m_buffer[i]));
+}
 
 if (EncodedJSValue* base = mallocBase())
 fastFree(base);
@@ -64,27 +79,16 @@
 
 m_buffer = newBuffer;
 m_capacity = newCapacity;
+}
 
+void MarkedArgumentBuffer::slowAppend(JSValue v)
+{
+if (m_size >= m_capacity)
+expandCapacity();
+
 slotFor(m_size) = JSValue::encode(v);
 ++m_size;
-
-if (m_markSet)
-return;
-
-// As long as our size stays within our Vector's inline 
-// capacity, all our values are allocated on the stack, and 
-// therefore don't need explicit marking. Once our size exceeds
-// our Vector's inline capacity, though, our values move to the 
-// heap, where they do need explicit marking.
-for (int i = 0; i < m_size; ++i) {
-Heap* heap = Heap::heap(JSValue::decode(slotFor(i)));
-if (!heap)
-continue;
-
-m_markSet = &heap->markListSet();
-m_markSet->add(this);
-break;
-}
+addMarkSet(v);
 }
 
 } // namespace JSC


Modified: branches/safari-601-branch/Source/_javascript_Core/runtime/ArgList.h (205151 => 205152)

--- branches/safari-601-branch/Source/_javascript_Core/runtime/ArgList.h	2016-08-29 22:10:09 UTC (rev 205151)
+++ branches/safari-601-branch/Source/_javascript_Core/runtime/ArgList.h	2016-08-29 22:42:07 UTC (rev 205152)
@@ -78,7 +78,7 @@
 
 void append(JSValue v)
 {
-if (m_size >= m_capacity)
+if (m_size >= m_capacity || mallocBase())
 return slowAppend(v);
 
 slotFor(m_size) = JSValue::encode(v);
@@ -100,6 +100,10 @@
 static void markLists(HeapRootVisitor&, ListSet&);
 
 private:
+void expandCapacity();
+
+void addMarkSet(JSValue);
+
 JS_EXPORT_

[webkit-changes] [205151] trunk/Source/WebInspectorUI

2016-08-29 Thread commit-queue

Title: [205151] trunk/Source/WebInspectorUI








Revision 205151
Author commit-qu...@webkit.org
Date 2016-08-29 15:10:09 -0700 (Mon, 29 Aug 2016)


Log Message
Web Inspector: Unify Resource entry context menus
https://bugs.webkit.org/show_bug.cgi?id=161301

Patch by Devin Rousso  on 2016-08-29
Reviewed by Brian Burg.

* UserInterface/Main.html:
Add ContextMenuUtilities.

* UserInterface/Views/ContextMenuUtilities.js: Added.
(WebInspector.appendContextMenuItemsForResource):
Appends the following items to the given WebInspector.ContextMenu object with a WebInspector.Resource:
 - Open in New Tab
 - Copy Link Address
 - Copy as cURL (only if the resource is not from a data URL)
 - Save File

* UserInterface/Views/ResourceTimelineDataGridNode.js:
(WebInspector.ResourceTimelineDataGridNode.prototype.appendContextMenuItems):
Make use of new WebInspector.appendContextMenuItemsForResource.

* UserInterface/Views/ResourceTreeElement.js:
(WebInspector.ResourceTreeElement.prototype._handleContextMenuEvent):
Make use of new WebInspector.appendContextMenuItemsForResource.

Modified Paths

trunk/Source/WebInspectorUI/ChangeLog
trunk/Source/WebInspectorUI/UserInterface/Main.html
trunk/Source/WebInspectorUI/UserInterface/Views/ResourceTimelineDataGridNode.js
trunk/Source/WebInspectorUI/UserInterface/Views/ResourceTreeElement.js


Added Paths

trunk/Source/WebInspectorUI/UserInterface/Views/ContextMenuUtilities.js




Diff

Modified: trunk/Source/WebInspectorUI/ChangeLog (205150 => 205151)

--- trunk/Source/WebInspectorUI/ChangeLog	2016-08-29 22:00:39 UTC (rev 205150)
+++ trunk/Source/WebInspectorUI/ChangeLog	2016-08-29 22:10:09 UTC (rev 205151)
@@ -1,3 +1,29 @@
+2016-08-29  Devin Rousso  
+
+Web Inspector: Unify Resource entry context menus
+https://bugs.webkit.org/show_bug.cgi?id=161301
+
+Reviewed by Brian Burg.
+
+* UserInterface/Main.html:
+Add ContextMenuUtilities.
+
+* UserInterface/Views/ContextMenuUtilities.js: Added.
+(WebInspector.appendContextMenuItemsForResource):
+Appends the following items to the given WebInspector.ContextMenu object with a WebInspector.Resource:
+ - Open in New Tab
+ - Copy Link Address
+ - Copy as cURL (only if the resource is not from a data URL)
+ - Save File
+
+* UserInterface/Views/ResourceTimelineDataGridNode.js:
+(WebInspector.ResourceTimelineDataGridNode.prototype.appendContextMenuItems):
+Make use of new WebInspector.appendContextMenuItemsForResource.
+
+* UserInterface/Views/ResourceTreeElement.js:
+(WebInspector.ResourceTreeElement.prototype._handleContextMenuEvent):
+Make use of new WebInspector.appendContextMenuItemsForResource.
+
 2016-08-27  Joseph Pecoraro  
 
 Web Inspector: Make localizedString.js diff and commit friendly (UTF16 -> UTF8)


Modified: trunk/Source/WebInspectorUI/UserInterface/Main.html (205150 => 205151)

--- trunk/Source/WebInspectorUI/UserInterface/Main.html	2016-08-29 22:00:39 UTC (rev 205150)
+++ trunk/Source/WebInspectorUI/UserInterface/Main.html	2016-08-29 22:10:09 UTC (rev 205151)
@@ -488,6 +488,7 @@

[webkit-changes] [205150] trunk/Source/WebKit2

2016-08-29 Thread andersca

Title: [205150] trunk/Source/WebKit2








Revision 205150
Author ander...@apple.com
Date 2016-08-29 15:00:39 -0700 (Mon, 29 Aug 2016)


Log Message
Get rid of another use of WebProcessProxy::fromConnection
https://bugs.webkit.org/show_bug.cgi?id=161331

Reviewed by Dan Bernstein.

WebUserContentControllerProxy::didPostMessage uses WebProcessProxy::fromConnection to be able to find a WebFrameProxy from
a given frame ID. Since the WebFrameProxy is only used to construct an API::FrameInfo object from, introduce a FrameInfoData object
and fill it in on the web process side and send it over to the UI process where an API::FrameInfo object can be created.

* Shared/FrameInfoData.cpp: Added.
(WebKit::FrameInfoData::encode):
(WebKit::FrameInfoData::decode):
* Shared/FrameInfoData.h: Added.
* UIProcess/API/APIFrameInfo.cpp:
(API::FrameInfo::create):
(API::FrameInfo::FrameInfo):
* UIProcess/API/APIFrameInfo.h:
* UIProcess/API/Cocoa/WKUserContentController.mm:
* UIProcess/UserContent/WebScriptMessageHandler.h:
* UIProcess/UserContent/WebUserContentControllerProxy.cpp:
(WebKit::WebUserContentControllerProxy::didPostMessage):
* UIProcess/UserContent/WebUserContentControllerProxy.h:
* UIProcess/UserContent/WebUserContentControllerProxy.messages.in:
* WebKit2.xcodeproj/project.pbxproj:
* WebProcess/UserContent/WebUserContentController.cpp:
* WebProcess/WebPage/WebFrame.cpp:
(WebKit::WebFrame::info):
* WebProcess/WebPage/WebFrame.h:

Modified Paths

trunk/Source/WebKit2/ChangeLog
trunk/Source/WebKit2/UIProcess/API/APIFrameInfo.cpp
trunk/Source/WebKit2/UIProcess/API/APIFrameInfo.h
trunk/Source/WebKit2/UIProcess/API/Cocoa/WKUserContentController.mm
trunk/Source/WebKit2/UIProcess/UserContent/WebScriptMessageHandler.h
trunk/Source/WebKit2/UIProcess/UserContent/WebUserContentControllerProxy.cpp
trunk/Source/WebKit2/UIProcess/UserContent/WebUserContentControllerProxy.h
trunk/Source/WebKit2/UIProcess/UserContent/WebUserContentControllerProxy.messages.in
trunk/Source/WebKit2/WebKit2.xcodeproj/project.pbxproj
trunk/Source/WebKit2/WebProcess/UserContent/WebUserContentController.cpp
trunk/Source/WebKit2/WebProcess/WebPage/WebFrame.cpp
trunk/Source/WebKit2/WebProcess/WebPage/WebFrame.h


Added Paths

trunk/Source/WebKit2/Shared/FrameInfoData.cpp
trunk/Source/WebKit2/Shared/FrameInfoData.h




Diff

Modified: trunk/Source/WebKit2/ChangeLog (205149 => 205150)

--- trunk/Source/WebKit2/ChangeLog	2016-08-29 21:51:35 UTC (rev 205149)
+++ trunk/Source/WebKit2/ChangeLog	2016-08-29 22:00:39 UTC (rev 205150)
@@ -1,5 +1,36 @@
 2016-08-29  Anders Carlsson  
 
+Get rid of another use of WebProcessProxy::fromConnection
+https://bugs.webkit.org/show_bug.cgi?id=161331
+
+Reviewed by Dan Bernstein.
+
+WebUserContentControllerProxy::didPostMessage uses WebProcessProxy::fromConnection to be able to find a WebFrameProxy from
+a given frame ID. Since the WebFrameProxy is only used to construct an API::FrameInfo object from, introduce a FrameInfoData object
+and fill it in on the web process side and send it over to the UI process where an API::FrameInfo object can be created.
+
+* Shared/FrameInfoData.cpp: Added.
+(WebKit::FrameInfoData::encode):
+(WebKit::FrameInfoData::decode):
+* Shared/FrameInfoData.h: Added.
+* UIProcess/API/APIFrameInfo.cpp:
+(API::FrameInfo::create):
+(API::FrameInfo::FrameInfo):
+* UIProcess/API/APIFrameInfo.h:
+* UIProcess/API/Cocoa/WKUserContentController.mm:
+* UIProcess/UserContent/WebScriptMessageHandler.h:
+* UIProcess/UserContent/WebUserContentControllerProxy.cpp:
+(WebKit::WebUserContentControllerProxy::didPostMessage):
+* UIProcess/UserContent/WebUserContentControllerProxy.h:
+* UIProcess/UserContent/WebUserContentControllerProxy.messages.in:
+* WebKit2.xcodeproj/project.pbxproj:
+* WebProcess/UserContent/WebUserContentController.cpp:
+* WebProcess/WebPage/WebFrame.cpp:
+(WebKit::WebFrame::info):
+* WebProcess/WebPage/WebFrame.h:
+
+2016-08-29  Anders Carlsson  
+
 Stop using WebProcessProxy::fromConnection in WebPageProxy
 https://bugs.webkit.org/show_bug.cgi?id=161322
 


Copied: trunk/Source/WebKit2/Shared/FrameInfoData.cpp (from rev 205137, trunk/Source/WebKit2/UIProcess/API/APIFrameInfo.cpp) (0 => 205150)

--- trunk/Source/WebKit2/Shared/FrameInfoData.cpp	(rev 0)
+++ trunk/Source/WebKit2/Shared/FrameInfoData.cpp	2016-08-29 22:00:39 UTC (rev 205150)
@@ -0,0 +1,55 @@
+/*
+ * Copyright (C) 2016 Apple Inc. All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * 1. Redistributions of source code must retain the above copyright
+ *notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above

[webkit-changes] [205149] trunk/Tools

2016-08-29 Thread aakash_jain

Title: [205149] trunk/Tools








Revision 205149
Author aakash_j...@apple.com
Date 2016-08-29 14:51:35 -0700 (Mon, 29 Aug 2016)


Log Message
EWS should indicate which bot processed the patch
https://bugs.webkit.org/show_bug.cgi?id=161222

Reviewed by Alexey Proskuryakov.

* QueueStatusServer/handlers/statusbubble.py:
(StatusBubble._build_bubble): Remove the bot id from bubbles as it is confusing.
bot ids are now displayed in patch status page.

Modified Paths

trunk/Tools/ChangeLog
trunk/Tools/QueueStatusServer/handlers/statusbubble.py




Diff

Modified: trunk/Tools/ChangeLog (205148 => 205149)

--- trunk/Tools/ChangeLog	2016-08-29 21:35:20 UTC (rev 205148)
+++ trunk/Tools/ChangeLog	2016-08-29 21:51:35 UTC (rev 205149)
@@ -1,3 +1,14 @@
+2016-08-29  Aakash Jain  
+
+EWS should indicate which bot processed the patch
+https://bugs.webkit.org/show_bug.cgi?id=161222
+
+Reviewed by Alexey Proskuryakov.
+
+* QueueStatusServer/handlers/statusbubble.py:
+(StatusBubble._build_bubble): Remove the bot id from bubbles as it is confusing.
+bot ids are now displayed in patch status page.
+
 2016-08-29  Alex Christensen  
 
 URLParser should parse about:blank


Modified: trunk/Tools/QueueStatusServer/handlers/statusbubble.py (205148 => 205149)

--- trunk/Tools/QueueStatusServer/handlers/statusbubble.py	2016-08-29 21:35:20 UTC (rev 205148)
+++ trunk/Tools/QueueStatusServer/handlers/statusbubble.py	2016-08-29 21:51:35 UTC (rev 205149)
@@ -132,7 +132,7 @@
 statuses))
 if not latest_resultative_status:
 bubble["state"] = "started"
-bubble["details_message"] = (statuses[0].bot_id + ", recent messages:\n\n"
+bubble["details_message"] = ("Recent messages:\n\n"
 + "\n".join([status.message for status in statuses]) + "\n\n" + self._iso_time(statuses[0].date))
 elif statuses[0].message == "Pass":
 bubble["state"] = "pass"






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

[webkit-changes] [205148] trunk

2016-08-29 Thread cdumez

Title: [205148] trunk








Revision 205148
Author cdu...@apple.com
Date 2016-08-29 14:35:20 -0700 (Mon, 29 Aug 2016)


Log Message
We should throw a SecurityError when denying setting a cross-origin Window property
https://bugs.webkit.org/show_bug.cgi?id=161339

Reviewed by Geoffrey Garen.

Source/WebCore:

We should throw a SecurityError when denying setting a cross-origin Window property:
- https://html.spec.whatwg.org/#crossoriginset-(-o,-p,-v,-receiver-)
- https://html.spec.whatwg.org/#crossorigingetownpropertyhelper-(-o,-p-)

e.g. crossOriginWindow.name = "" should throw.

Firefox and Chrome already throw but WebKit merely ignores the call and logs an
error message.

No new tests, updated existing tests.

* bindings/js/JSDOMWindowCustom.cpp:
(WebCore::JSDOMWindow::put):

LayoutTests:

Update existing tests to reflect behavior change.

* http/tests/history/cross-origin-replace-history-object-expected.txt:
* http/tests/history/resources/cross-origin-replaces-history-object-iframe.html:
* http/tests/security/cross-frame-access-custom-expected.txt:
* http/tests/security/cross-frame-access-put-expected.txt:

Modified Paths

trunk/LayoutTests/ChangeLog
trunk/LayoutTests/http/tests/history/cross-origin-replace-history-object-expected.txt
trunk/LayoutTests/http/tests/history/resources/cross-origin-replaces-history-object-iframe.html
trunk/LayoutTests/http/tests/security/cross-frame-access-custom-expected.txt
trunk/LayoutTests/http/tests/security/cross-frame-access-put-expected.txt
trunk/Source/WebCore/ChangeLog
trunk/Source/WebCore/bindings/js/JSDOMWindowCustom.cpp




Diff

Modified: trunk/LayoutTests/ChangeLog (205147 => 205148)

--- trunk/LayoutTests/ChangeLog	2016-08-29 21:15:27 UTC (rev 205147)
+++ trunk/LayoutTests/ChangeLog	2016-08-29 21:35:20 UTC (rev 205148)
@@ -1,3 +1,17 @@
+2016-08-29  Chris Dumez  
+
+We should throw a SecurityError when denying setting a cross-origin Window property
+https://bugs.webkit.org/show_bug.cgi?id=161339
+
+Reviewed by Geoffrey Garen.
+
+Update existing tests to reflect behavior change.
+
+* http/tests/history/cross-origin-replace-history-object-expected.txt:
+* http/tests/history/resources/cross-origin-replaces-history-object-iframe.html:
+* http/tests/security/cross-frame-access-custom-expected.txt:
+* http/tests/security/cross-frame-access-put-expected.txt:
+
 2016-08-29  Ryan Haddad  
 
 Rebaseline imported/w3c/web-platform-tests/dom/events/EventTarget-dispatchEvent.html for ios-simulator after r205138.


Modified: trunk/LayoutTests/http/tests/history/cross-origin-replace-history-object-expected.txt (205147 => 205148)

--- trunk/LayoutTests/http/tests/history/cross-origin-replace-history-object-expected.txt	2016-08-29 21:15:27 UTC (rev 205147)
+++ trunk/LayoutTests/http/tests/history/cross-origin-replace-history-object-expected.txt	2016-08-29 21:35:20 UTC (rev 205148)
@@ -1,5 +1,5 @@
 ALERT: Parent window's history object before attempt to clear: [object History]
 ALERT: Attempting to clear parent window's history object:
-CONSOLE MESSAGE: line 9: Blocked a frame with origin "http://localhost:8000" from accessing a frame with origin "http://127.0.0.1:8000". Protocols, domains, and ports must match.
+CONSOLE MESSAGE: line 21: SecurityError (DOM Exception 18): Blocked a frame with origin "http://localhost:8000" from accessing a frame with origin "http://127.0.0.1:8000". Protocols, domains, and ports must match.
 ALERT: Parent window's history object after attempt to clear: [object History]
 


Modified: trunk/LayoutTests/http/tests/history/resources/cross-origin-replaces-history-object-iframe.html (205147 => 205148)

--- trunk/LayoutTests/http/tests/history/resources/cross-origin-replaces-history-object-iframe.html	2016-08-29 21:15:27 UTC (rev 205147)
+++ trunk/LayoutTests/http/tests/history/resources/cross-origin-replaces-history-object-iframe.html	2016-08-29 21:35:20 UTC (rev 205148)
@@ -15,7 +15,11 @@
 function setHistoryLength()
 {
 alert("Attempting to clear parent window's history object:");
-parent.window.history = "";
+try {
+parent.window.history = "";
+} catch (e) {
+console.log(e);
+}
 parent.window.postMessage("done", "*");
 }
 


Modified: trunk/LayoutTests/http/tests/security/cross-frame-access-custom-expected.txt (205147 => 205148)

--- trunk/LayoutTests/http/tests/security/cross-frame-access-custom-expected.txt	2016-08-29 21:15:27 UTC (rev 205147)
+++ trunk/LayoutTests/http/tests/security/cross-frame-access-custom-expected.txt	2016-08-29 21:35:20 UTC (rev 205148)
@@ -1,22 +1,15 @@
 CONSOLE MESSAGE: line 57: SecurityError (DOM Exception 18): Blocked a frame with origin "http://127.0.0.1:8000" from accessing a frame with origin "http://localhost:8000". Protocols, domains, and ports must match.
-CONSOLE MESSAGE: line 83: Blocked a frame with origin "http://127.0.0.1:8000" from accessing a frame with origin "http://localhost:8000". Protocols, d

[webkit-changes] [205147] trunk

2016-08-29 Thread achristensen

Title: [205147] trunk








Revision 205147
Author achristen...@apple.com
Date 2016-08-29 14:15:27 -0700 (Mon, 29 Aug 2016)


Log Message
URLParser should parse about:blank
https://bugs.webkit.org/show_bug.cgi?id=161324

Reviewed by Brady Eidson.

Source/WebCore:

Covered by new API tests.

* platform/URLParser.cpp:
(WebCore::URLParser::parse):

Tools:

* TestWebKitAPI/Tests/WebCore/URLParser.cpp:
(TestWebKitAPI::TEST_F):

Modified Paths

trunk/Source/WebCore/ChangeLog
trunk/Source/WebCore/platform/URLParser.cpp
trunk/Tools/ChangeLog
trunk/Tools/TestWebKitAPI/Tests/WebCore/URLParser.cpp




Diff

Modified: trunk/Source/WebCore/ChangeLog (205146 => 205147)

--- trunk/Source/WebCore/ChangeLog	2016-08-29 21:12:29 UTC (rev 205146)
+++ trunk/Source/WebCore/ChangeLog	2016-08-29 21:15:27 UTC (rev 205147)
@@ -1,3 +1,15 @@
+2016-08-29  Alex Christensen  
+
+URLParser should parse about:blank
+https://bugs.webkit.org/show_bug.cgi?id=161324
+
+Reviewed by Brady Eidson.
+
+Covered by new API tests.
+
+* platform/URLParser.cpp:
+(WebCore::URLParser::parse):
+
 2016-08-29  Chris Dumez  
 
 document.createEvent("popstateevent") should create a PopStateEvent


Modified: trunk/Source/WebCore/platform/URLParser.cpp (205146 => 205147)

--- trunk/Source/WebCore/platform/URLParser.cpp	2016-08-29 21:12:29 UTC (rev 205146)
+++ trunk/Source/WebCore/platform/URLParser.cpp	2016-08-29 21:15:27 UTC (rev 205147)
@@ -220,8 +220,15 @@
 m_buffer.append('/');
 state = State::PathOrAuthority;
 ++c;
-} else
+} else {
+m_url.m_userStart = m_buffer.length();
+m_url.m_userEnd = m_url.m_userStart;
+m_url.m_passwordEnd = m_url.m_userStart;
+m_url.m_hostEnd = m_url.m_userStart;
+m_url.m_portEnd = m_url.m_userStart;
+m_url.m_pathAfterLastSlash = m_url.m_userStart,
 state = State::CannotBeABaseURLPath;
+}
 break;
 case State::NoScheme:
 LOG_STATE("NoScheme");
@@ -404,8 +411,17 @@
 break;
 case State::CannotBeABaseURLPath:
 LOG_STATE("CannotBeABaseURLPath");
-notImplemented();
-++c;
+if (*c == '?') {
+m_url.m_pathEnd = m_buffer.length();
+state = State::Query;
+} else if (*c == '#') {
+m_url.m_pathEnd = m_buffer.length();
+m_url.m_queryEnd = m_url.m_pathEnd;
+state = State::Fragment;
+} else {
+m_buffer.append(*c);
+++c;
+}
 break;
 case State::Query:
 LOG_STATE("Query");
@@ -492,6 +508,9 @@
 break;
 case State::CannotBeABaseURLPath:
 LOG_FINAL_STATE("CannotBeABaseURLPath");
+m_url.m_pathEnd = m_buffer.length();
+m_url.m_queryEnd = m_url.m_pathEnd;
+m_url.m_fragmentEnd = m_url.m_pathEnd;
 break;
 case State::Query:
 LOG_FINAL_STATE("Query");


Modified: trunk/Tools/ChangeLog (205146 => 205147)

--- trunk/Tools/ChangeLog	2016-08-29 21:12:29 UTC (rev 205146)
+++ trunk/Tools/ChangeLog	2016-08-29 21:15:27 UTC (rev 205147)
@@ -1,3 +1,13 @@
+2016-08-29  Alex Christensen  
+
+URLParser should parse about:blank
+https://bugs.webkit.org/show_bug.cgi?id=161324
+
+Reviewed by Brady Eidson.
+
+* TestWebKitAPI/Tests/WebCore/URLParser.cpp:
+(TestWebKitAPI::TEST_F):
+
 2016-08-29  JF Bastien  
 
 Bug 161165 - check-webkit-style doesn't understand macros containing `else` or labels


Modified: trunk/Tools/TestWebKitAPI/Tests/WebCore/URLParser.cpp (205146 => 205147)

--- trunk/Tools/TestWebKitAPI/Tests/WebCore/URLParser.cpp	2016-08-29 21:12:29 UTC (rev 205146)
+++ trunk/Tools/TestWebKitAPI/Tests/WebCore/URLParser.cpp	2016-08-29 21:15:27 UTC (rev 205147)
@@ -91,7 +91,9 @@
 checkURL("http://127.0.0.1", {"http", "", "", "127.0.0.1", 0, "/", "", "", "http://127.0.0.1/"});
 checkURL("http://webkit.org/", {"http", "", "", "webkit.org", 0, "/", "", "", "http://webkit.org/"});
 checkURL("http://webkit.org/path1/path2/index.html", {"http", "", "", "webkit.org", 0, "/path1/path2/index.html", "", "", "http://webkit.org/path1/path2/index.html"});
-
+checkURL("about:blank", {"about", "", "", "", 0, "blank", "", "", "about:blank"});
+checkURL("about:blank?query", {"about", "", "", "", 0, "blank", "query", "", "about:blank?query"});
+checkURL("about:blank#fragment", {"about", "", "", "", 0, "blank", "", "fragment", "about:blank#fragment"});
 }
 
 static void checkRelativeURL(const String& urlString, const String& baseURLString, const ExpectedParts& parts)






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

[webkit-changes] [205146] trunk/LayoutTests

2016-08-29 Thread ryanhaddad

Title: [205146] trunk/LayoutTests








Revision 205146
Author ryanhad...@apple.com
Date 2016-08-29 14:12:29 -0700 (Mon, 29 Aug 2016)


Log Message
Rebaseline imported/w3c/web-platform-tests/dom/events/EventTarget-dispatchEvent.html for ios-simulator after r205138.

Unreviewed test gardening.

* platform/ios-simulator/imported/w3c/web-platform-tests/dom/events/EventTarget-dispatchEvent-expected.txt:

Modified Paths

trunk/LayoutTests/ChangeLog
trunk/LayoutTests/platform/ios-simulator/imported/w3c/web-platform-tests/dom/events/EventTarget-dispatchEvent-expected.txt




Diff

Modified: trunk/LayoutTests/ChangeLog (205145 => 205146)

--- trunk/LayoutTests/ChangeLog	2016-08-29 21:08:42 UTC (rev 205145)
+++ trunk/LayoutTests/ChangeLog	2016-08-29 21:12:29 UTC (rev 205146)
@@ -1,3 +1,11 @@
+2016-08-29  Ryan Haddad  
+
+Rebaseline imported/w3c/web-platform-tests/dom/events/EventTarget-dispatchEvent.html for ios-simulator after r205138.
+
+Unreviewed test gardening.
+
+* platform/ios-simulator/imported/w3c/web-platform-tests/dom/events/EventTarget-dispatchEvent-expected.txt:
+
 2016-08-29  Jiewen Tan  
 
 Unreviewed, mark editing tests of iOS simulator WK1 release build that have different delegate callback order than those of debug build as failure


Modified: trunk/LayoutTests/platform/ios-simulator/imported/w3c/web-platform-tests/dom/events/EventTarget-dispatchEvent-expected.txt (205145 => 205146)

--- trunk/LayoutTests/platform/ios-simulator/imported/w3c/web-platform-tests/dom/events/EventTarget-dispatchEvent-expected.txt	2016-08-29 21:08:42 UTC (rev 205145)
+++ trunk/LayoutTests/platform/ios-simulator/imported/w3c/web-platform-tests/dom/events/EventTarget-dispatchEvent-expected.txt	2016-08-29 21:12:29 UTC (rev 205146)
@@ -21,7 +21,7 @@
 PASS If the event's initialized flag is not set, an InvalidStateError must be thrown (MouseEvent). 
 PASS If the event's initialized flag is not set, an InvalidStateError must be thrown (MouseEvents). 
 FAIL If the event's initialized flag is not set, an InvalidStateError must be thrown (PageTransitionEvent). The operation is not supported.
-FAIL If the event's initialized flag is not set, an InvalidStateError must be thrown (PopStateEvent). The operation is not supported.
+PASS If the event's initialized flag is not set, an InvalidStateError must be thrown (PopStateEvent). 
 FAIL If the event's initialized flag is not set, an InvalidStateError must be thrown (ProgressEvent). The operation is not supported.
 PASS If the event's initialized flag is not set, an InvalidStateError must be thrown (StorageEvent). 
 FAIL If the event's initialized flag is not set, an InvalidStateError must be thrown (SVGEvents). The operation is not supported.






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

[webkit-changes] [205145] trunk/LayoutTests

2016-08-29 Thread jiewen_tan

Title: [205145] trunk/LayoutTests








Revision 205145
Author jiewen_...@apple.com
Date 2016-08-29 14:08:42 -0700 (Mon, 29 Aug 2016)


Log Message
Unreviewed, mark editing tests of iOS simulator WK1 release build that have different delegate callback order than those of debug build as failure

* platform/ios-simulator-wk1/TestExpectations:

Modified Paths

trunk/LayoutTests/ChangeLog
trunk/LayoutTests/platform/ios-simulator-wk1/TestExpectations




Diff

Modified: trunk/LayoutTests/ChangeLog (205144 => 205145)

--- trunk/LayoutTests/ChangeLog	2016-08-29 21:06:00 UTC (rev 205144)
+++ trunk/LayoutTests/ChangeLog	2016-08-29 21:08:42 UTC (rev 205145)
@@ -1,5 +1,11 @@
 2016-08-29  Jiewen Tan  
 
+Unreviewed, mark editing tests of iOS simulator WK1 release build that have different delegate callback order than those of debug build as failure
+
+* platform/ios-simulator-wk1/TestExpectations:
+
+2016-08-29  Jiewen Tan  
+
 Unreviewed, rolling out r205143.
 
 Needs better description


Modified: trunk/LayoutTests/platform/ios-simulator-wk1/TestExpectations (205144 => 205145)

--- trunk/LayoutTests/platform/ios-simulator-wk1/TestExpectations	2016-08-29 21:06:00 UTC (rev 205144)
+++ trunk/LayoutTests/platform/ios-simulator-wk1/TestExpectations	2016-08-29 21:08:42 UTC (rev 205145)
@@ -1630,3 +1630,85 @@
 svg/W3C-I18N/g-dirLTR-ubNone.svg [ Pass Failure ]
 tables/hittesting/filltable-stress.html [ Failure Pass ]
 transitions/svg-text-shadow-transition.html [ Failure Pass ]
+
+# Mark editing tests of iOS simulator WK1 release build that have different delegate callback order than those of debug build as failure
+[ Release ] editing/deleting/delete-at-paragraph-boundaries-002.html [ Failure ]
+[ Release ] editing/deleting/delete-at-paragraph-boundaries-003.html [ Failure ]
+[ Release ] editing/deleting/delete-at-paragraph-boundaries-005.html [ Failure ]
+[ Release ] editing/deleting/delete-at-paragraph-boundaries-006.html [ Failure ]
+[ Release ] editing/deleting/delete-at-paragraph-boundaries-007.html [ Failure ]
+[ Release ] editing/deleting/delete-at-paragraph-boundaries-008.html [ Failure ]
+[ Release ] editing/deleting/delete-at-paragraph-boundaries-009.html [ Failure ]
+[ Release ] editing/deleting/delete-at-paragraph-boundaries-011.html [ Failure ]
+[ Release ] editing/deleting/delete-block-contents-003.html [ Failure ]
+[ Release ] editing/deleting/delete-block-merge-contents-002.html [ Failure ]
+[ Release ] editing/deleting/delete-block-merge-contents-006.html [ Failure ]
+[ Release ] editing/deleting/delete-block-merge-contents-007.html [ Failure ]
+[ Release ] editing/deleting/delete-block-merge-contents-009.html [ Failure ]
+[ Release ] editing/deleting/delete-block-merge-contents-012.html [ Failure ]
+[ Release ] editing/deleting/delete-block-merge-contents-013.html [ Failure ]
+[ Release ] editing/deleting/delete-block-merge-contents-014.html [ Failure ]
+[ Release ] editing/deleting/delete-block-merge-contents-016.html [ Failure ]
+[ Release ] editing/deleting/delete-block-merge-contents-018.html [ Failure ]
+[ Release ] editing/deleting/delete-block-merge-contents-022.html [ Failure ]
+[ Release ] editing/deleting/delete-block-merge-contents-023.html [ Failure ]
+[ Release ] editing/deleting/delete-block-merge-contents-024.html [ Failure ]
+[ Release ] editing/deleting/delete-br-003.html [ Failure ]
+[ Release ] editing/deleting/delete-br-010.html [ Failure ]
+[ Release ] editing/deleting/delete-character-001.html [ Failure ]
+[ Release ] editing/deleting/delete-hr.html [ Failure ]
+[ Release ] editing/deleting/delete-image-001.html [ Failure ]
+[ Release ] editing/deleting/delete-image-002.html [ Failure ]
+[ Release ] editing/deleting/delete-image-003.html [ Failure ]
+[ Release ] editing/deleting/delete-line-001.html [ Failure ]
+[ Release ] editing/deleting/delete-line-002.html [ Failure ]
+[ Release ] editing/deleting/delete-line-005.html [ Failure ]
+[ Release ] editing/deleting/delete-line-007.html [ Failure ]
+[ Release ] editing/deleting/delete-line-009.html [ Failure ]
+[ Release ] editing/deleting/delete-line-010.html [ Failure ]
+[ Release ] editing/deleting/delete-line-013.html [ Failure ]
+[ Release ] editing/deleting/delete-line-015.html [ Failure ]
+[ Release ] editing/deleting/delete-line-017.html [ Failure ]
+[ Release ] editing/deleting/delete-listitem-001.html [ Failure ]
+[ Release ] editing/deleting/delete-listitem-002.html [ Failure ]
+[ Release ] editing/deleting/delete-selection-001.html [ Failure ]
+[ Release ] editing/deleting/delete-trailing-ws-001.html [ Failure ]
+[ Release ] editing/deleting/delete-trailing-ws-002.html [ Failure ]
+[ Release ] editing/deleting/delete-ws-fixup-002.html [ Failure ]
+[ Release ] editing/deleting/list-item-1.html [ Failure ]
+[ Release ] editing/deleting/merge-unrendered-space.html [ Failure ]
+[ Release ] editing/deleting/non-smart-delete.html [ Failure ]
+[ Release ] editing/inserting/insert-3778059-fix.html

[webkit-changes] [205144] trunk/LayoutTests

2016-08-29 Thread jiewen_tan

Title: [205144] trunk/LayoutTests








Revision 205144
Author jiewen_...@apple.com
Date 2016-08-29 14:06:00 -0700 (Mon, 29 Aug 2016)


Log Message
Unreviewed, rolling out r205143.

Needs better description

Reverted changeset:

"Unreviewed, mark editing tests of iOS simulator WK1 release
build that have different delegate callbacks than those of
debug build as failure"
http://trac.webkit.org/changeset/205143

Modified Paths

trunk/LayoutTests/ChangeLog
trunk/LayoutTests/platform/ios-simulator-wk1/TestExpectations




Diff

Modified: trunk/LayoutTests/ChangeLog (205143 => 205144)

--- trunk/LayoutTests/ChangeLog	2016-08-29 20:46:01 UTC (rev 205143)
+++ trunk/LayoutTests/ChangeLog	2016-08-29 21:06:00 UTC (rev 205144)
@@ -1,5 +1,18 @@
 2016-08-29  Jiewen Tan  
 
+Unreviewed, rolling out r205143.
+
+Needs better description
+
+Reverted changeset:
+
+"Unreviewed, mark editing tests of iOS simulator WK1 release
+build that have different delegate callbacks than those of
+debug build as failure"
+http://trac.webkit.org/changeset/205143
+
+2016-08-29  Jiewen Tan  
+
 Unreviewed, mark editing tests of iOS simulator WK1 release build that have different delegate callbacks than those of debug build as failure
 
 * platform/ios-simulator-wk1/TestExpectations:


Modified: trunk/LayoutTests/platform/ios-simulator-wk1/TestExpectations (205143 => 205144)

--- trunk/LayoutTests/platform/ios-simulator-wk1/TestExpectations	2016-08-29 20:46:01 UTC (rev 205143)
+++ trunk/LayoutTests/platform/ios-simulator-wk1/TestExpectations	2016-08-29 21:06:00 UTC (rev 205144)
@@ -1630,85 +1630,3 @@
 svg/W3C-I18N/g-dirLTR-ubNone.svg [ Pass Failure ]
 tables/hittesting/filltable-stress.html [ Failure Pass ]
 transitions/svg-text-shadow-transition.html [ Failure Pass ]
-
-# Mark editing tests of iOS simulator WK1 release build that have different delegate callbacks than those of debug build as failure
-[ Release ] editing/deleting/delete-at-paragraph-boundaries-002.html [ Failure ]
-[ Release ] editing/deleting/delete-at-paragraph-boundaries-003.html [ Failure ]
-[ Release ] editing/deleting/delete-at-paragraph-boundaries-005.html [ Failure ]
-[ Release ] editing/deleting/delete-at-paragraph-boundaries-006.html [ Failure ]
-[ Release ] editing/deleting/delete-at-paragraph-boundaries-007.html [ Failure ]
-[ Release ] editing/deleting/delete-at-paragraph-boundaries-008.html [ Failure ]
-[ Release ] editing/deleting/delete-at-paragraph-boundaries-009.html [ Failure ]
-[ Release ] editing/deleting/delete-at-paragraph-boundaries-011.html [ Failure ]
-[ Release ] editing/deleting/delete-block-contents-003.html [ Failure ]
-[ Release ] editing/deleting/delete-block-merge-contents-002.html [ Failure ]
-[ Release ] editing/deleting/delete-block-merge-contents-006.html [ Failure ]
-[ Release ] editing/deleting/delete-block-merge-contents-007.html [ Failure ]
-[ Release ] editing/deleting/delete-block-merge-contents-009.html [ Failure ]
-[ Release ] editing/deleting/delete-block-merge-contents-012.html [ Failure ]
-[ Release ] editing/deleting/delete-block-merge-contents-013.html [ Failure ]
-[ Release ] editing/deleting/delete-block-merge-contents-014.html [ Failure ]
-[ Release ] editing/deleting/delete-block-merge-contents-016.html [ Failure ]
-[ Release ] editing/deleting/delete-block-merge-contents-018.html [ Failure ]
-[ Release ] editing/deleting/delete-block-merge-contents-022.html [ Failure ]
-[ Release ] editing/deleting/delete-block-merge-contents-023.html [ Failure ]
-[ Release ] editing/deleting/delete-block-merge-contents-024.html [ Failure ]
-[ Release ] editing/deleting/delete-br-003.html [ Failure ]
-[ Release ] editing/deleting/delete-br-010.html [ Failure ]
-[ Release ] editing/deleting/delete-character-001.html [ Failure ]
-[ Release ] editing/deleting/delete-hr.html [ Failure ]
-[ Release ] editing/deleting/delete-image-001.html [ Failure ]
-[ Release ] editing/deleting/delete-image-002.html [ Failure ]
-[ Release ] editing/deleting/delete-image-003.html [ Failure ]
-[ Release ] editing/deleting/delete-line-001.html [ Failure ]
-[ Release ] editing/deleting/delete-line-002.html [ Failure ]
-[ Release ] editing/deleting/delete-line-005.html [ Failure ]
-[ Release ] editing/deleting/delete-line-007.html [ Failure ]
-[ Release ] editing/deleting/delete-line-009.html [ Failure ]
-[ Release ] editing/deleting/delete-line-010.html [ Failure ]
-[ Release ] editing/deleting/delete-line-013.html [ Failure ]
-[ Release ] editing/deleting/delete-line-015.html [ Failure ]
-[ Release ] editing/deleting/delete-line-017.html [ Failure ]
-[ Release ] editing/deleting/delete-listitem-001.html [ Failure ]
-[ Release ] editing/deleting/delete-listitem-002.html [ Failure ]
-[ Release ] editing/deleting/delete-selection-001.html [ Failure ]
-[ Release ] editing/deleting/delete-trailing-ws-001.html [ Failure ]
-[ Release ] editing/deleting/delete-trailing-ws-002.html [ Failure ]
-[

[webkit-changes] [205143] trunk/LayoutTests

2016-08-29 Thread jiewen_tan

Title: [205143] trunk/LayoutTests








Revision 205143
Author jiewen_...@apple.com
Date 2016-08-29 13:46:01 -0700 (Mon, 29 Aug 2016)


Log Message
Unreviewed, mark editing tests of iOS simulator WK1 release build that have different delegate callbacks than those of debug build as failure

* platform/ios-simulator-wk1/TestExpectations:

Modified Paths

trunk/LayoutTests/ChangeLog
trunk/LayoutTests/platform/ios-simulator-wk1/TestExpectations




Diff

Modified: trunk/LayoutTests/ChangeLog (205142 => 205143)

--- trunk/LayoutTests/ChangeLog	2016-08-29 20:26:17 UTC (rev 205142)
+++ trunk/LayoutTests/ChangeLog	2016-08-29 20:46:01 UTC (rev 205143)
@@ -1,5 +1,11 @@
 2016-08-29  Jiewen Tan  
 
+Unreviewed, mark editing tests of iOS simulator WK1 release build that have different delegate callbacks than those of debug build as failure
+
+* platform/ios-simulator-wk1/TestExpectations:
+
+2016-08-29  Jiewen Tan  
+
 Unreviewed, update iOS simulator WK1 flaky tests.
 
 * platform/ios-simulator-wk1/TestExpectations:


Modified: trunk/LayoutTests/platform/ios-simulator-wk1/TestExpectations (205142 => 205143)

--- trunk/LayoutTests/platform/ios-simulator-wk1/TestExpectations	2016-08-29 20:26:17 UTC (rev 205142)
+++ trunk/LayoutTests/platform/ios-simulator-wk1/TestExpectations	2016-08-29 20:46:01 UTC (rev 205143)
@@ -1630,3 +1630,85 @@
 svg/W3C-I18N/g-dirLTR-ubNone.svg [ Pass Failure ]
 tables/hittesting/filltable-stress.html [ Failure Pass ]
 transitions/svg-text-shadow-transition.html [ Failure Pass ]
+
+# Mark editing tests of iOS simulator WK1 release build that have different delegate callbacks than those of debug build as failure
+[ Release ] editing/deleting/delete-at-paragraph-boundaries-002.html [ Failure ]
+[ Release ] editing/deleting/delete-at-paragraph-boundaries-003.html [ Failure ]
+[ Release ] editing/deleting/delete-at-paragraph-boundaries-005.html [ Failure ]
+[ Release ] editing/deleting/delete-at-paragraph-boundaries-006.html [ Failure ]
+[ Release ] editing/deleting/delete-at-paragraph-boundaries-007.html [ Failure ]
+[ Release ] editing/deleting/delete-at-paragraph-boundaries-008.html [ Failure ]
+[ Release ] editing/deleting/delete-at-paragraph-boundaries-009.html [ Failure ]
+[ Release ] editing/deleting/delete-at-paragraph-boundaries-011.html [ Failure ]
+[ Release ] editing/deleting/delete-block-contents-003.html [ Failure ]
+[ Release ] editing/deleting/delete-block-merge-contents-002.html [ Failure ]
+[ Release ] editing/deleting/delete-block-merge-contents-006.html [ Failure ]
+[ Release ] editing/deleting/delete-block-merge-contents-007.html [ Failure ]
+[ Release ] editing/deleting/delete-block-merge-contents-009.html [ Failure ]
+[ Release ] editing/deleting/delete-block-merge-contents-012.html [ Failure ]
+[ Release ] editing/deleting/delete-block-merge-contents-013.html [ Failure ]
+[ Release ] editing/deleting/delete-block-merge-contents-014.html [ Failure ]
+[ Release ] editing/deleting/delete-block-merge-contents-016.html [ Failure ]
+[ Release ] editing/deleting/delete-block-merge-contents-018.html [ Failure ]
+[ Release ] editing/deleting/delete-block-merge-contents-022.html [ Failure ]
+[ Release ] editing/deleting/delete-block-merge-contents-023.html [ Failure ]
+[ Release ] editing/deleting/delete-block-merge-contents-024.html [ Failure ]
+[ Release ] editing/deleting/delete-br-003.html [ Failure ]
+[ Release ] editing/deleting/delete-br-010.html [ Failure ]
+[ Release ] editing/deleting/delete-character-001.html [ Failure ]
+[ Release ] editing/deleting/delete-hr.html [ Failure ]
+[ Release ] editing/deleting/delete-image-001.html [ Failure ]
+[ Release ] editing/deleting/delete-image-002.html [ Failure ]
+[ Release ] editing/deleting/delete-image-003.html [ Failure ]
+[ Release ] editing/deleting/delete-line-001.html [ Failure ]
+[ Release ] editing/deleting/delete-line-002.html [ Failure ]
+[ Release ] editing/deleting/delete-line-005.html [ Failure ]
+[ Release ] editing/deleting/delete-line-007.html [ Failure ]
+[ Release ] editing/deleting/delete-line-009.html [ Failure ]
+[ Release ] editing/deleting/delete-line-010.html [ Failure ]
+[ Release ] editing/deleting/delete-line-013.html [ Failure ]
+[ Release ] editing/deleting/delete-line-015.html [ Failure ]
+[ Release ] editing/deleting/delete-line-017.html [ Failure ]
+[ Release ] editing/deleting/delete-listitem-001.html [ Failure ]
+[ Release ] editing/deleting/delete-listitem-002.html [ Failure ]
+[ Release ] editing/deleting/delete-selection-001.html [ Failure ]
+[ Release ] editing/deleting/delete-trailing-ws-001.html [ Failure ]
+[ Release ] editing/deleting/delete-trailing-ws-002.html [ Failure ]
+[ Release ] editing/deleting/delete-ws-fixup-002.html [ Failure ]
+[ Release ] editing/deleting/list-item-1.html [ Failure ]
+[ Release ] editing/deleting/merge-unrendered-space.html [ Failure ]
+[ Release ] editing/deleting/non-smart-delete.html [ Failure ]
+[ Release ] editing/inserting/

[webkit-changes] [205142] trunk/LayoutTests

2016-08-29 Thread jiewen_tan

Title: [205142] trunk/LayoutTests








Revision 205142
Author jiewen_...@apple.com
Date 2016-08-29 13:26:17 -0700 (Mon, 29 Aug 2016)


Log Message
Unreviewed, update iOS simulator WK1 flaky tests.

* platform/ios-simulator-wk1/TestExpectations:

Modified Paths

trunk/LayoutTests/ChangeLog
trunk/LayoutTests/platform/ios-simulator-wk1/TestExpectations




Diff

Modified: trunk/LayoutTests/ChangeLog (205141 => 205142)

--- trunk/LayoutTests/ChangeLog	2016-08-29 20:22:30 UTC (rev 205141)
+++ trunk/LayoutTests/ChangeLog	2016-08-29 20:26:17 UTC (rev 205142)
@@ -1,3 +1,9 @@
+2016-08-29  Jiewen Tan  
+
+Unreviewed, update iOS simulator WK1 flaky tests.
+
+* platform/ios-simulator-wk1/TestExpectations:
+
 2016-08-29  Ryan Haddad  
 
 Unreviewed, rolling out r205140.


Modified: trunk/LayoutTests/platform/ios-simulator-wk1/TestExpectations (205141 => 205142)

--- trunk/LayoutTests/platform/ios-simulator-wk1/TestExpectations	2016-08-29 20:22:30 UTC (rev 205141)
+++ trunk/LayoutTests/platform/ios-simulator-wk1/TestExpectations	2016-08-29 20:26:17 UTC (rev 205142)
@@ -310,11 +310,11 @@
 editing/selection/focus-body.html [ Failure ]
 editing/selection/internal-caret-rect.html [ Failure ]
 editing/selection/last-empty-inline.html [ Failure ]
-editing/selection/mixed-editability-4.html [ Failure ]
+editing/selection/mixed-editability-4.html [ Failure Pass ]
 editing/selection/mixed-editability-9.html [ Failure ]
 editing/selection/modify-up-on-rtl-wrapping-text.html [ Failure ]
 editing/selection/move-begin-end.html [ Failure ]
-editing/selection/move-between-blocks-no-001.html [ Failure ]
+editing/selection/move-between-blocks-no-001.html [ Failure Pass ]
 editing/selection/move-between-lines-of-different-editabilities.html [ Failure ]
 editing/selection/move-by-character-6.html [ Failure ]
 editing/selection/move-by-word-001.html [ Failure ]
@@ -387,15 +387,13 @@
 editing/unsupported-content/list-delete-003.html [ Failure ]
 editing/unsupported-content/table-delete-002.html [ Failure ]
 fast/forms/textfield-outline.html [ Failure ]
-platform/ios-simulator/ios/fast/coordinates/document-elementFromPoint.html [ Failure ]
 platform/ios-simulator/ios/fast/coordinates/element-client-rects.html [ Failure ]
 platform/ios-simulator/ios/fast/coordinates/mouse-event-zoomed.html [ Failure ]
 platform/ios-simulator/ios/fast/coordinates/page-offsets.html [ Failure ]
 platform/ios-simulator/ios/fast/coordinates/range-client-rects.html [ Failure ]
-platform/ios-simulator/ios/fast/coordinates/touch-event-zoomed.html [ Failure ]
 
 #  Many editing/inserting tests have editing delegate callbacks out of order from baseline
-editing/inserting/editing-empty-divs.html [ Failure ]
+editing/inserting/editing-empty-divs.html [ Failure Pass ]
 editing/inserting/insert-3654864-fix.html [ Failure ]
 editing/inserting/insert-3775316-fix.html [ Failure ]
 editing/inserting/insert-after-delete-001.html [ Failure ]
@@ -871,7 +869,6 @@
 fast/loader/user-stylesheet-fast-path.html [ Failure ]
 fast/misc/subtree-layouts.html [ ImageOnlyFailure ]
 fast/multicol/nested-filter-in-columns.html [ ImageOnlyFailure ]
-fast/multicol/newmulticol/hide-box-horizontal-bt.html [ ImageOnlyFailure ]
 fast/overflow/child-100percent-height-inside-fixed-container-with-overflow-auto.html [ Failure ]
 fast/overflow/overflow-focus-ring.html [ Failure ]
 fast/overflow/overflow-update-transform.html [ Failure ]
@@ -1123,8 +1120,6 @@
 http/tests/security/canvas-remote-read-remote-video-allowed-with-credentials.html [ Failure ]
 http/tests/security/contentSecurityPolicy/_javascript_-url-blocked-by-default-src-star.html [ Failure ]
 http/tests/security/contentSecurityPolicy/object-src-url-allowed.html [ Failure ]
-http/tests/security/cross-origin-indexeddb-allowed.html [ Failure ]
-http/tests/security/cross-origin-indexeddb.html [ Failure ]
 http/tests/security/cross-origin-local-storage-wk1.html [ Failure ]
 http/tests/security/dataURL/xss-DENIED-to-data-url-in-foreign-domain-window-open.html [ Failure ]
 http/tests/security/dataURL/xss-DENIED-to-data-url-window-open.html [ Failure ]
@@ -1292,8 +1287,6 @@
 compositing/iframes/display-none-subframe.html [ Failure ]
 css3/blending/blend-mode-simple-composited.html [ Failure ]
 editing/selection/caret-mode-document-begin-end.html [ Failure ]
-fast/events/onchange-passwordfield.html [ Failure ]
-fast/forms/input-text-option-delete.html [ Failure ]
 fast/xsl/xslt-extra-content-at-end.xml [ Failure ]
 fast/xsl/xslt-missing-namespace-in-xslt.xml [ Failure ]
 js/dom/JSON-stringify.html [ Failure ]
@@ -1360,7 +1353,6 @@
 imported/blink/fast/masking/mask-serializing.html [ Crash Pass ]
 imported/blink/fast/multicol/newmulticol/multicol-becomes-regular-block.html [ Crash Pass ]
 imported/blink/fast/scrolling/fractional-scroll-offset-iframe-fixed-position.html [ ImageOnlyFailure ]
-imported/blink/editing/undo/crash-redo-with-iframes.html [ Failure Pass ]
 imported/blink/fast/css-grid-layout/stale-grid-layout-2.html

[webkit-changes] [205141] trunk/LayoutTests

2016-08-29 Thread ryanhaddad

Title: [205141] trunk/LayoutTests








Revision 205141
Author ryanhad...@apple.com
Date 2016-08-29 13:22:30 -0700 (Mon, 29 Aug 2016)


Log Message
Unreviewed, rolling out r205140.

The changelog entry for this commit is incorrect and
misattributed.

Reverted changeset:

"Marking media/track/text-track-cue-is-reachable.html as flaky
on ios-simulator-wk2."
https://bugs.webkit.org/show_bug.cgi?id=161323
http://trac.webkit.org/changeset/205140

Modified Paths

trunk/LayoutTests/ChangeLog
trunk/LayoutTests/platform/ios-simulator-wk1/TestExpectations




Diff

Modified: trunk/LayoutTests/ChangeLog (205140 => 205141)

--- trunk/LayoutTests/ChangeLog	2016-08-29 20:18:19 UTC (rev 205140)
+++ trunk/LayoutTests/ChangeLog	2016-08-29 20:22:30 UTC (rev 205141)
@@ -1,5 +1,19 @@
 2016-08-29  Ryan Haddad  
 
+Unreviewed, rolling out r205140.
+
+The changelog entry for this commit is incorrect and
+misattributed.
+
+Reverted changeset:
+
+"Marking media/track/text-track-cue-is-reachable.html as flaky
+on ios-simulator-wk2."
+https://bugs.webkit.org/show_bug.cgi?id=161323
+http://trac.webkit.org/changeset/205140
+
+2016-08-29  Ryan Haddad  
+
 Marking media/track/text-track-cue-is-reachable.html as flaky on ios-simulator-wk2.
 https://bugs.webkit.org/show_bug.cgi?id=161323
 


Modified: trunk/LayoutTests/platform/ios-simulator-wk1/TestExpectations (205140 => 205141)

--- trunk/LayoutTests/platform/ios-simulator-wk1/TestExpectations	2016-08-29 20:18:19 UTC (rev 205140)
+++ trunk/LayoutTests/platform/ios-simulator-wk1/TestExpectations	2016-08-29 20:22:30 UTC (rev 205141)
@@ -310,11 +310,11 @@
 editing/selection/focus-body.html [ Failure ]
 editing/selection/internal-caret-rect.html [ Failure ]
 editing/selection/last-empty-inline.html [ Failure ]
-editing/selection/mixed-editability-4.html [ Failure Pass ]
+editing/selection/mixed-editability-4.html [ Failure ]
 editing/selection/mixed-editability-9.html [ Failure ]
 editing/selection/modify-up-on-rtl-wrapping-text.html [ Failure ]
 editing/selection/move-begin-end.html [ Failure ]
-editing/selection/move-between-blocks-no-001.html [ Failure Pass ]
+editing/selection/move-between-blocks-no-001.html [ Failure ]
 editing/selection/move-between-lines-of-different-editabilities.html [ Failure ]
 editing/selection/move-by-character-6.html [ Failure ]
 editing/selection/move-by-word-001.html [ Failure ]
@@ -387,13 +387,15 @@
 editing/unsupported-content/list-delete-003.html [ Failure ]
 editing/unsupported-content/table-delete-002.html [ Failure ]
 fast/forms/textfield-outline.html [ Failure ]
+platform/ios-simulator/ios/fast/coordinates/document-elementFromPoint.html [ Failure ]
 platform/ios-simulator/ios/fast/coordinates/element-client-rects.html [ Failure ]
 platform/ios-simulator/ios/fast/coordinates/mouse-event-zoomed.html [ Failure ]
 platform/ios-simulator/ios/fast/coordinates/page-offsets.html [ Failure ]
 platform/ios-simulator/ios/fast/coordinates/range-client-rects.html [ Failure ]
+platform/ios-simulator/ios/fast/coordinates/touch-event-zoomed.html [ Failure ]
 
 #  Many editing/inserting tests have editing delegate callbacks out of order from baseline
-editing/inserting/editing-empty-divs.html [ Failure Pass ]
+editing/inserting/editing-empty-divs.html [ Failure ]
 editing/inserting/insert-3654864-fix.html [ Failure ]
 editing/inserting/insert-3775316-fix.html [ Failure ]
 editing/inserting/insert-after-delete-001.html [ Failure ]
@@ -869,6 +871,7 @@
 fast/loader/user-stylesheet-fast-path.html [ Failure ]
 fast/misc/subtree-layouts.html [ ImageOnlyFailure ]
 fast/multicol/nested-filter-in-columns.html [ ImageOnlyFailure ]
+fast/multicol/newmulticol/hide-box-horizontal-bt.html [ ImageOnlyFailure ]
 fast/overflow/child-100percent-height-inside-fixed-container-with-overflow-auto.html [ Failure ]
 fast/overflow/overflow-focus-ring.html [ Failure ]
 fast/overflow/overflow-update-transform.html [ Failure ]
@@ -1120,6 +1123,8 @@
 http/tests/security/canvas-remote-read-remote-video-allowed-with-credentials.html [ Failure ]
 http/tests/security/contentSecurityPolicy/_javascript_-url-blocked-by-default-src-star.html [ Failure ]
 http/tests/security/contentSecurityPolicy/object-src-url-allowed.html [ Failure ]
+http/tests/security/cross-origin-indexeddb-allowed.html [ Failure ]
+http/tests/security/cross-origin-indexeddb.html [ Failure ]
 http/tests/security/cross-origin-local-storage-wk1.html [ Failure ]
 http/tests/security/dataURL/xss-DENIED-to-data-url-in-foreign-domain-window-open.html [ Failure ]
 http/tests/security/dataURL/xss-DENIED-to-data-url-window-open.html [ Failure ]
@@ -1287,6 +1292,8 @@
 compositing/iframes/display-none-subframe.html [ Failure ]
 css3/blending/blend-mode-simple-composited.html [ Failure ]
 editing/selection/caret-mode-document-begin-end.html [ Failure ]
+fast/events/onchange-passwordfield.html [ Failure ]
+fast/forms/input-text-option-delete.html [ Failure ]
 fast/

[webkit-changes] [205140] trunk/LayoutTests

2016-08-29 Thread jiewen_tan

Title: [205140] trunk/LayoutTests








Revision 205140
Author jiewen_...@apple.com
Date 2016-08-29 13:18:19 -0700 (Mon, 29 Aug 2016)


Log Message
Marking media/track/text-track-cue-is-reachable.html as flaky on ios-simulator-wk2.
https://bugs.webkit.org/show_bug.cgi?id=161323

Unreviewed test gardening.

Patch by Ryan Haddad  on 2016-08-29

* platform/ios-simulator-wk2/TestExpectations:

Modified Paths

trunk/LayoutTests/ChangeLog
trunk/LayoutTests/platform/ios-simulator-wk1/TestExpectations




Diff

Modified: trunk/LayoutTests/ChangeLog (205139 => 205140)

--- trunk/LayoutTests/ChangeLog	2016-08-29 20:08:50 UTC (rev 205139)
+++ trunk/LayoutTests/ChangeLog	2016-08-29 20:18:19 UTC (rev 205140)
@@ -147,6 +147,12 @@
 Unreviewed, update iOS simulator WK1 flaky tests.
 
 * platform/ios-simulator-wk1/TestExpectations:
+
+2016-08-29  Jiewen Tan  
+
+Unreviewed, update iOS simulator WK1 flaky tests.
+
+* platform/ios-simulator-wk1/TestExpectations:
 * platform/ios-simulator-wk2/TestExpectations:
 * platform/ios-simulator/TestExpectations:
 


Modified: trunk/LayoutTests/platform/ios-simulator-wk1/TestExpectations (205139 => 205140)

--- trunk/LayoutTests/platform/ios-simulator-wk1/TestExpectations	2016-08-29 20:08:50 UTC (rev 205139)
+++ trunk/LayoutTests/platform/ios-simulator-wk1/TestExpectations	2016-08-29 20:18:19 UTC (rev 205140)
@@ -310,11 +310,11 @@
 editing/selection/focus-body.html [ Failure ]
 editing/selection/internal-caret-rect.html [ Failure ]
 editing/selection/last-empty-inline.html [ Failure ]
-editing/selection/mixed-editability-4.html [ Failure ]
+editing/selection/mixed-editability-4.html [ Failure Pass ]
 editing/selection/mixed-editability-9.html [ Failure ]
 editing/selection/modify-up-on-rtl-wrapping-text.html [ Failure ]
 editing/selection/move-begin-end.html [ Failure ]
-editing/selection/move-between-blocks-no-001.html [ Failure ]
+editing/selection/move-between-blocks-no-001.html [ Failure Pass ]
 editing/selection/move-between-lines-of-different-editabilities.html [ Failure ]
 editing/selection/move-by-character-6.html [ Failure ]
 editing/selection/move-by-word-001.html [ Failure ]
@@ -387,15 +387,13 @@
 editing/unsupported-content/list-delete-003.html [ Failure ]
 editing/unsupported-content/table-delete-002.html [ Failure ]
 fast/forms/textfield-outline.html [ Failure ]
-platform/ios-simulator/ios/fast/coordinates/document-elementFromPoint.html [ Failure ]
 platform/ios-simulator/ios/fast/coordinates/element-client-rects.html [ Failure ]
 platform/ios-simulator/ios/fast/coordinates/mouse-event-zoomed.html [ Failure ]
 platform/ios-simulator/ios/fast/coordinates/page-offsets.html [ Failure ]
 platform/ios-simulator/ios/fast/coordinates/range-client-rects.html [ Failure ]
-platform/ios-simulator/ios/fast/coordinates/touch-event-zoomed.html [ Failure ]
 
 #  Many editing/inserting tests have editing delegate callbacks out of order from baseline
-editing/inserting/editing-empty-divs.html [ Failure ]
+editing/inserting/editing-empty-divs.html [ Failure Pass ]
 editing/inserting/insert-3654864-fix.html [ Failure ]
 editing/inserting/insert-3775316-fix.html [ Failure ]
 editing/inserting/insert-after-delete-001.html [ Failure ]
@@ -871,7 +869,6 @@
 fast/loader/user-stylesheet-fast-path.html [ Failure ]
 fast/misc/subtree-layouts.html [ ImageOnlyFailure ]
 fast/multicol/nested-filter-in-columns.html [ ImageOnlyFailure ]
-fast/multicol/newmulticol/hide-box-horizontal-bt.html [ ImageOnlyFailure ]
 fast/overflow/child-100percent-height-inside-fixed-container-with-overflow-auto.html [ Failure ]
 fast/overflow/overflow-focus-ring.html [ Failure ]
 fast/overflow/overflow-update-transform.html [ Failure ]
@@ -1123,8 +1120,6 @@
 http/tests/security/canvas-remote-read-remote-video-allowed-with-credentials.html [ Failure ]
 http/tests/security/contentSecurityPolicy/_javascript_-url-blocked-by-default-src-star.html [ Failure ]
 http/tests/security/contentSecurityPolicy/object-src-url-allowed.html [ Failure ]
-http/tests/security/cross-origin-indexeddb-allowed.html [ Failure ]
-http/tests/security/cross-origin-indexeddb.html [ Failure ]
 http/tests/security/cross-origin-local-storage-wk1.html [ Failure ]
 http/tests/security/dataURL/xss-DENIED-to-data-url-in-foreign-domain-window-open.html [ Failure ]
 http/tests/security/dataURL/xss-DENIED-to-data-url-window-open.html [ Failure ]
@@ -1292,8 +1287,6 @@
 compositing/iframes/display-none-subframe.html [ Failure ]
 css3/blending/blend-mode-simple-composited.html [ Failure ]
 editing/selection/caret-mode-document-begin-end.html [ Failure ]
-fast/events/onchange-passwordfield.html [ Failure ]
-fast/forms/input-text-option-delete.html [ Failure ]
 fast/xsl/xslt-extra-content-at-end.xml [ Failure ]
 fast/xsl/xslt-missing-namespace-in-xslt.xml [ Failure ]
 js/dom/JSON-stringify.html [ Failure ]
@@ -1360,7 +1353,6 @@
 imported/blink/fast/masking/mask-serializing.html [ Crash Pass ]
 imported/blink/fast/multic

[webkit-changes] [205139] trunk/LayoutTests

2016-08-29 Thread ryanhaddad

Title: [205139] trunk/LayoutTests








Revision 205139
Author ryanhad...@apple.com
Date 2016-08-29 13:08:50 -0700 (Mon, 29 Aug 2016)


Log Message
Marking media/track/text-track-cue-is-reachable.html as flaky on ios-simulator-wk2.
https://bugs.webkit.org/show_bug.cgi?id=161323

Unreviewed test gardening.

* platform/ios-simulator-wk2/TestExpectations:

Modified Paths

trunk/LayoutTests/ChangeLog
trunk/LayoutTests/platform/ios-simulator-wk2/TestExpectations




Diff

Modified: trunk/LayoutTests/ChangeLog (205138 => 205139)

--- trunk/LayoutTests/ChangeLog	2016-08-29 19:55:11 UTC (rev 205138)
+++ trunk/LayoutTests/ChangeLog	2016-08-29 20:08:50 UTC (rev 205139)
@@ -1,3 +1,12 @@
+2016-08-29  Ryan Haddad  
+
+Marking media/track/text-track-cue-is-reachable.html as flaky on ios-simulator-wk2.
+https://bugs.webkit.org/show_bug.cgi?id=161323
+
+Unreviewed test gardening.
+
+* platform/ios-simulator-wk2/TestExpectations:
+
 2016-08-29  Chris Dumez  
 
 We should throw a SecurityError when denying access to cross-origin Window properties


Modified: trunk/LayoutTests/platform/ios-simulator-wk2/TestExpectations (205138 => 205139)

--- trunk/LayoutTests/platform/ios-simulator-wk2/TestExpectations	2016-08-29 19:55:11 UTC (rev 205138)
+++ trunk/LayoutTests/platform/ios-simulator-wk2/TestExpectations	2016-08-29 20:08:50 UTC (rev 205139)
@@ -1859,6 +1859,7 @@
 webkit.org/b/160368 [ Release ] http/tests/xmlhttprequest/workers/methods-async.html [ Pass Timeout ]
 
 webkit.org/b/160367 media/track/media-element-enqueue-event-crash.html [ Pass Crash ]
+webkit.org/b/161323 media/track/text-track-cue-is-reachable.html [ Pass Crash ]
 webkit.org/b/142152 media/track/track-in-band-cues-added-once.html [ Pass Failure ]
 webkit.org/b/160367 media/track/track-remove-crash.html [ Skip ]
 






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

[webkit-changes] [205138] trunk

2016-08-29 Thread cdumez

Title: [205138] trunk








Revision 205138
Author cdu...@apple.com
Date 2016-08-29 12:55:11 -0700 (Mon, 29 Aug 2016)


Log Message
document.createEvent("popstateevent") should create a PopStateEvent
https://bugs.webkit.org/show_bug.cgi?id=161321

Reviewed by Darin Adler.

LayoutTests/imported/w3c:

Rebaseline several W3C tests now that more checks are passing.

* web-platform-tests/dom/events/EventTarget-dispatchEvent-expected.txt:
* web-platform-tests/dom/nodes/Document-createEvent-expected.txt:
* web-platform-tests/html/browsers/browsing-the-web/history-traversal/PopStateEvent-expected.txt:

Source/WebCore:

document.createEvent("popstateevent") should create a PopStateEvent as per:
- https://dom.spec.whatwg.org/#dom-document-createevent

Firefox and Chrome match the specification but WebKit throws an exception,
which is risky compatibility-wise.

No new tests, rebaselined existing tests.

* dom/Document.cpp:
(WebCore::Document::createEvent):
* dom/PopStateEvent.cpp:
(WebCore::PopStateEvent::createForBindings):
* dom/PopStateEvent.h:

Modified Paths

trunk/LayoutTests/imported/w3c/ChangeLog
trunk/LayoutTests/imported/w3c/web-platform-tests/dom/events/EventTarget-dispatchEvent-expected.txt
trunk/LayoutTests/imported/w3c/web-platform-tests/dom/nodes/Document-createEvent-expected.txt
trunk/LayoutTests/imported/w3c/web-platform-tests/html/browsers/browsing-the-web/history-traversal/PopStateEvent-expected.txt
trunk/Source/WebCore/ChangeLog
trunk/Source/WebCore/dom/Document.cpp
trunk/Source/WebCore/dom/PopStateEvent.cpp
trunk/Source/WebCore/dom/PopStateEvent.h




Diff

Modified: trunk/LayoutTests/imported/w3c/ChangeLog (205137 => 205138)

--- trunk/LayoutTests/imported/w3c/ChangeLog	2016-08-29 19:44:48 UTC (rev 205137)
+++ trunk/LayoutTests/imported/w3c/ChangeLog	2016-08-29 19:55:11 UTC (rev 205138)
@@ -1,5 +1,18 @@
 2016-08-29  Chris Dumez  
 
+document.createEvent("popstateevent") should create a PopStateEvent
+https://bugs.webkit.org/show_bug.cgi?id=161321
+
+Reviewed by Darin Adler.
+
+Rebaseline several W3C tests now that more checks are passing.
+
+* web-platform-tests/dom/events/EventTarget-dispatchEvent-expected.txt:
+* web-platform-tests/dom/nodes/Document-createEvent-expected.txt:
+* web-platform-tests/html/browsers/browsing-the-web/history-traversal/PopStateEvent-expected.txt:
+
+2016-08-29  Chris Dumez  
+
 We should throw a SecurityError when denying access to cross-origin Window properties
 https://bugs.webkit.org/show_bug.cgi?id=161316
 


Modified: trunk/LayoutTests/imported/w3c/web-platform-tests/dom/events/EventTarget-dispatchEvent-expected.txt (205137 => 205138)

--- trunk/LayoutTests/imported/w3c/web-platform-tests/dom/events/EventTarget-dispatchEvent-expected.txt	2016-08-29 19:44:48 UTC (rev 205137)
+++ trunk/LayoutTests/imported/w3c/web-platform-tests/dom/events/EventTarget-dispatchEvent-expected.txt	2016-08-29 19:55:11 UTC (rev 205138)
@@ -21,7 +21,7 @@
 PASS If the event's initialized flag is not set, an InvalidStateError must be thrown (MouseEvent). 
 PASS If the event's initialized flag is not set, an InvalidStateError must be thrown (MouseEvents). 
 FAIL If the event's initialized flag is not set, an InvalidStateError must be thrown (PageTransitionEvent). The operation is not supported.
-FAIL If the event's initialized flag is not set, an InvalidStateError must be thrown (PopStateEvent). The operation is not supported.
+PASS If the event's initialized flag is not set, an InvalidStateError must be thrown (PopStateEvent). 
 FAIL If the event's initialized flag is not set, an InvalidStateError must be thrown (ProgressEvent). The operation is not supported.
 PASS If the event's initialized flag is not set, an InvalidStateError must be thrown (StorageEvent). 
 FAIL If the event's initialized flag is not set, an InvalidStateError must be thrown (SVGEvents). The operation is not supported.


Modified: trunk/LayoutTests/imported/w3c/web-platform-tests/dom/nodes/Document-createEvent-expected.txt (205137 => 205138)

--- trunk/LayoutTests/imported/w3c/web-platform-tests/dom/nodes/Document-createEvent-expected.txt	2016-08-29 19:44:48 UTC (rev 205137)
+++ trunk/LayoutTests/imported/w3c/web-platform-tests/dom/nodes/Document-createEvent-expected.txt	2016-08-29 19:55:11 UTC (rev 205138)
@@ -135,12 +135,12 @@
 FAIL PAGETRANSITIONEVENT should be an alias for PageTransitionEvent. The operation is not supported.
 FAIL createEvent('PAGETRANSITIONEVENT') should be initialized correctly. undefined is not an object (evaluating 'ev.type')
 PASS Should throw NOT_SUPPORTED_ERR for pluralized legacy event interface "PageTransitionEvents" 
-FAIL PopStateEvent should be an alias for PopStateEvent. The operation is not supported.
-FAIL createEvent('PopStateEvent') should be initialized correctly. undefined is not an object (evaluating 'ev.type')
-FAIL popstateevent should be an alias for PopStateEvent. The operation is not supported.
-FAIL

[webkit-changes] [205137] trunk/Source/WebKit2

2016-08-29 Thread andersca

Title: [205137] trunk/Source/WebKit2








Revision 205137
Author ander...@apple.com
Date 2016-08-29 12:44:48 -0700 (Mon, 29 Aug 2016)


Log Message
Stop using WebProcessProxy::fromConnection in WebPageProxy
https://bugs.webkit.org/show_bug.cgi?id=161322

Reviewed by Dan Bernstein.

The WebPageProxy already knows its WebProcessProxy - no need to look it up from the connection.

* UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::handleMessage):
(WebKit::WebPageProxy::handleSynchronousMessage):

Modified Paths

trunk/Source/WebKit2/ChangeLog
trunk/Source/WebKit2/UIProcess/WebPageProxy.cpp




Diff

Modified: trunk/Source/WebKit2/ChangeLog (205136 => 205137)

--- trunk/Source/WebKit2/ChangeLog	2016-08-29 19:37:01 UTC (rev 205136)
+++ trunk/Source/WebKit2/ChangeLog	2016-08-29 19:44:48 UTC (rev 205137)
@@ -1,5 +1,18 @@
 2016-08-29  Anders Carlsson  
 
+Stop using WebProcessProxy::fromConnection in WebPageProxy
+https://bugs.webkit.org/show_bug.cgi?id=161322
+
+Reviewed by Dan Bernstein.
+
+The WebPageProxy already knows its WebProcessProxy - no need to look it up from the connection.
+
+* UIProcess/WebPageProxy.cpp:
+(WebKit::WebPageProxy::handleMessage):
+(WebKit::WebPageProxy::handleSynchronousMessage):
+
+2016-08-29  Anders Carlsson  
+
 Move some structs into Connection.cpp
 https://bugs.webkit.org/show_bug.cgi?id=161320
 


Modified: trunk/Source/WebKit2/UIProcess/WebPageProxy.cpp (205136 => 205137)

--- trunk/Source/WebKit2/UIProcess/WebPageProxy.cpp	2016-08-29 19:37:01 UTC (rev 205136)
+++ trunk/Source/WebKit2/UIProcess/WebPageProxy.cpp	2016-08-29 19:44:48 UTC (rev 205137)
@@ -669,23 +669,26 @@
 
 void WebPageProxy::handleMessage(IPC::Connection& connection, const String& messageName, const WebKit::UserData& messageBody)
 {
-auto* webProcessProxy = WebProcessProxy::fromConnection(&connection);
-if (!webProcessProxy || !m_injectedBundleClient)
+ASSERT(m_process->connection() == &connection);
+
+if (!m_injectedBundleClient)
 return;
-m_injectedBundleClient->didReceiveMessageFromInjectedBundle(this, messageName, webProcessProxy->transformHandlesToObjects(messageBody.object()).get());
+
+m_injectedBundleClient->didReceiveMessageFromInjectedBundle(this, messageName, m_process->transformHandlesToObjects(messageBody.object()).get());
 }
 
 void WebPageProxy::handleSynchronousMessage(IPC::Connection& connection, const String& messageName, const UserData& messageBody, UserData& returnUserData)
 {
-if (!WebProcessProxy::fromConnection(&connection) || !m_injectedBundleClient)
+ASSERT(m_process->connection() == &connection);
+
+if (!m_injectedBundleClient)
 return;
 
 RefPtr returnData;
-m_injectedBundleClient->didReceiveSynchronousMessageFromInjectedBundle(this, messageName, WebProcessProxy::fromConnection(&connection)->transformHandlesToObjects(messageBody.object()).get(), returnData);
-returnUserData = UserData(WebProcessProxy::fromConnection(&connection)->transformObjectsToHandles(returnData.get()));
+m_injectedBundleClient->didReceiveSynchronousMessageFromInjectedBundle(this, messageName, m_process->transformHandlesToObjects(messageBody.object()).get(), returnData);
+returnUserData = UserData(m_process->transformObjectsToHandles(returnData.get()));
 }
 
-
 void WebPageProxy::reattachToWebProcess()
 {
 ASSERT(!m_isClosed);






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

[webkit-changes] [205135] trunk/Tools

2016-08-29 Thread commit-queue

Title: [205135] trunk/Tools








Revision 205135
Author commit-qu...@webkit.org
Date 2016-08-29 12:36:36 -0700 (Mon, 29 Aug 2016)


Log Message
Bug 161165 - check-webkit-style doesn't understand macros containing `else` or labels
https://bugs.webkit.org/show_bug.cgi?id=161165

Patch by JF Bastien  on 2016-08-29
Reviewed by Geoffrey Garen.

Fix handling of `else` in a macro. Don't handle labels in a macro:
the checker is already pretty messy in that area and it should be
a rare thing.

* Scripts/webkitpy/style/checkers/cpp.py:
(check_braces):
* Scripts/webkitpy/style/checkers/cpp_unittest.py:
(CppStyleTest.test_else_clause_not_on_same_line_as_else):

Modified Paths

trunk/Tools/ChangeLog
trunk/Tools/Scripts/webkitpy/style/checkers/cpp.py
trunk/Tools/Scripts/webkitpy/style/checkers/cpp_unittest.py




Diff

Modified: trunk/Tools/ChangeLog (205134 => 205135)

--- trunk/Tools/ChangeLog	2016-08-29 19:07:59 UTC (rev 205134)
+++ trunk/Tools/ChangeLog	2016-08-29 19:36:36 UTC (rev 205135)
@@ -1,3 +1,19 @@
+2016-08-29  JF Bastien  
+
+Bug 161165 - check-webkit-style doesn't understand macros containing `else` or labels
+https://bugs.webkit.org/show_bug.cgi?id=161165
+
+Reviewed by Geoffrey Garen.
+
+Fix handling of `else` in a macro. Don't handle labels in a macro:
+the checker is already pretty messy in that area and it should be
+a rare thing.
+
+* Scripts/webkitpy/style/checkers/cpp.py:
+(check_braces):
+* Scripts/webkitpy/style/checkers/cpp_unittest.py:
+(CppStyleTest.test_else_clause_not_on_same_line_as_else):
+
 2016-08-29  Alex Christensen  
 
 API test URLParserTest.ParserFailures failing ASSERT_NOT_REACHED


Modified: trunk/Tools/Scripts/webkitpy/style/checkers/cpp.py (205134 => 205135)

--- trunk/Tools/Scripts/webkitpy/style/checkers/cpp.py	2016-08-29 19:07:59 UTC (rev 205134)
+++ trunk/Tools/Scripts/webkitpy/style/checkers/cpp.py	2016-08-29 19:36:36 UTC (rev 205135)
@@ -2507,7 +2507,9 @@
   'An else should appear on the same line as the preceding }')
 
 # Likewise, an else should never have the else clause on the same line
-if search(r'\belse [^\s{]', line) and not search(r'\belse if\b', line):
+if (search(r'\belse [^\s{]', line)
+and not search(r'\belse if\b', line)
+and not search(r'\belse\s*\\$', line)):
 error(line_number, 'whitespace/newline', 4,
   'Else clause should never be on same line as else (use 2 lines)')
 


Modified: trunk/Tools/Scripts/webkitpy/style/checkers/cpp_unittest.py (205134 => 205135)

--- trunk/Tools/Scripts/webkitpy/style/checkers/cpp_unittest.py	2016-08-29 19:07:59 UTC (rev 205134)
+++ trunk/Tools/Scripts/webkitpy/style/checkers/cpp_unittest.py	2016-08-29 19:36:36 UTC (rev 205135)
@@ -2271,6 +2271,7 @@
  '(use 2 lines)  [whitespace/newline] [4]')
 self.assert_lint('else if (blah) {', '')
 self.assert_lint('variable_ends_in_else = true;', '')
+self.assert_lint('else \\', '')
 
 def test_comma(self):
 self.assert_lint('a = f(1,2);',






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

[webkit-changes] [205134] trunk

2016-08-29 Thread commit-queue

Title: [205134] trunk








Revision 205134
Author commit-qu...@webkit.org
Date 2016-08-29 12:07:59 -0700 (Mon, 29 Aug 2016)


Log Message
Image Loader should use FetchOptions::mode according its crossOrigin attribute
https://bugs.webkit.org/show_bug.cgi?id=161309

Patch by Youenn Fablet  on 2016-08-29
Reviewed by Darin Adler.

Source/WebCore:

Test: http/tests/security/load-image-after-redirection.html

* loader/ImageLoader.cpp:
(WebCore::ImageLoader::updateFromElement): Using CachedResourceRequest::setAsPotentiallyCrossOrigin to set fetch mode.

LayoutTests:

* http/tests/security/load-image-after-redirection-expected.txt: Added.
* http/tests/security/load-image-after-redirection.html: Added.

Modified Paths

trunk/LayoutTests/ChangeLog
trunk/Source/WebCore/ChangeLog
trunk/Source/WebCore/loader/ImageLoader.cpp


Added Paths

trunk/LayoutTests/http/tests/security/load-image-after-redirection-expected.txt
trunk/LayoutTests/http/tests/security/load-image-after-redirection.html




Diff

Modified: trunk/LayoutTests/ChangeLog (205133 => 205134)

--- trunk/LayoutTests/ChangeLog	2016-08-29 19:07:22 UTC (rev 205133)
+++ trunk/LayoutTests/ChangeLog	2016-08-29 19:07:59 UTC (rev 205134)
@@ -1,3 +1,13 @@
+2016-08-29  Youenn Fablet  
+
+Image Loader should use FetchOptions::mode according its crossOrigin attribute
+https://bugs.webkit.org/show_bug.cgi?id=161309
+
+Reviewed by Darin Adler.
+
+* http/tests/security/load-image-after-redirection-expected.txt: Added.
+* http/tests/security/load-image-after-redirection.html: Added.
+
 2016-08-29  Jiewen Tan  
 
 Unreviewed, update iOS simulator WK1 flaky tests.


Added: trunk/LayoutTests/http/tests/security/load-image-after-redirection-expected.txt (0 => 205134)

--- trunk/LayoutTests/http/tests/security/load-image-after-redirection-expected.txt	(rev 0)
+++ trunk/LayoutTests/http/tests/security/load-image-after-redirection-expected.txt	2016-08-29 19:07:59 UTC (rev 205134)
@@ -0,0 +1,7 @@
+CONSOLE MESSAGE: Cross-origin redirection to http://127.0.0.1:8000/security/resources/abe-allow-star.php denied by Cross-Origin Resource Sharing policy: Origin http://127.0.0.1:8000 is not allowed by Access-Control-Allow-Origin.
+CONSOLE MESSAGE: Cross-origin image load denied by Cross-Origin Resource Sharing policy.
+ 
+
+PASS Loading an image in no-cors mode after a CORS invalid redirection is ok 
+PASS Loading an image in cors mode after a CORS invalid redirection is stopped 
+


Added: trunk/LayoutTests/http/tests/security/load-image-after-redirection.html (0 => 205134)

--- trunk/LayoutTests/http/tests/security/load-image-after-redirection.html	(rev 0)
+++ trunk/LayoutTests/http/tests/security/load-image-after-redirection.html	2016-08-29 19:07:59 UTC (rev 205134)
@@ -0,0 +1,33 @@
+
+
+
+Testing redirection with images fetched with cors mode
+
+
+
+
+
+