Move test for navigator.deviceMemory to its own dir with OWNERS [chromium/src : master]

0 views
Skip to first unread message

Philip Jägenstedt (Gerrit)

unread,
Jan 19, 2018, 11:20:12 AM1/19/18
to blink-revie...@chromium.org, blink-...@chromium.org, Fadi Meawad, Shubhie Panicker, Commit Bot, chromium...@chromium.org

PTAL, in particular to see if I got the TEAM and COMPONENT right.

And thanks for upstreaming your tests to wpt :)

View Change

    To view, visit change 876423. To unsubscribe, or for help writing mail filters, visit settings.

    Gerrit-Project: chromium/src
    Gerrit-Branch: master
    Gerrit-MessageType: comment
    Gerrit-Change-Id: If32f07b659aca91a63fd17ca80de4afb1cd525c2
    Gerrit-Change-Number: 876423
    Gerrit-PatchSet: 1
    Gerrit-Owner: Philip Jägenstedt <foo...@chromium.org>
    Gerrit-Reviewer: Fadi Meawad <fme...@chromium.org>
    Gerrit-Reviewer: Philip Jägenstedt <foo...@chromium.org>
    Gerrit-Reviewer: Shubhie Panicker <pani...@chromium.org>
    Gerrit-CC: Commit Bot <commi...@chromium.org>
    Gerrit-Comment-Date: Fri, 19 Jan 2018 16:20:07 +0000
    Gerrit-HasComments: No
    Gerrit-HasLabels: No

    Blink WPT Bot (Gerrit)

    unread,
    Jan 19, 2018, 11:24:23 AM1/19/18
    to Philip Jägenstedt, blink-revie...@chromium.org, blink-...@chromium.org, Fadi Meawad, Shubhie Panicker, Commit Bot, chromium...@chromium.org

    Exportable changes to web-platform-tests were detected in this CL and a pull request in the upstream repo has been made: https://github.com/w3c/web-platform-tests/pull/9107.

    If this CL lands and Travis CI upstream is green, we will auto-merge the PR.

    Note: Please check the Travis CI status (at the bottom of the PR) before landing this CL and only land this CL if the status is green. Otherwise a human needs to step in and resolve it manually. (This may be automated in the future, see https://crbug.com/711447)

    WPT Export docs:
    https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md#Automatic-export-process

    View Change

      To view, visit change 876423. To unsubscribe, or for help writing mail filters, visit settings.

      Gerrit-Project: chromium/src
      Gerrit-Branch: master
      Gerrit-MessageType: comment
      Gerrit-Change-Id: If32f07b659aca91a63fd17ca80de4afb1cd525c2
      Gerrit-Change-Number: 876423
      Gerrit-PatchSet: 1
      Gerrit-Owner: Philip Jägenstedt <foo...@chromium.org>
      Gerrit-Reviewer: Fadi Meawad <fme...@chromium.org>
      Gerrit-Reviewer: Philip Jägenstedt <foo...@chromium.org>
      Gerrit-Reviewer: Shubhie Panicker <pani...@chromium.org>
      Gerrit-CC: Blink WPT Bot <blink-w3c-te...@chromium.org>
      Gerrit-CC: Commit Bot <commi...@chromium.org>
      Gerrit-Comment-Date: Fri, 19 Jan 2018 16:24:19 +0000
      Gerrit-HasComments: No
      Gerrit-HasLabels: No

      Fadi Meawad (Gerrit)

      unread,
      Jan 19, 2018, 4:13:39 PM1/19/18
      to Philip Jägenstedt, blink-revie...@chromium.org, blink-...@chromium.org, Fadi Meawad, Blink WPT Bot, Shubhie Panicker, Commit Bot, chromium...@chromium.org

      Patch set 1:Code-Review +1

      View Change

      1 comment:

      To view, visit change 876423. To unsubscribe, or for help writing mail filters, visit settings.

      Gerrit-Project: chromium/src
      Gerrit-Branch: master
      Gerrit-MessageType: comment
      Gerrit-Change-Id: If32f07b659aca91a63fd17ca80de4afb1cd525c2
      Gerrit-Change-Number: 876423
      Gerrit-PatchSet: 1
      Gerrit-Owner: Philip Jägenstedt <foo...@chromium.org>
      Gerrit-Reviewer: Fadi Meawad <fme...@chromium.org>
      Gerrit-Reviewer: Philip Jägenstedt <foo...@chromium.org>
      Gerrit-Reviewer: Shubhie Panicker <pani...@chromium.org>
      Gerrit-CC: Blink WPT Bot <blink-w3c-te...@chromium.org>
      Gerrit-CC: Commit Bot <commi...@chromium.org>
      Gerrit-Comment-Date: Fri, 19 Jan 2018 21:13:37 +0000
      Gerrit-HasComments: Yes
      Gerrit-HasLabels: Yes

      Shubhie Panicker (Gerrit)

      unread,
      Jan 20, 2018, 6:54:21 PM1/20/18
      to Timothy Dresser, blink-revie...@chromium.org, blink-...@chromium.org, Philip Jägenstedt, Fadi Meawad

      Shubhie Panicker would like Timothy Dresser to review this change.

      View Change

      Move test for navigator.deviceMemory to its own dir with OWNERS

      Directory name matching https://w3c.github.io/device-memory/.

      Change-Id: If32f07b659aca91a63fd17ca80de4afb1cd525c2
      ---
      A third_party/WebKit/LayoutTests/external/wpt/device-memory/OWNERS
      R third_party/WebKit/LayoutTests/external/wpt/device-memory/device-memory.https.any.js
      2 files changed, 4 insertions(+), 0 deletions(-)

      diff --git a/third_party/WebKit/LayoutTests/external/wpt/device-memory/OWNERS b/third_party/WebKit/LayoutTests/external/wpt/device-memory/OWNERS
      new file mode 100644
      index 0000000..d7c51dc
      --- /dev/null
      +++ b/third_party/WebKit/LayoutTests/external/wpt/device-memory/OWNERS
      @@ -0,0 +1,4 @@
      +# TEAM: progressive...@chromium.org
      +# COMPONENT: Blink>PerformanceAPIs
      +fme...@chromium.org
      +pani...@chromium.org
      \ No newline at end of file
      diff --git a/third_party/WebKit/LayoutTests/external/wpt/navigator/device-memory.https.any.js b/third_party/WebKit/LayoutTests/external/wpt/device-memory/device-memory.https.any.js
      similarity index 100%
      rename from third_party/WebKit/LayoutTests/external/wpt/navigator/device-memory.https.any.js
      rename to third_party/WebKit/LayoutTests/external/wpt/device-memory/device-memory.https.any.js

      To view, visit change 876423. To unsubscribe, or for help writing mail filters, visit settings.

      Gerrit-Project: chromium/src
      Gerrit-Branch: master
      Gerrit-MessageType: newchange
      Gerrit-Change-Id: If32f07b659aca91a63fd17ca80de4afb1cd525c2
      Gerrit-Change-Number: 876423
      Gerrit-PatchSet: 1
      Gerrit-Owner: Philip Jägenstedt <foo...@chromium.org>
      Gerrit-Reviewer: Fadi Meawad <fme...@chromium.org>
      Gerrit-Reviewer: Philip Jägenstedt <foo...@chromium.org>
      Gerrit-Reviewer: Shubhie Panicker <pani...@chromium.org>
      Gerrit-Reviewer: Timothy Dresser <tdre...@chromium.org>

      Shubhie Panicker (Gerrit)

      unread,
      Jan 20, 2018, 6:54:21 PM1/20/18
      to Philip Jägenstedt, blink-revie...@chromium.org, blink-...@chromium.org, Timothy Dresser, Fadi Meawad, Blink WPT Bot, Commit Bot, chromium...@chromium.org

      Patch set 1:Code-Review +1

      View Change

      1 comment:

        • I don't think this is the right team, or maybe it is. […]

          we haven't asked them about owning device memory, but it would be a natural fit.
          Tim?

      To view, visit change 876423. To unsubscribe, or for help writing mail filters, visit settings.

      Gerrit-Project: chromium/src
      Gerrit-Branch: master
      Gerrit-MessageType: comment
      Gerrit-Change-Id: If32f07b659aca91a63fd17ca80de4afb1cd525c2
      Gerrit-Change-Number: 876423
      Gerrit-PatchSet: 1
      Gerrit-Owner: Philip Jägenstedt <foo...@chromium.org>
      Gerrit-Reviewer: Fadi Meawad <fme...@chromium.org>
      Gerrit-Reviewer: Philip Jägenstedt <foo...@chromium.org>
      Gerrit-Reviewer: Shubhie Panicker <pani...@chromium.org>
      Gerrit-Reviewer: Timothy Dresser <tdre...@chromium.org>
      Gerrit-CC: Blink WPT Bot <blink-w3c-te...@chromium.org>
      Gerrit-CC: Commit Bot <commi...@chromium.org>
      Gerrit-Comment-Date: Sat, 20 Jan 2018 23:54:20 +0000
      Gerrit-HasComments: Yes
      Gerrit-HasLabels: Yes

      Timothy Dresser (Gerrit)

      unread,
      Jan 22, 2018, 8:43:48 AM1/22/18
      to Philip Jägenstedt, blink-revie...@chromium.org, blink-...@chromium.org, Shubhie Panicker, Fadi Meawad, Blink WPT Bot, Commit Bot, chromium...@chromium.org

      View Change

      1 comment:

        • we haven't asked them about owning device memory, but it would be a natural fit. […]

          Yeah, this is reasonable.

      To view, visit change 876423. To unsubscribe, or for help writing mail filters, visit settings.

      Gerrit-Project: chromium/src
      Gerrit-Branch: master
      Gerrit-MessageType: comment
      Gerrit-Change-Id: If32f07b659aca91a63fd17ca80de4afb1cd525c2
      Gerrit-Change-Number: 876423
      Gerrit-PatchSet: 1
      Gerrit-Owner: Philip Jägenstedt <foo...@chromium.org>
      Gerrit-Reviewer: Fadi Meawad <fme...@chromium.org>
      Gerrit-Reviewer: Philip Jägenstedt <foo...@chromium.org>
      Gerrit-Reviewer: Shubhie Panicker <pani...@chromium.org>
      Gerrit-Reviewer: Timothy Dresser <tdre...@chromium.org>
      Gerrit-CC: Blink WPT Bot <blink-w3c-te...@chromium.org>
      Gerrit-CC: Commit Bot <commi...@chromium.org>
      Gerrit-Comment-Date: Mon, 22 Jan 2018 13:43:46 +0000
      Gerrit-HasComments: Yes
      Gerrit-HasLabels: No

      Philip Jägenstedt (Gerrit)

      unread,
      Jan 22, 2018, 9:05:09 AM1/22/18
      to blink-revie...@chromium.org, blink-...@chromium.org, Timothy Dresser, Shubhie Panicker, Fadi Meawad, Blink WPT Bot, Commit Bot, chromium...@chromium.org

      Patch set 1:Commit-Queue +2

      View Change

      1 comment:

        • Yeah, this is reasonable.

          Thanks, sending to CQ!

      To view, visit change 876423. To unsubscribe, or for help writing mail filters, visit settings.

      Gerrit-Project: chromium/src
      Gerrit-Branch: master
      Gerrit-MessageType: comment
      Gerrit-Change-Id: If32f07b659aca91a63fd17ca80de4afb1cd525c2
      Gerrit-Change-Number: 876423
      Gerrit-PatchSet: 1
      Gerrit-Owner: Philip Jägenstedt <foo...@chromium.org>
      Gerrit-Reviewer: Fadi Meawad <fme...@chromium.org>
      Gerrit-Reviewer: Philip Jägenstedt <foo...@chromium.org>
      Gerrit-Reviewer: Shubhie Panicker <pani...@chromium.org>
      Gerrit-Reviewer: Timothy Dresser <tdre...@chromium.org>
      Gerrit-CC: Blink WPT Bot <blink-w3c-te...@chromium.org>
      Gerrit-CC: Commit Bot <commi...@chromium.org>
      Gerrit-Comment-Date: Mon, 22 Jan 2018 14:05:05 +0000
      Gerrit-HasComments: Yes
      Gerrit-HasLabels: Yes

      Commit Bot (Gerrit)

      unread,
      Jan 22, 2018, 9:49:50 AM1/22/18
      to Philip Jägenstedt, blink-revie...@chromium.org, blink-...@chromium.org, Timothy Dresser, Shubhie Panicker, Fadi Meawad, Blink WPT Bot, chromium...@chromium.org

      Commit Bot merged this change.

      View Change

      Approvals: Fadi Meawad: Looks good to me Shubhie Panicker: Looks good to me Philip Jägenstedt: Commit
      Move test for navigator.deviceMemory to its own dir with OWNERS

      Directory name matching https://w3c.github.io/device-memory/.

      Change-Id: If32f07b659aca91a63fd17ca80de4afb1cd525c2
      Reviewed-on: https://chromium-review.googlesource.com/876423
      Reviewed-by: Fadi Meawad <fme...@chromium.org>
      Reviewed-by: Shubhie Panicker <pani...@chromium.org>
      Commit-Queue: Philip Jägenstedt <foo...@chromium.org>
      Cr-Commit-Position: refs/heads/master@{#530859}

      ---
      A third_party/WebKit/LayoutTests/external/wpt/device-memory/OWNERS
      R third_party/WebKit/LayoutTests/external/wpt/device-memory/device-memory.https.any.js
      2 files changed, 4 insertions(+), 0 deletions(-)

      diff --git a/third_party/WebKit/LayoutTests/external/wpt/device-memory/OWNERS b/third_party/WebKit/LayoutTests/external/wpt/device-memory/OWNERS
      new file mode 100644
      index 0000000..d7c51dc
      --- /dev/null
      +++ b/third_party/WebKit/LayoutTests/external/wpt/device-memory/OWNERS
      @@ -0,0 +1,4 @@
      +# TEAM: progressive...@chromium.org
      +# COMPONENT: Blink>PerformanceAPIs
      +fme...@chromium.org
      +pani...@chromium.org
      \ No newline at end of file
      diff --git a/third_party/WebKit/LayoutTests/external/wpt/navigator/device-memory.https.any.js b/third_party/WebKit/LayoutTests/external/wpt/device-memory/device-memory.https.any.js
      similarity index 100%
      rename from third_party/WebKit/LayoutTests/external/wpt/navigator/device-memory.https.any.js
      rename to third_party/WebKit/LayoutTests/external/wpt/device-memory/device-memory.https.any.js

      To view, visit change 876423. To unsubscribe, or for help writing mail filters, visit settings.

      Gerrit-Project: chromium/src
      Gerrit-Branch: master
      Gerrit-MessageType: merged
      Gerrit-Change-Id: If32f07b659aca91a63fd17ca80de4afb1cd525c2
      Gerrit-Change-Number: 876423
      Gerrit-PatchSet: 2
      Gerrit-Owner: Philip Jägenstedt <foo...@chromium.org>
      Gerrit-Reviewer: Commit Bot <commi...@chromium.org>
      Gerrit-Reviewer: Fadi Meawad <fme...@chromium.org>
      Gerrit-Reviewer: Philip Jägenstedt <foo...@chromium.org>
      Gerrit-Reviewer: Shubhie Panicker <pani...@chromium.org>
      Gerrit-Reviewer: Timothy Dresser <tdre...@chromium.org>
      Gerrit-CC: Blink WPT Bot <blink-w3c-te...@chromium.org>

      Blink WPT Bot (Gerrit)

      unread,
      Jan 22, 2018, 10:24:15 AM1/22/18
      to Philip Jägenstedt, Commit Bot, blink-revie...@chromium.org, blink-...@chromium.org, Timothy Dresser, Shubhie Panicker, Fadi Meawad, chromium...@chromium.org

      The WPT PR for this CL has been merged upstream! https://github.com/w3c/web-platform-tests/pull/9107

      View Change

        To view, visit change 876423. To unsubscribe, or for help writing mail filters, visit settings.

        Gerrit-Project: chromium/src
        Gerrit-Branch: master
        Gerrit-MessageType: comment
        Gerrit-Change-Id: If32f07b659aca91a63fd17ca80de4afb1cd525c2
        Gerrit-Change-Number: 876423
        Gerrit-PatchSet: 2
        Gerrit-Owner: Philip Jägenstedt <foo...@chromium.org>
        Gerrit-Reviewer: Commit Bot <commi...@chromium.org>
        Gerrit-Reviewer: Fadi Meawad <fme...@chromium.org>
        Gerrit-Reviewer: Philip Jägenstedt <foo...@chromium.org>
        Gerrit-Reviewer: Shubhie Panicker <pani...@chromium.org>
        Gerrit-Reviewer: Timothy Dresser <tdre...@chromium.org>
        Gerrit-CC: Blink WPT Bot <blink-w3c-te...@chromium.org>
        Gerrit-Comment-Date: Mon, 22 Jan 2018 15:24:13 +0000
        Gerrit-HasComments: No
        Gerrit-HasLabels: No
        Reply all
        Reply to author
        Forward
        0 new messages