[net] docs: Update info in bug-triage-labels.md about Blink networking APIs

[net] docs: Update info in bug-triage-labels.md about Blink networking APIs

BUG=none
R=asanka,mmenke

Committed: https://crrev.com/c1ece4bd616db69b8aa2d025b50ff9ccac2196a7
Cr-Commit-Position: refs/heads/master@{#362381}

[tyoshino (SeeGerritForStatus), 2015-11-30 08:29:55]

Description was changed from

==========
[net/docs] Update info in bug-triage-labels.md about Blink networking APIs

BUG=none
==========

to

==========
[net] docs: Update info in bug-triage-labels.md about Blink networking APIs

BUG=none
==========

[tyoshino (SeeGerritForStatus), 2015-11-30 08:29:55]

tyoshino@chromium.org changed reviewers:
+ asanka@chromium.org

[tyoshino (SeeGerritForStatus), 2015-11-30 08:30:28]

tyoshino@chromium.org changed reviewers:
+ yhirano@chromium.org

[tyoshino (SeeGerritForStatus), 2015-11-30 08:30:34]

+yhirano

[tyoshino (SeeGerritForStatus), 2015-11-30 08:41:21]

Hmm, I tried to review the rendered result, but it seems the Gitiles MD syntax
is different from one of GitHub. asanka@, sorry but please tell me the right way
to verify.

I tried md_browser, too. But it also doesn't give the same result as Gitiles,
especially about parsing the lines starting with :, i.e. ": Blah Blah..."
https://chromium.googlesource.com/chromium/src/+/master/tools/md_browser/

[asanka, 2015-11-30 17:01:20]

asanka@chromium.org changed reviewers:
+ mmenke@chromium.org

[asanka, 2015-11-30 17:04:38]

On 2015/11/30 at 08:41:21, tyoshino wrote:
> Hmm, I tried to review the rendered result, but it seems the Gitiles MD syntax
is different from one of GitHub. asanka@, sorry but please tell me the right way
to verify.
> 
> I tried md_browser, too. But it also doesn't give the same result as Gitiles,
especially about parsing the lines starting with :, i.e. ": Blah Blah..."
> https://chromium.googlesource.com/chromium/src/+/master/tools/md_browser/

I'll pass the baton to mmenke@ since he's largely been overseeing the triage
process and should approve the change to the labeling.

WRT Markdown syntax: Gitiles uses a Markdown processor called Pegdown. Both
Pegdown and Gitiles specifically support the definition list extension that we
are using here. To make them work with md_browser.py, you'll need to add the
'markdown.extensions.def_list' to the list of extensions to enable in Python
Markdown. You can do so with something like this:

diff --git a/tools/md_browser/md_browser.py b/tools/md_browser/md_browser.py
index 22864c2..3f0bac8 100644
--- a/tools/md_browser/md_browser.py
+++ b/tools/md_browser/md_browser.py
@@ -103,6 +103,7 @@ class Handler(SimpleHTTPServer.SimpleHTTPRequestHandler):
     extensions = [
         'markdown.extensions.fenced_code',
         'markdown.extensions.tables',
+        'markdown.extensions.def_list',
         'markdown.extensions.toc',
         'gitiles_ext_blocks',
     ]

I'll fire up a CL to get definition lists working in md_browser.

[mmenke, 2015-11-30 17:24:25]

On 2015/11/30 17:04:38, asanka wrote:
> On 2015/11/30 at 08:41:21, tyoshino wrote:
> > Hmm, I tried to review the rendered result, but it seems the Gitiles MD
syntax
> is different from one of GitHub. asanka@, sorry but please tell me the right
way
> to verify.
> > 
> > I tried md_browser, too. But it also doesn't give the same result as
Gitiles,
> especially about parsing the lines starting with :, i.e. ": Blah Blah..."
> > https://chromium.googlesource.com/chromium/src/+/master/tools/md_browser/
> 
> I'll pass the baton to mmenke@ since he's largely been overseeing the triage
> process and should approve the change to the labeling.
> 
> WRT Markdown syntax: Gitiles uses a Markdown processor called Pegdown. Both
> Pegdown and Gitiles specifically support the definition list extension that we
> are using here. To make them work with md_browser.py, you'll need to add the
> 'markdown.extensions.def_list' to the list of extensions to enable in Python
> Markdown. You can do so with something like this:
> 
> diff --git a/tools/md_browser/md_browser.py b/tools/md_browser/md_browser.py
> index 22864c2..3f0bac8 100644
> --- a/tools/md_browser/md_browser.py
> +++ b/tools/md_browser/md_browser.py
> @@ -103,6 +103,7 @@ class Handler(SimpleHTTPServer.SimpleHTTPRequestHandler):
>      extensions = [
>          'markdown.extensions.fenced_code',
>          'markdown.extensions.tables',
> +        'markdown.extensions.def_list',
>          'markdown.extensions.toc',
>          'gitiles_ext_blocks',
>      ]
> 
> I'll fire up a CL to get definition lists working in md_browser.

To test the changes, use ninja on net_docs.

[mmenke, 2015-11-30 17:25:32]

These changes LGTM

[tyoshino (SeeGerritForStatus), 2015-12-01 07:58:27]

On 2015/11/30 17:24:25, mmenke wrote:
> On 2015/11/30 17:04:38, asanka wrote:
> > On 2015/11/30 at 08:41:21, tyoshino wrote:
> > > Hmm, I tried to review the rendered result, but it seems the Gitiles MD
> syntax
> > is different from one of GitHub. asanka@, sorry but please tell me the right
> way
> > to verify.
> > > 
> > > I tried md_browser, too. But it also doesn't give the same result as
> Gitiles,
> > especially about parsing the lines starting with :, i.e. ": Blah Blah..."
> > > https://chromium.googlesource.com/chromium/src/+/master/tools/md_browser/
> > 
> > I'll pass the baton to mmenke@ since he's largely been overseeing the triage
> > process and should approve the change to the labeling.
> > 
> > WRT Markdown syntax: Gitiles uses a Markdown processor called Pegdown. Both
> > Pegdown and Gitiles specifically support the definition list extension that
we
> > are using here. To make them work with md_browser.py, you'll need to add the
> > 'markdown.extensions.def_list' to the list of extensions to enable in Python
> > Markdown. You can do so with something like this:
> > 
> > diff --git a/tools/md_browser/md_browser.py b/tools/md_browser/md_browser.py
> > index 22864c2..3f0bac8 100644
> > --- a/tools/md_browser/md_browser.py
> > +++ b/tools/md_browser/md_browser.py
> > @@ -103,6 +103,7 @@ class
Handler(SimpleHTTPServer.SimpleHTTPRequestHandler):
> >      extensions = [
> >          'markdown.extensions.fenced_code',
> >          'markdown.extensions.tables',
> > +        'markdown.extensions.def_list',
> >          'markdown.extensions.toc',
> >          'gitiles_ext_blocks',
> >      ]
> > 
> > I'll fire up a CL to get definition lists working in md_browser.
> 

Thanks for the change! https://codereview.chromium.org/1488693002/ I can now see
the def lists rendered correctly.

> To test the changes, use ninja on net_docs.

Thanks for the info! I'm creating a CL for net_docs.py to use def_list as well.

[tyoshino (SeeGerritForStatus), 2015-12-01 07:58:45]

Description was changed from

==========
[net] docs: Update info in bug-triage-labels.md about Blink networking APIs

BUG=none
==========

to

==========
[net] docs: Update info in bug-triage-labels.md about Blink networking APIs

BUG=none
R=asanka,mmenke
==========

[tyoshino (SeeGerritForStatus), 2015-12-01 07:58:55]

The CQ bit was checked by tyoshino@chromium.org

[commit-bot: I haz the power, 2015-12-01 07:59:32]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1480223003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1480223003/1

[commit-bot: I haz the power, 2015-12-01 10:52:58]

Description was changed from

==========
[net] docs: Update info in bug-triage-labels.md about Blink networking APIs

BUG=none
R=asanka,mmenke
==========

to

==========
[net] docs: Update info in bug-triage-labels.md about Blink networking APIs

BUG=none
R=asanka,mmenke
==========

[commit-bot: I haz the power, 2015-12-01 10:53:00]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-01 10:54:18]

Description was changed from

==========
[net] docs: Update info in bug-triage-labels.md about Blink networking APIs

BUG=none
R=asanka,mmenke
==========

to

==========
[net] docs: Update info in bug-triage-labels.md about Blink networking APIs

BUG=none
R=asanka,mmenke

Committed: https://crrev.com/c1ece4bd616db69b8aa2d025b50ff9ccac2196a7
Cr-Commit-Position: refs/heads/master@{#362381}
==========

[commit-bot: I haz the power, 2015-12-01 10:54:19]

Patchset 1 (id:??) landed as
https://crrev.com/c1ece4bd616db69b8aa2d025b50ff9ccac2196a7
Cr-Commit-Position: refs/heads/master@{#362381}