Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add "0xCA <=> U+05BA" mapping to windows-1255 (spec change) #4090

Merged
merged 1 commit into from Nov 7, 2016

Conversation

jungshik
Copy link
Contributor

@jungshik jungshik commented Oct 27, 2016

In whatwg/encoding#73, the encoding spec was changed to add "0xCA<=>U+05BA" mapping to
windows-1255.

@annevk , can you review? Thanks

@wpt-pr-bot
Copy link
Collaborator

Reviewers for this pull request are: @inexorabletash and @sideshowbarker.

@Ms2ger
Copy link
Contributor

Ms2ger commented Oct 27, 2016

Also submitted independently in https://bugzilla.mozilla.org/show_bug.cgi?id=1312384

@annevk
Copy link
Member

annevk commented Oct 27, 2016

@jgraham which should we land with this kind of conflict?

@jungshik
Copy link
Contributor Author

I'm confused. wpt is getting an automatic push from hg.mozilla.org ?
Anyway, https://hg.mozilla.org/integration/mozilla-inbound/rev/6963f5ab0000 seems identical to my pull request.

@inexorabletash
Copy link
Contributor

wpt is getting an automatic push from hg.mozilla.org ?

Yes - isn't that awesome? (Hopefully coming soon to chromium/blink, too.)

@jungshik
Copy link
Contributor Author

wpt is getting an automatic push from hg.mozilla.org ?
Yes - isn't that awesome? (Hopefully coming soon to chromium/blink, too.)

I see. Thank you. In that case, feel free to close this one if the automatic push from hg.mozilla.org is gonna be done soon (say, in a couple of days).

@jgraham
Copy link
Contributor

jgraham commented Oct 31, 2016

I can start a new one today.

@jgraham jgraham closed this Oct 31, 2016
@annevk
Copy link
Member

annevk commented Oct 31, 2016

Sorry about the conflict @jungshik, hope we don't get too many of these. 😊

@jungshik
Copy link
Contributor Author

jungshik commented Nov 1, 2016

No problem, @annevk

BTW, was the change (mozilla's : identical to mine) landed? I don't see it in the tree.

@annevk
Copy link
Member

annevk commented Nov 2, 2016

@jgraham can best answer that. Syncing is manual still.

@jungshik
Copy link
Contributor Author

jungshik commented Nov 5, 2016

Gee. I thought it's automatic and pretty frequent (if not daily). Given that it's neither, just merging this world have been quicker.

@annevk annevk reopened this Nov 7, 2016
@annevk annevk merged commit 2a4e5cf into web-platform-tests:master Nov 7, 2016
@annevk
Copy link
Member

annevk commented Nov 7, 2016

Agreed, I went ahead and did so, given that I can't seem to chat with @jgraham.

timniederhausen pushed a commit to timniederhausen/gn that referenced this pull request Nov 26, 2016
See https://codereview.chromium.org/2442923002 for actual changes.

Fix base_unittests:NumberFormattingTest.FormatPercent per new percent formats
for Arabic and Persian in CLDR 30/ICU 58.

Adjust IsIDNComponentSafe() to keep the current behavior (therefore, no
changes in tests) with ICU 58.1 changes in uspoof API.

Rebaselined a few layout tests (no substantial changes) and changed one test.

  fast/text/midword-break-before-surrogate-pair: expectation was wrong about the
    line breaking behavior of regional indiactors.

The following two wpt tests are directly changed in our tree with an upstream
pull request made for the change:
  web-platform-tests/wpt#4090

  imported/wpt/encoding/textdecoder-fatal-single-byte
  imported/wpt/encoding/textdecoder-labels

BUG=637001
TEST=Layout tests, all unittests, browser tests

Review-Url: https://codereview.chromium.org/2447513002
Cr-Original-Commit-Position: refs/heads/master@{#429312}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: b2b928ac0d2e7caabffd9a10b94e71ef534b206a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants