Changed all @code to <code> and @endcode to </code> as per dmichael

Changed all @code to <code> and @endcode to </code> as per dmichael. Adding Polina to the review to look at url_loader.h and give approval of text that points to example.
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=98856

[dmichael (off chromium), 2011-08-23 16:53:52]

http://codereview.chromium.org/7715005/diff/1/ppapi/c/ppb_audio_config.h
File ppapi/c/ppb_audio_config.h (right):

http://codereview.chromium.org/7715005/diff/1/ppapi/c/ppb_audio_config.h#newc...
ppapi/c/ppb_audio_config.h:163: * for more on sample frame counts.
Did you intend to do this change? I thought you just made it the other way?

http://codereview.chromium.org/7715005/diff/1/ppapi/c/ppp_messaging.h
File ppapi/c/ppp_messaging.h (right):

http://codereview.chromium.org/7715005/diff/1/ppapi/c/ppp_messaging.h#newcode59
ppapi/c/ppp_messaging.h:59: * <strong>Example:</strong>
Now there are two 'Example:' lines. Please remove one (I don't really care
which)

[jond, 2011-08-23 20:02:36]

http://codereview.chromium.org/7715005/diff/1/ppapi/c/ppb_audio_config.h
File ppapi/c/ppb_audio_config.h (right):

http://codereview.chromium.org/7715005/diff/1/ppapi/c/ppb_audio_config.h#newc...
ppapi/c/ppb_audio_config.h:163: * for more on sample frame counts.
No I didn't. Strange.

On 2011/08/23 16:53:52, dmichael wrote:
> Did you intend to do this change? I thought you just made it the other way?

http://codereview.chromium.org/7715005/diff/1/ppapi/c/ppp_messaging.h
File ppapi/c/ppp_messaging.h (right):

http://codereview.chromium.org/7715005/diff/1/ppapi/c/ppp_messaging.h#newcode59
ppapi/c/ppp_messaging.h:59: * <strong>Example:</strong>
Oops. Good catch.

On 2011/08/23 16:53:52, dmichael wrote:
> Now there are two 'Example:' lines. Please remove one (I don't really care
> which)

[dmichael (off chromium), 2011-08-23 20:16:11]

http://codereview.chromium.org/7715005/diff/6002/ppapi/api/ppb_audio.idl
File ppapi/api/ppb_audio.idl (right):

http://codereview.chromium.org/7715005/diff/6002/ppapi/api/ppb_audio.idl#newc...
ppapi/api/ppb_audio.idl:46: * audio_interface</code>...
missing first <code>

http://codereview.chromium.org/7715005/diff/6002/ppapi/api/ppb_audio.idl#newc...
ppapi/api/ppb_audio.idl:72: * @param[in] instance A <code>PP_Instance<code>
identifying one instance of a
ending tag needs to be </code>

http://codereview.chromium.org/7715005/diff/6002/ppapi/api/ppb_audio.idl#newc...
ppapi/api/ppb_audio.idl:74: * @param[in] config A <code>PP_Resource<code>
corresponding to an audio
here too

http://codereview.chromium.org/7715005/diff/6002/ppapi/api/ppb_audio.idl#newc...
ppapi/api/ppb_audio.idl:76: * @param[in] audio_callback A
<code>PPB_Audio_Callback<code> callback
here too

http://codereview.chromium.org/7715005/diff/6002/ppapi/api/ppb_audio.idl#newc...
ppapi/api/ppb_audio.idl:123: * @return A <code>PP_Bool</code> containing
<code>PP_TRUE</code> if
PP_TRUE->PP_True, and two more similar problems in the 2 lines below.

http://codereview.chromium.org/7715005/diff/6002/ppapi/c/ppb_audio_config.h
File ppapi/c/ppb_audio_config.h (right):

