Second try of 157576 https://chromiumcodereview.appspot.com/10780013/

Second try of 157576 https://chromiumcodereview.appspot.com/10780013/

Add reverse URL handler for shortening uber URLs

BUG=120493
TEST=Open the settings page. The address bar should show "chrome://settings". Click the "extensions" button on the left. The address bar should show "chrome://extensions". Click the "settings" button on the left, "show advanced settings", "content settings". The address bar should show "chrome://settings/content".


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

[Cem Kocagil, 2012-09-19 22:23:56]

+ben: I haven't touched the files you reviewed last time (chrome_tests.gypi,
browser_navigator_browsertest.cc,
history_browsertest.cc, lazy_background_page_apitest.cc), can I have a lgtm here
too?

+creis: I haven't touched navigation_controller_impl.cc, can I have a lgtm here
too?

+asvitkine: Could you review the new
chrome_content_browser_client_browsertest.cc?

[Alexei Svitkine (slow), 2012-09-20 14:41:24]

LGTM.

I've sent off some try jobs for you.

[Charlie Reis, 2012-09-20 17:11:35]

navigation_controller_impl.cc still LGTM.

[Ben Goodger (Google), 2012-09-24 16:56:38]

lgtm

[commit-bot: I haz the power, 2012-09-24 22:57:27]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/[email protected]/10941050/2001

[commit-bot: I haz the power, 2012-09-25 03:06:28]

Change committed as 158504