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

Issue 9896002: Fix some grammar in the ppapi_crash.html test (Closed)

Created:
8 years, 9 months ago by gavinp
Modified:
8 years, 8 months ago
CC:
chromium-reviews, dhollowa+watch_chromium.org, sadrul, ben+watch_chromium.org
Visibility:
Public.

Description

Fix some grammar in the ppapi_crash.html test BUG=None Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=130359

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M ppapi/native_client/tests/ppapi_browser/crash/ppapi_crash.html View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 2 (0 generated)
gavinp
wdyt?
8 years, 9 months ago (2012-03-28 23:35:44 UTC) #1
dmichael (off chromium)
8 years, 9 months ago (2012-03-29 00:01:24 UTC) #2
Lgtm, thanks
On Mar 28, 2012 5:35 PM, <gavinp@chromium.org> wrote:

> Reviewers: dmichael,
>
> Message:
> wdyt?
>
> Description:
> Fix some grammar in the ppapi_crash.html test
>
>
> BUG=None
>
>
> Please review this at
https://chromiumcodereview.**appspot.com/9896002/<https://chromiumcodereview....
>
> SVN Base:
svn://svn.chromium.org/chrome/**trunk/src<http://svn.chromium.org/chrome/trunk/src>
>
> Affected files:
>  M ppapi/native_client/tests/**ppapi_browser/crash/ppapi_**crash.html
>
>
> Index: ppapi/native_client/tests/**ppapi_browser/crash/ppapi_**crash.html
> diff --git
a/ppapi/native_client/tests/**ppapi_browser/crash/ppapi_**crash.html
> b/ppapi/native_client/tests/**ppapi_browser/crash/ppapi_**crash.html
> index c7df1607f8beab40a1524b4b793159**776d59cb7a..**
> 1c4eadca9c23e56296b080a9308773**00b429e7fc 100644
> --- a/ppapi/native_client/tests/**ppapi_browser/crash/ppapi_**crash.html
> +++ b/ppapi/native_client/tests/**ppapi_browser/crash/ppapi_**crash.html
> @@ -44,7 +44,7 @@
>                            function(e) { test.fail(); });
>           plugin.postMessage(testName);
>           // In case the nexe does not crash right away, we will ping it
> -          // until we detect that it's death. DidChangeView and other
> events
> +          // until we detect that it's dead. DidChangeView and other
> events
>           // can do this too, but are less reliable.
>           if (pingToDetectCrash) {
>             function PingBack(message) {
>
>
>

Powered by Google App Engine
This is Rietveld 408576698