http://codereview.chromium.org/7715005/diff/6002/ppapi/c/ppb_audio_config.h#n...
ppapi/c/ppb_audio_config.h:1: /* Copyright (c) 2011 The Chromium Authors. All
rights reserved.
I don't see ppb_audio_config.idl in the CL. Please make the changes there and
include it in the CL.

http://codereview.chromium.org/7715005/diff/6002/ppapi/c/ppb_audio_config.h#n...
ppapi/c/ppb_audio_config.h:143: * @return A <code>PP_Bool</code> containing
<code>PP_TRUE</code> if the given
PP_TRUE->PP_True, PP_FALSE->PP_False (globally, just in case)

http://codereview.chromium.org/7715005/diff/6002/ppapi/cpp/url_loader.h
File ppapi/cpp/url_loader.h (right):

http://codereview.chromium.org/7715005/diff/6002/ppapi/cpp/url_loader.h#newco...
ppapi/cpp/url_loader.h:96: /// resource.
Maybe say 'URLLoader resource' instead of just 'resource'? It won't work if
given another kind of resource, and the current comment doesn't really seem to
say anything.

[jond, 2011-08-23 21:15:29]

http://codereview.chromium.org/7715005/diff/6002/ppapi/api/ppb_audio.idl
File ppapi/api/ppb_audio.idl (right):

http://codereview.chromium.org/7715005/diff/6002/ppapi/api/ppb_audio.idl#newc...
ppapi/api/ppb_audio.idl:46: * audio_interface</code>...
On 2011/08/23 20:16:11, dmichael wrote:
> missing first <code>

Done.

http://codereview.chromium.org/7715005/diff/6002/ppapi/api/ppb_audio.idl#newc...
ppapi/api/ppb_audio.idl:72: * @param[in] instance A <code>PP_Instance<code>
identifying one instance of a
On 2011/08/23 20:16:11, dmichael wrote:
> ending tag needs to be </code>

Done.

http://codereview.chromium.org/7715005/diff/6002/ppapi/api/ppb_audio.idl#newc...
ppapi/api/ppb_audio.idl:74: * @param[in] config A <code>PP_Resource<code>
corresponding to an audio
On 2011/08/23 20:16:11, dmichael wrote:
> here too

Done.

http://codereview.chromium.org/7715005/diff/6002/ppapi/api/ppb_audio.idl#newc...
ppapi/api/ppb_audio.idl:76: * @param[in] audio_callback A
<code>PPB_Audio_Callback<code> callback
On 2011/08/23 20:16:11, dmichael wrote:
> here too

Done.

http://codereview.chromium.org/7715005/diff/6002/ppapi/api/ppb_audio.idl#newc...
ppapi/api/ppb_audio.idl:76: * @param[in] audio_callback A
<code>PPB_Audio_Callback<code> callback
On 2011/08/23 20:16:11, dmichael wrote:
> here too

Done.

http://codereview.chromium.org/7715005/diff/6002/ppapi/api/ppb_audio.idl#newc...
ppapi/api/ppb_audio.idl:123: * @return A <code>PP_Bool</code> containing
<code>PP_TRUE</code> if
On 2011/08/23 20:16:11, dmichael wrote:
> PP_TRUE->PP_True, and two more similar problems in the 2 lines below.

Done.

http://codereview.chromium.org/7715005/diff/6002/ppapi/c/ppb_audio_config.h
File ppapi/c/ppb_audio_config.h (right):

http://codereview.chromium.org/7715005/diff/6002/ppapi/c/ppb_audio_config.h#n...
ppapi/c/ppb_audio_config.h:1: /* Copyright (c) 2011 The Chromium Authors. All
rights reserved.
Strange. Looks like I made the changes in the IDL.

On 2011/08/23 20:16:11, dmichael wrote:
> I don't see ppb_audio_config.idl in the CL. Please make the changes there and
> include it in the CL.

http://codereview.chromium.org/7715005/diff/6002/ppapi/c/ppb_audio_config.h#n...
ppapi/c/ppb_audio_config.h:143: * @return A <code>PP_Bool</code> containing
<code>PP_TRUE</code> if the given
On 2011/08/23 20:16:11, dmichael wrote:
> PP_TRUE->PP_True, PP_FALSE->PP_False (globally, just in case)

Done.

http://codereview.chromium.org/7715005/diff/6002/ppapi/cpp/url_loader.h
File ppapi/cpp/url_loader.h (right):

http://codereview.chromium.org/7715005/diff/6002/ppapi/cpp/url_loader.h#newco...
ppapi/cpp/url_loader.h:96: /// resource.
On 2011/08/23 20:16:11, dmichael wrote:
> Maybe say 'URLLoader resource' instead of just 'resource'? It won't work if
> given another kind of resource, and the current comment doesn't really seem to
> say anything.

Done.

[dmichael (off chromium), 2011-08-24 15:18:02]

I don't see your changes. Could you reupload please?

In general, I would recommend looking at your change in rietveld before hitting
'publish' to make sure you have the changes you expect. That's what I do.

[jond, 2011-08-24 15:22:05]

Should be there now...

- Jon

On Wed, Aug 24, 2011 at 9:18 AM, <dmichael@chromium.org> wrote:

> I don't see your changes. Could you reupload please?
>
> In general, I would recommend looking at your change in rietveld before
> hitting
> 'publish' to make sure you have the changes you expect. That's what I do.
>
>
http://codereview.chromium.**org/7715005/<http://codereview.chromium.org/7715...
>

[dmichael (off chromium), 2011-08-24 15:38:01]

Please put the audio config files together (probably in this CL makes the most
sense) and unbreak the PP_True/PP_False. (That was my fault, sorry again).

http://codereview.chromium.org/7715005/diff/20/ppapi/api/ppb_audio.idl
File ppapi/api/ppb_audio.idl (right):

http://codereview.chromium.org/7715005/diff/20/ppapi/api/ppb_audio.idl#newcode96
ppapi/api/ppb_audio.idl:96: * @return A <code>PP_Bool</code> containing
containing <code>PP_True</code>
Sorry about the temporary insanity. PP_True->PP_TRUE, PP_False->PP_FALSE. My
bad.

[jond, 2011-08-29 19:15:32]

http://codereview.chromium.org/7715005/diff/20/ppapi/api/ppb_audio.idl
File ppapi/api/ppb_audio.idl (right):

http://codereview.chromium.org/7715005/diff/20/ppapi/api/ppb_audio.idl#newcode96
ppapi/api/ppb_audio.idl:96: * @return A <code>PP_Bool</code> containing
containing <code>PP_True</code>
On 2011/08/24 15:38:01, dmichael wrote:
> Sorry about the temporary insanity. PP_True->PP_TRUE, PP_False->PP_FALSE. My
> bad.

Done.

[dmichael (off chromium), 2011-08-29 19:29:32]

LGTM