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

Issue 344183006: Remove references to pepperversion and peppernum for NaCl docs. (Closed)

Created:
6 years, 6 months ago by binji
Modified:
6 years, 6 months ago
CC:
chromium-reviews, binji+watch_chromium.org, chromium-apps-reviews_chromium.org, Sam Clegg, extensions-reviews_chromium.org
Project:
chromium
Visibility:
Public.

Description

Remove references to pepperversion and peppernum for NaCl docs. BUG=none R=kalman@chromium.org, sbc@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=278862

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -7 lines) Patch
M chrome/common/extensions/docs/server2/template_renderer.py View 1 chunk +0 lines, -4 lines 1 comment Download
M native_client_sdk/doc_generated/community/application-gallery.html View 1 chunk +0 lines, -3 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
binji
6 years, 6 months ago (2014-06-20 23:22:32 UTC) #1
binji
https://codereview.chromium.org/344183006/diff/1/chrome/common/extensions/docs/server2/template_renderer.py File chrome/common/extensions/docs/server2/template_renderer.py (left): https://codereview.chromium.org/344183006/diff/1/chrome/common/extensions/docs/server2/template_renderer.py#oldcode42 chrome/common/extensions/docs/server2/template_renderer.py:42: 'pepperversion': '', This used to be in the pepper ...
6 years, 6 months ago (2014-06-20 23:24:07 UTC) #2
not at google - send to devlin
p.s. if the presubmit warning tells you to bump app/cron.yaml you can ignore it.
6 years, 6 months ago (2014-06-20 23:24:19 UTC) #3
not at google - send to devlin
lgtm
6 years, 6 months ago (2014-06-20 23:24:22 UTC) #4
Sam Clegg
lgtm
6 years, 6 months ago (2014-06-20 23:26:29 UTC) #5
binji
6 years, 6 months ago (2014-06-20 23:37:15 UTC) #6
Message was sent while issue was closed.
Committed patchset #1 manually as r278862 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698