Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(63)

Issue 2181233004: [Origin Trials] Add tests for static methods on partial interfaces (Closed)

Created:
4 years, 4 months ago by iclelland
Modified:
4 years, 4 months ago
Reviewers:
haraken, chasej
CC:
chromium-reviews, blink-reviews, blink-reviews-bindings_chromium.org, chasej+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

[Origin Trials] Add tests for static methods on partial interfaces This adds bindings an layout tests for static methods defined on partial interfaces under an origin trial, which were missing from the test suite. BUG=614352, 621641 R=chasej@chromium.org Committed: https://crrev.com/6e7a752bb54185b0ebd44c29682964570184ac50 Cr-Commit-Position: refs/heads/master@{#407903}

Patch Set 1 #

Messages

Total messages: 8 (2 generated)
iclelland
+r chasej
4 years, 4 months ago (2016-07-26 17:16:36 UTC) #1
haraken
owner LGTM
4 years, 4 months ago (2016-07-26 17:34:46 UTC) #2
chasej
lgtm
4 years, 4 months ago (2016-07-26 19:01:04 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2181233004/1
4 years, 4 months ago (2016-07-26 19:16:40 UTC) #5
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 4 months ago (2016-07-26 20:34:11 UTC) #6
commit-bot: I haz the power
4 years, 4 months ago (2016-07-26 20:36:31 UTC) #8
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/6e7a752bb54185b0ebd44c29682964570184ac50
Cr-Commit-Position: refs/heads/master@{#407903}

Powered by Google App Engine
This is Rietveld 408576698