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

Issue 11823042: Add args to DidCreate (Closed)

Created:
7 years, 11 months ago by noelallen1
Modified:
7 years, 11 months ago
Reviewers:
binji
CC:
chromium-reviews
Visibility:
Public.

Description

Add args to DidCreate Adds a optional dictionary passed to createNaClModule as a set of key value pairs to the DidCreate function. R=sbc@chromium.org,binji@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=176161

Patch Set 1 #

Total comments: 1

Patch Set 2 : Fix comment nit #

Unified diffs Side-by-side diffs Delta from patch set Stats (+10 lines, -1 line) Patch
M native_client_sdk/src/examples/common.js View 1 1 chunk +10 lines, -1 line 0 comments Download

Messages

Total messages: 2 (0 generated)
noelallen1
ptal
7 years, 11 months ago (2013-01-10 17:18:50 UTC) #1
binji
7 years, 11 months ago (2013-01-10 19:45:56 UTC) #2
lgtm

https://codereview.chromium.org/11823042/diff/1/native_client_sdk/src/example...
File native_client_sdk/src/examples/common.js (right):

https://codereview.chromium.org/11823042/diff/1/native_client_sdk/src/example...
native_client_sdk/src/examples/common.js:22: * @param {args} optional dictionary
of args to send to DidCreateInstance
nit: @param {Object} args ...

Powered by Google App Engine
This is Rietveld 408576698