Skip to main content

Browse the archive

Revision 15b0fb362111166b6d279074071e27dbae6aee37 authored by Majid Valipour on 25 October 2017, 18:18:27 UTC, committed by Blink WPT Bot on 25 October 2017, 18:28:47 UTC 
Revert "Rename CSS scroll-boundary-behavior to overscroll-behavior"
 
This reverts commit 74e8a0f74fc589a58258655f59aa7f1efa666155.

Reason for revert: This caused an issue on Linux64 no-op build.
Bug: 778310

Original change's description:
> Rename CSS scroll-boundary-behavior to overscroll-behavior
>
> The name change was decided here [1].
>
> This is the minimal patch to change the publicly exposed CSS property.
> It is intentionally small to make it easier to merge with M63. So,
> internally Blink and content still use ScrollBoundaryBehavior name which will
> be updated in the follow up patch larger patch.
>
>
> [1] https://github.com/WICG/scroll-boundary-behavior/issues/24
>
> Bug: 776776
> Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
> Change-Id: Iaa6ad62253ed7fe9ed7f0ee9865ffda852b17801
> Reviewed-on: https://chromium-review.googlesource.com/737090
> Reviewed-by: Dimitri Glazkov <dglazkov@chromium.org>
> Reviewed-by: Sandra Sun <sunyunjia@chromium.org>
> Commit-Queue: Majid Valipour <majidvp@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#511493}

TBR=majidvp@chromium.org,sunyunjia@chromium.org,dglazkov@chromium.org

Change-Id: I1d71b71c5d3c263fb2ff13c9d7a9184399df946f
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 776776
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Reviewed-on: https://chromium-review.googlesource.com/738294
Reviewed-by: Majid Valipour <majidvp@chromium.org>
Commit-Queue: Majid Valipour <majidvp@chromium.org>
Cr-Commit-Position: refs/heads/master@{#511515}
1 parent 511cb08
Raw File
block-string-assignment-to-script-src.tentative.html 
<!DOCTYPE html>
<html>
<head>
  <script src="/resources/testharness.js"></script>
  <script src="/resources/testharnessreport.js"></script>
  <script src="./support/helper.js"></script>

  <meta http-equiv="Content-Security-Policy" content="require-trusted-types">
</head>
<body>
<script>
  // String assignments throw.
  test(t => {
    var s = document.createElement('script');
    assert_throws(new TypeError(), _ => {
      s.src = URLS.safe;
    });
    assert_equals('', s.src);
  }, "'string'");

  // TrustedURL assignments throw.
  test(t => {
    var url = TrustedURL.unsafelyCreate(URLS.safe);

    var s = document.createElement('script');
    assert_throws(new TypeError(), _ => {
      s.src = url;
    });
    assert_equals('', s.src);
  }, "TrustedURL(safe)");

  // TrustedScriptURL assignments work.
  test(t => {
    var url = TrustedScriptURL.unsafelyCreate(URLS.safe);

    var s = document.createElement('script');
    s.src = url;
    assert_equals(url + '', s.src);
  }, "TrustedScriptURL(safe)");

  test(t => {
    var url = TrustedScriptURL.unsafelyCreate(URLS.javascript);

    var s = document.createElement('script');
    s.src = url;
    assert_equals(url + '', s.src);
  }, "TrustedScriptURL(javascript)");

  test(t => {
    var url = TrustedScriptURL.unsafelyCreate(URLS.external);

    var s = document.createElement('script');
    s.src = url;
    assert_equals(url + '', s.src);
  }, "TrustedScriptURL(external)");
</script>
back to top