Updating extension hosting docs

Updating extension hosting docs

NOTRY=true

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

[mkearney1, 2015-05-13 18:17:59]

mkearney@chromium.org changed reviewers:
+ kalman@chromium.org

[mkearney1, 2015-05-13 18:18:49]

Ben, this one is ready for review.

I've tested the docs in the local server build, so we should be good.

[not at google - send to devlin, 2015-05-13 18:23:27]

lgtm

https://codereview.chromium.org/1137753005/diff/40001/chrome/common/extension...
File chrome/common/extensions/docs/templates/articles/external_extensions.html
(right):

https://codereview.chromium.org/1137753005/diff/40001/chrome/common/extension...
chrome/common/extensions/docs/templates/articles/external_extensions.html:6: <a
href="https://developer.chrome.com/webstore/inline_installation">inline
installation</a>,
For all of these links, you might as well link to
"/webstore/inline_installation" rather than the full URL
"https://developer.chrome.com/webstore/inline_installation". Better forwards
compatibility if we do ever change the host, and less typing (and typo-ability)
besides.

https://codereview.chromium.org/1137753005/diff/40001/chrome/common/extension...
File chrome/common/extensions/docs/templates/articles/single_purpose.html
(right):

https://codereview.chromium.org/1137753005/diff/40001/chrome/common/extension...
chrome/common/extensions/docs/templates/articles/single_purpose.html:12:
Beginning with the July release of Chrome,
You may want to say "July 2014" here?

https://codereview.chromium.org/1137753005/diff/40001/chrome/common/extension...
chrome/common/extensions/docs/templates/articles/single_purpose.html:18:
Starting in Chrome M44 in July 2015,
I think publicly we don't put the "M" prefix, it's just "Chrome 44"?

[not at google - send to devlin, 2015-05-13 18:23:37]

(but fix comments before submitting)

[mkearney1, 2015-05-13 18:36:36]

Review comments actioned.

https://codereview.chromium.org/1137753005/diff/40001/chrome/common/extension...
File chrome/common/extensions/docs/templates/articles/external_extensions.html
(right):

https://codereview.chromium.org/1137753005/diff/40001/chrome/common/extension...
chrome/common/extensions/docs/templates/articles/external_extensions.html:6: <a
href="https://developer.chrome.com/webstore/inline_installation">inline
installation</a>,
On 2015/05/13 18:23:27, kalman wrote:
> For all of these links, you might as well link to
> "/webstore/inline_installation" rather than the full URL
> "https://developer.chrome.com/webstore/inline_installation". Better forwards
> compatibility if we do ever change the host, and less typing (and
typo-ability)
> besides.

Done.

https://codereview.chromium.org/1137753005/diff/40001/chrome/common/extension...
File chrome/common/extensions/docs/templates/articles/single_purpose.html
(right):

https://codereview.chromium.org/1137753005/diff/40001/chrome/common/extension...
chrome/common/extensions/docs/templates/articles/single_purpose.html:12:
Beginning with the July release of Chrome,
On 2015/05/13 18:23:27, kalman wrote:
> You may want to say "July 2014" here?

Done.

https://codereview.chromium.org/1137753005/diff/40001/chrome/common/extension...
chrome/common/extensions/docs/templates/articles/single_purpose.html:18:
Starting in Chrome M44 in July 2015,
On 2015/05/13 18:23:27, kalman wrote:
> I think publicly we don't put the "M" prefix, it's just "Chrome 44"?

Done.

[mkearney1, 2015-05-13 18:37:00]

The CQ bit was checked by mkearney@chromium.org

[mkearney1, 2015-05-13 18:37:00]

The patchset sent to the CQ was uploaded after l-g-t-m from kalman@chromium.org
Link to the patchset: https://codereview.chromium.org/1137753005/#ps60001
(title: "Ben's fixes")

[commit-bot: I haz the power, 2015-05-13 18:37:46]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1137753005/60001

[commit-bot: I haz the power, 2015-05-13 18:43:41]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-05-13 18:44:35]

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