Skip to content

Region lines cannot be negative now that they are unsigned long.#423

Merged
silviapfeiffer merged 3 commits intow3c:gh-pagesfrom
silviapfeiffer:regionLinesError
Jan 31, 2018
Merged

Region lines cannot be negative now that they are unsigned long.#423
silviapfeiffer merged 3 commits intow3c:gh-pagesfrom
silviapfeiffer:regionLinesError

Conversation

@silviapfeiffer
Copy link
Member

Closes #414

@k2nagaraju
Copy link

please merge this change.

@silviapfeiffer
Copy link
Member Author

@dwsinger I need a review

Copy link

@dwsinger dwsinger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@silviapfeiffer silviapfeiffer merged commit fbbe1d5 into w3c:gh-pages Jan 31, 2018
@silviapfeiffer silviapfeiffer deleted the regionLinesError branch January 31, 2018 18:12
chromium-wpt-export-bot pushed a commit to web-platform-tests/wpt that referenced this pull request Feb 1, 2018
updated idl file and VTTScanner
updated test cases and expected results

https://w3c.github.io/webvtt/#dom-vttregion-lines
w3c/webvtt#423

Bug: 755487
Change-Id: Ifd1ef26ede8cbc4f6bfe817aae9bce1191c18d24
chromium-wpt-export-bot pushed a commit to web-platform-tests/wpt that referenced this pull request Feb 1, 2018
updated idl file and VTTScanner
updated test cases and expected results

https://w3c.github.io/webvtt/#dom-vttregion-lines
w3c/webvtt#423

Bug: 755487
Change-Id: Ifd1ef26ede8cbc4f6bfe817aae9bce1191c18d24
Reviewed-on: https://chromium-review.googlesource.com/826666
Commit-Queue: srirama chandra sekhar <srirama.m@samsung.com>
Reviewed-by: srirama chandra sekhar <srirama.m@samsung.com>
Reviewed-by: Fredrik Söderquist <fs@opera.com>
Cr-Commit-Position: refs/heads/master@{#533586}
chromium-wpt-export-bot pushed a commit to web-platform-tests/wpt that referenced this pull request Feb 1, 2018
updated idl file and VTTScanner
updated test cases and expected results

https://w3c.github.io/webvtt/#dom-vttregion-lines
w3c/webvtt#423

Bug: 755487
Change-Id: Ifd1ef26ede8cbc4f6bfe817aae9bce1191c18d24
Reviewed-on: https://chromium-review.googlesource.com/826666
Commit-Queue: srirama chandra sekhar <srirama.m@samsung.com>
Reviewed-by: srirama chandra sekhar <srirama.m@samsung.com>
Reviewed-by: Fredrik Söderquist <fs@opera.com>
Cr-Commit-Position: refs/heads/master@{#533586}
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants