NNTP: Merge the recent activities into one list and show more items...

NNTP: Merge the recent activities into one list and show more items
when the section is larger.

Make the window menu look and behave more like a tooltip.

Return more recently closed windows and tabs when using the NNTP.

BUG=16539, 16835, 14963
TEST=Hide and show the tips section. More items (if there are more
items) should be shown. Hover over a closed window and the tooltip
should look like a tooltip on Vista/Win7


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

[Glen Murphy, 2009-07-21 20:14:01]

LGTM with minors:

http://codereview.chromium.org/155865/diff/13/1014
File chrome/browser/dom_ui/new_tab_ui.cc (right):

http://codereview.chromium.org/155865/diff/13/1014#newcode1221
Line 1221: const int max_count = NewTabUI::UseOldNewTabPage() ? 3 : 6;
Fussy people might ask you to make these static const ints at the top of the
file.

http://codereview.chromium.org/155865/diff/13/1011
File chrome/browser/resources/new_new_tab.css (right):

http://codereview.chromium.org/155865/diff/13/1011#newcode444
Line 444: to(rgb(228,229,240)));
(Meta-question) How are we going to theme all these hardcoded colors sanely?

http://codereview.chromium.org/155865/diff/13/1012
File chrome/browser/resources/new_new_tab.html (right):

http://codereview.chromium.org/155865/diff/13/1012#newcode244
Line 244: .style.backgroundImage:'url(chrome://theme/downloads_favicon)';
80col? (can't tell)

[arv (Not doing code reviews), 2009-07-21 20:44:47]

http://codereview.chromium.org/155865/diff/13/1011
File chrome/browser/resources/new_new_tab.css (right):

http://codereview.chromium.org/155865/diff/13/1011#newcode444
Line 444: to(rgb(228,229,240)));
On 2009/07/21 20:14:01, Glen Murphy wrote:
> (Meta-question) How are we going to theme all these hardcoded colors sanely?

These ones should be covered by the CSS appearance.

The rest should just have a master color and we should calculate the other
colors from it by making things brighter and darker. Gmail did this pretty
efficiently. The theme provided 4 or so basic colors and the rest where
calculated based on that.

http://codesearch/p#google3/java/com/google/caribou/ui/pinto/gss/leaves/leave...

I think one thing we could do is to use some CSS template language that allows
us to use named constants for the colors.

http://codereview.chromium.org/155865/diff/13/1012
File chrome/browser/resources/new_new_tab.html (right):

http://codereview.chromium.org/155865/diff/13/1012#newcode244
Line 244: .style.backgroundImage:'url(chrome://theme/downloads_favicon)';
On 2009/07/21 20:14:01, Glen Murphy wrote:
> 80col? (can't tell)

Done.