InstantExtended: Committed NTP

content/public/common/url_constants.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_PUBLIC_COMMON_URL_CONSTANTS_H_
 #define CONTENT_PUBLIC_COMMON_URL_CONSTANTS_H_
 
 #include "content/common/content_export.h"
 
 // Contains constants for known URLs and portions thereof.
@@ skipping 38 matching lines @@
 CONTENT_EXPORT extern const char kChromeUIResourcesHost[];
 CONTENT_EXPORT extern const char kChromeUITcmallocHost[];
 
 // Full about URLs (including schemes).
 CONTENT_EXPORT extern const char kChromeUICrashURL[];
 CONTENT_EXPORT extern const char kChromeUIGpuCleanURL[];
 CONTENT_EXPORT extern const char kChromeUIGpuCrashURL[];
 CONTENT_EXPORT extern const char kChromeUIGpuHangURL[];
 CONTENT_EXPORT extern const char kChromeUIHangURL[];
 CONTENT_EXPORT extern const char kChromeUIKillURL[];
+CONTENT_EXPORT extern const char kChromeUINewTabURL[];

[jam, 2013/01/29 17:51:55]

content layer should not know about chrome layer's features, i.e. new tab page

[samarth, 2013/01/31 16:24:57]

Do you have other suggestions for how best to communicate to the content layer
that a particular contents should have FocusLocationBarByDefault() set to true?
Some options:

(1) Add some explicit API that chrome/ code can call on a web contents to set
FocusLocationBarByDefault(true)
(2) Add another channel for communicating this bit?  (Similar to how it's done
right now with webui but a new one since we don't have a webui in this case
anymore.)
(3) Don't do it content/ at all and just try to cover all the cases on the
chrome side.  It may be possible but it would definitely be more bug-prone.
(4) Keep it like this. It's simplest and it's how we're currently handling
focus-by-default for about://blank.  Also it looks like there's already a bunch
of chrome-specific URLs in this list (e.g. ChromeUIAppCacheInternals,
ChromeUIDevTools, etc.)

 CONTENT_EXPORT extern const char kChromeUIPpapiFlashCrashURL[];
 CONTENT_EXPORT extern const char kChromeUIPpapiFlashHangURL[];
 
 }  // namespace chrome
 
 namespace content {
 
 // Used to separate a standard scheme and the hostname: "://".
 CONTENT_EXPORT extern const char kStandardSchemeSeparator[];
 
@@ skipping 11 matching lines @@
 // invoked on any thread.
 CONTENT_EXPORT const char* const* GetSavableSchemes();
 
 // Returns true if the url has a scheme for WebUI.  See also
 // WebUIControllerFactory::UseWebUIForURL in the browser process.
 CONTENT_EXPORT bool HasWebUIScheme(const GURL& url);
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_COMMON_URL_CONSTANTS_H_