[uber page] Integrate history into uber page.

[uber page] Integrate history into uber page.

Insert the history page into uber, redirect history menu command and urls to the uber
page.  Also included in this CL:
* Use localized strings for frame URLs in uber.html.
* Turn off uppercase on page titles per latest mocks.
* Re-order page titles per latest mocks. Reflected order in code for readability.

BUG=116447
TEST=Exercise history in uber page, invoking from menu commands.
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=124872

[csilv, 2012-03-02 23:05:54]

+estade for review

https://chromiumcodereview.appspot.com/9582032/diff/2003/chrome/browser/brows...
File chrome/browser/browser_about_handler.cc (right):

https://chromiumcodereview.appspot.com/9582032/diff/2003/chrome/browser/brows...
chrome/browser/browser_about_handler.cc:137: host =
chrome::kChromeUIHistoryFrameHost;
This isn't ideal... exposing the 'history-frame' url if uber is disabled, but I
think we're 100% committed to uber for M19 so I'm taking this shortcut.

[Evan Stade, 2012-03-02 23:29:54]

https://chromiumcodereview.appspot.com/9582032/diff/2003/chrome/browser/brows...
File chrome/browser/browser_about_handler.cc (right):

https://chromiumcodereview.appspot.com/9582032/diff/2003/chrome/browser/brows...
chrome/browser/browser_about_handler.cc:137: host =
chrome::kChromeUIHistoryFrameHost;
On 2012/03/02 23:05:54, csilv wrote:
> This isn't ideal... exposing the 'history-frame' url if uber is disabled, but
I
> think we're 100% committed to uber for M19 so I'm taking this shortcut.

it's ok. History looks pretty funny without the uber frame. That's a bigger deal
than the url.

https://chromiumcodereview.appspot.com/9582032/diff/2003/chrome/browser/resou...
File chrome/browser/resources/uber/uber.html (right):

https://chromiumcodereview.appspot.com/9582032/diff/2003/chrome/browser/resou...
chrome/browser/resources/uber/uber.html:30:
i18n-values="id:settingsHost;data-url:settingsFrameURL;"
you should use a space to separate the attributes

https://chromiumcodereview.appspot.com/9582032/diff/2003/chrome/browser/ui/we...
File chrome/browser/ui/webui/uber/uber_ui.cc (right):

https://chromiumcodereview.appspot.com/9582032/diff/2003/chrome/browser/ui/we...
chrome/browser/ui/webui/uber/uber_ui.cc:96:
RegisterSubpage(chrome::kChromeUIUberFrameURL);
I would alphabetize these instead but I don't feel strongly. The order here is
not obvious, also if someone were to move the order of the pages in the HTML I
highly doubt they'd think to hunt this down and change it, so it will get out of
sync.

[csilv, 2012-03-02 23:48:09]

https://chromiumcodereview.appspot.com/9582032/diff/2003/chrome/browser/resou...
File chrome/browser/resources/uber/uber.html (right):

https://chromiumcodereview.appspot.com/9582032/diff/2003/chrome/browser/resou...
chrome/browser/resources/uber/uber.html:30:
i18n-values="id:settingsHost;data-url:settingsFrameURL;"
On 2012/03/02 23:29:54, Evan Stade wrote:
> you should use a space to separate the attributes

Done.

https://chromiumcodereview.appspot.com/9582032/diff/2003/chrome/browser/ui/we...
File chrome/browser/ui/webui/uber/uber_ui.cc (right):

https://chromiumcodereview.appspot.com/9582032/diff/2003/chrome/browser/ui/we...
chrome/browser/ui/webui/uber/uber_ui.cc:96:
RegisterSubpage(chrome::kChromeUIUberFrameURL);
On 2012/03/02 23:29:54, Evan Stade wrote:
> I would alphabetize these instead but I don't feel strongly. The order here is
> not obvious, also if someone were to move the order of the pages in the HTML I
> highly doubt they'd think to hunt this down and change it, so it will get out
of
> sync.

Done.

[Evan Stade, 2012-03-02 23:49:47]

lgtm