ntp.html -> override.html

ntp.html -> override.html

This is exactly what you reviewed before, Erik. I plan to update the screenshots next week.

BUG=none
TEST=none
TBR=erikkay

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=31158

[David Glasser, 2009-12-01 18:28:32]

(This is my first time using Chromium Code Reviews; I hope I'm using proper
etiquette!)

http://codereview.chromium.org/376001/diff/2001/2035
File chrome/common/extensions/docs/manifest.html (right):

http://codereview.chromium.org/376001/diff/2001/2035#newcode265
chrome/common/extensions/docs/manifest.html:265: "<a
href="ntp.html">chrome_url_overrides</a>": {...},
You missed updating this link to override.html; the broken link is live on the
site at http://code.google.com/chrome/extensions/manifest.html

[Kathy Walrath, 2009-12-01 22:00:59]

Oops. Thanks for catching this. I've fixed it:
http://codereview.chromium.org/449072

Responding to the code review was fine. Another possibility is filing
a bug (http://code.google.com/p/chromium/issues/entry) with the labels
Feature-Extensions & Area-Content. (Or just assign to me, and I'll put
those labels on.)

-k-

On Tue, Dec 1, 2009 at 10:28 AM,  <glasser@google.com> wrote:
> (This is my first time using Chromium Code Reviews; I hope I'm using proper
> etiquette!)
>
>
> http://codereview.chromium.org/376001/diff/2001/2035
> File chrome/common/extensions/docs/manifest.html (right):
>
> http://codereview.chromium.org/376001/diff/2001/2035#newcode265
> chrome/common/extensions/docs/manifest.html:265: "<a
> href="ntp.html">chrome_url_overrides</a>": {...},
> You missed updating this link to override.html; the broken link is live
> on the site at http://code.google.com/chrome/extensions/manifest.html
>
> http://codereview.chromium.org/376001
>