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

Issue 831263005: NaCl doc: update release notes for M41 (Closed)

Created:
5 years, 11 months ago by JF
Modified:
5 years, 11 months ago
CC:
chromium-reviews, binji+watch_chromium.org, native-client-reviews_googlegroups.com, Sam Clegg
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

NaCl doc: update release notes for M41 BUG= none R= jmedley@chromium.org, bradnelson@chromium.org, dmichael@chromium.org NOTRY=true Committed: https://crrev.com/09aa0d8aca2b05ca13238fa2a952cb5fcb48783f Cr-Commit-Position: refs/heads/master@{#311565}

Patch Set 1 #

Total comments: 3

Patch Set 2 : More notes. #

Total comments: 2

Patch Set 3 : Fix typo. #

Total comments: 2

Patch Set 4 : Add link for postMessageAndAwaitResponse. #

Patch Set 5 : Update release notes post-branch. #

Patch Set 6 : Move VideoDecoder to 40. #

Patch Set 7 : Add generated HTML. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+109 lines, -32 lines) Patch
M native_client_sdk/doc_generated/sdk/release-notes.html View 1 2 3 4 5 6 19 chunks +58 lines, -26 lines 0 comments Download
M native_client_sdk/doc_generated/sitemap.html View 1 2 3 4 5 6 1 chunk +3 lines, -0 lines 0 comments Download
M native_client_sdk/src/doc/sdk/release-notes.rst View 1 2 3 4 5 1 chunk +48 lines, -6 lines 0 comments Download

Messages

Total messages: 19 (3 generated)
JF
5 years, 11 months ago (2015-01-05 21:47:01 UTC) #1
bradn
lgtm, but maybe hold off until Bill can comment on whether video decoder is really ...
5 years, 11 months ago (2015-01-05 21:49:58 UTC) #3
jpmedley
lgtm
5 years, 11 months ago (2015-01-05 21:51:28 UTC) #4
shyamsundarr
https://codereview.chromium.org/831263005/diff/1/native_client_sdk/src/doc/sdk/release-notes.rst File native_client_sdk/src/doc/sdk/release-notes.rst (right): https://codereview.chromium.org/831263005/diff/1/native_client_sdk/src/doc/sdk/release-notes.rst#newcode10 native_client_sdk/src/doc/sdk/release-notes.rst:10: * The x86 NaCl validators now accept instructions from ...
5 years, 11 months ago (2015-01-05 21:58:05 UTC) #6
JF
https://codereview.chromium.org/831263005/diff/1/native_client_sdk/src/doc/sdk/release-notes.rst File native_client_sdk/src/doc/sdk/release-notes.rst (right): https://codereview.chromium.org/831263005/diff/1/native_client_sdk/src/doc/sdk/release-notes.rst#newcode10 native_client_sdk/src/doc/sdk/release-notes.rst:10: * The x86 NaCl validators now accept instructions from ...
5 years, 11 months ago (2015-01-05 21:59:39 UTC) #7
jpmedley
PTAL https://codereview.chromium.org/831263005/diff/20001/native_client_sdk/src/doc/sdk/release-notes.rst File native_client_sdk/src/doc/sdk/release-notes.rst (right): https://codereview.chromium.org/831263005/diff/20001/native_client_sdk/src/doc/sdk/release-notes.rst#newcode22 native_client_sdk/src/doc/sdk/release-notes.rst:22: * PNaCl now break up many integer operations ...
5 years, 11 months ago (2015-01-06 16:37:19 UTC) #8
JF
https://codereview.chromium.org/831263005/diff/20001/native_client_sdk/src/doc/sdk/release-notes.rst File native_client_sdk/src/doc/sdk/release-notes.rst (right): https://codereview.chromium.org/831263005/diff/20001/native_client_sdk/src/doc/sdk/release-notes.rst#newcode22 native_client_sdk/src/doc/sdk/release-notes.rst:22: * PNaCl now break up many integer operations over ...
5 years, 11 months ago (2015-01-06 17:16:53 UTC) #9
jpmedley
lgtm
5 years, 11 months ago (2015-01-06 17:45:27 UTC) #10
dmichael (off chromium)
lgtm https://codereview.chromium.org/831263005/diff/40001/native_client_sdk/src/doc/sdk/release-notes.rst File native_client_sdk/src/doc/sdk/release-notes.rst (right): https://codereview.chromium.org/831263005/diff/40001/native_client_sdk/src/doc/sdk/release-notes.rst#newcode40 native_client_sdk/src/doc/sdk/release-notes.rst:40: communicate synchronously with PNaCl embeds. The documentation pages ...
5 years, 11 months ago (2015-01-07 23:29:02 UTC) #11
JF
https://codereview.chromium.org/831263005/diff/40001/native_client_sdk/src/doc/sdk/release-notes.rst File native_client_sdk/src/doc/sdk/release-notes.rst (right): https://codereview.chromium.org/831263005/diff/40001/native_client_sdk/src/doc/sdk/release-notes.rst#newcode40 native_client_sdk/src/doc/sdk/release-notes.rst:40: communicate synchronously with PNaCl embeds. On 2015/01/07 23:29:01, dmichael ...
5 years, 11 months ago (2015-01-08 23:20:08 UTC) #12
JF
I just updated the release notes. The last DEPS roll missed the branch by 22 ...
5 years, 11 months ago (2015-01-14 19:33:03 UTC) #13
bbudge
LGTM As it turns out, the VideoDecoder wasn't really available for M39 because I left ...
5 years, 11 months ago (2015-01-14 19:47:56 UTC) #14
JF
On 2015/01/14 19:47:56, bbudge wrote: > LGTM > > As it turns out, the VideoDecoder ...
5 years, 11 months ago (2015-01-14 20:47:18 UTC) #15
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/831263005/120001
5 years, 11 months ago (2015-01-14 22:49:08 UTC) #17
commit-bot: I haz the power
Committed patchset #7 (id:120001)
5 years, 11 months ago (2015-01-14 22:50:25 UTC) #18
commit-bot: I haz the power
5 years, 11 months ago (2015-01-14 22:51:45 UTC) #19
Message was sent while issue was closed.
Patchset 7 (id:??) landed as
https://crrev.com/09aa0d8aca2b05ca13238fa2a952cb5fcb48783f
Cr-Commit-Position: refs/heads/master@{#311565}

Powered by Google App Engine
This is Rietveld 408576698