Chromium Code Reviews Chromium Code Reviews
Issue 219213007:
Remove .html extension from links (Closed)
Base URL: https://chromium.googlesource.com/chromium/src.git@master| Index: chrome/common/extensions/docs/templates/articles/background_pages.html |
| diff --git a/chrome/common/extensions/docs/templates/articles/background_pages.html b/chrome/common/extensions/docs/templates/articles/background_pages.html |
| index a96bbc1113f66752a50d9cf40f2959b2fe6ec459..c19a61819bc2b0660717eae3baa37c1dfd2c9a68 100644 |
| --- a/chrome/common/extensions/docs/templates/articles/background_pages.html |
| +++ b/chrome/common/extensions/docs/templates/articles/background_pages.html |
| @@ -3,7 +3,7 @@ |
| <p id="eventPageWarning" class="warning"> |
| <em>Caution:</em> Consider using event pages instead. |
| - <a href="event_pages.html">Learn more</a>. |
| + <a href="event_pages">Learn more</a>. |
| </p> |
| <p> |
| @@ -13,11 +13,11 @@ Background pages to the rescue. |
| </p> |
| <p> |
| -As the <a href="overview.html#arch">architecture overview</a> explains, |
| +As the <a href="overview#arch">architecture overview</a> explains, |
| the background page is an HTML page that runs in the extension process. |
| It exists for the lifetime of your extension, |
| and only one instance of it at a time is active. (Exception: if your |
| -extension uses <a href="manifest/incognito.html">incognito</a> |
| +extension uses <a href="manifest/incognito">incognito</a> |
| "split" mode, a second instance is created for incognito windows.) |
| </p> |
| @@ -37,7 +37,7 @@ the background page tells the views to update. |
| <p> |
| Register your background page in the |
| -<a href="manifest.html">extension manifest</a>. |
| +<a href="manifest">extension manifest</a>. |
| In the common case, a background page |
| does not require any HTML markup. |
| These kind of background pages can be |
| @@ -74,7 +74,7 @@ property instead: |
| "name": "My extension", |
| ... |
| <b>"background": { |
| - "page": "background.html" |
| + "page": "background" |
|
mkearney1
2014/04/09 19:43:30
Keep .html in as it refers to filename.
|
| }</b>, |
| ... |
| }</pre> |
| @@ -83,7 +83,7 @@ property instead: |
| If you need the browser to start up early—so |
| you can display notifications, for example—then |
| you might also want to specify the |
| -<a href="declare_permissions.html#background">"background" permission</a>. |
| +<a href="declare_permissions#background">"background" permission</a>. |
| </p> |
| @@ -117,14 +117,14 @@ has a background page |
| and multiple pages created |
| (with |
| $(ref:tabs.create)) |
| -from a file named <code>image.html</code>. |
| +from a file named <code>image</code>. |
|
mkearney1
2014/04/09 19:43:30
Keep .html in as it refers to filename.
|
| </p> |
| <pre data-filename="background.js"> |
| // React when a browser action's icon is clicked. |
| chrome.browserAction.onClicked.addListener(function(tab) { |
| - var viewTabUrl = chrome.extension.getURL('image.html'); |
| + var viewTabUrl = chrome.extension.getURL('image'); |
|
mkearney1
2014/04/09 19:43:30
Keep .html in as it refers to filename.
|
| var imageUrl = <em>/* an image's URL */</em>; |
| // Look through all the pages in this extension to find one we can use. |
| @@ -143,7 +143,7 @@ chrome.browserAction.onClicked.addListener(function(tab) { |
| } |
| }); |
| </pre> |
| -<pre data-filename="image.html"> |
| +<pre data-filename="image"> |
|
mkearney1
2014/04/09 19:43:30
Keep .html in as it refers to filename.
|
| <html> |
| <script> |
| function setImageUrl(url) { |
