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 5560005: Update PDF user actions. (Closed)

Created:
10 years ago by jam
Modified:
9 years, 7 months ago
Reviewers:
gene
CC:
chromium-reviews, pam+watch_chromium.org
Visibility:
Public.

Description

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -0 lines) Patch
M chrome/tools/chromeactions.txt View 2 chunks +3 lines, -0 lines 1 comment Download
M chrome/tools/extract_actions.py View 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
jam
10 years ago (2010-12-04 00:46:15 UTC) #1
gene
LGTM A small question below. http://codereview.chromium.org/5560005/diff/1/chrome/tools/chromeactions.txt File chrome/tools/chromeactions.txt (right): http://codereview.chromium.org/5560005/diff/1/chrome/tools/chromeactions.txt#newcode16 chrome/tools/chromeactions.txt:16: 0x67b7031bb00a6917 AboutFlags_instant-type is this ...
10 years ago (2010-12-04 00:49:22 UTC) #2
jam
10 years ago (2010-12-04 01:07:34 UTC) #3
On Fri, Dec 3, 2010 at 4:49 PM, <gene@chromium.org> wrote:

> LGTM
> A small question below.
>
>
>
> http://codereview.chromium.org/5560005/diff/1/chrome/tools/chromeactions.txt
> File chrome/tools/chromeactions.txt (right):
>
>
>
http://codereview.chromium.org/5560005/diff/1/chrome/tools/chromeactions.txt#...
> chrome/tools/chromeactions.txt:16:
> 0x67b7031bb00a6917      AboutFlags_instant-type
> is this needed for PDF actions?


Someone added it without running the script to update the dashboard (which
is what builds the txt file).

>
>
> http://codereview.chromium.org/5560005/
>

Powered by Google App Engine
This is Rietveld 408576698