Move about://-related constants from //content to //url

Move about://-related constants from //content to //url

kAboutScheme and kAboutBlankURL are used in code shared by iOS, and are general
enough to live in //url instead of //content.

TBR=yoz,benm

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=275988

[blundell, 2014-06-06 18:56:07]

jam: //content and //chrome (sorry)
brettw: //url

[jam, 2014-06-06 19:39:23]

lgtm

https://codereview.chromium.org/325443002/diff/1/content/renderer/render_fram...
File content/renderer/render_frame_impl.cc (right):

https://codereview.chromium.org/325443002/diff/1/content/renderer/render_fram...
content/renderer/render_frame_impl.cc:3239: if (!frame->parent() &&
is_content_initiated && !url.SchemeIs(url::kAboutScheme)) {
nit: 80 chars

[blundell, 2014-06-08 18:47:23]

https://codereview.chromium.org/325443002/diff/1/content/renderer/render_fram...
File content/renderer/render_frame_impl.cc (right):

https://codereview.chromium.org/325443002/diff/1/content/renderer/render_fram...
content/renderer/render_frame_impl.cc:3239: if (!frame->parent() &&
is_content_initiated && !url.SchemeIs(url::kAboutScheme)) {
On 2014/06/06 19:39:23, jam wrote:
> nit: 80 chars

Done.

[brettw, 2014-06-09 16:38:02]

lgtm

[blundell, 2014-06-09 16:53:25]

TBR=benm for //android_webview
TBR=yoz for //apps, //extensions

[blundell, 2014-06-09 16:53:32]

The CQ bit was checked by blundell@chromium.org

[commit-bot: I haz the power, 2014-06-09 16:54:02]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/blundell@chromium.org/325443002/60001

[commit-bot: I haz the power, 2014-06-10 11:00:04]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_clang_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_clang_dbg/bui...)

[commit-bot: I haz the power, 2014-06-10 11:46:26]

Change committed as 275988