NTP: Fix startup visual state ...

NTP: Fix startup visual state 

This inject the template data earlier in the document so that we can set
the initial visual state based on the user preference. This prevents
flickering when the user is not showing the most visited thumbnails.

BUG=24513
TEST=Use a theme with a background image to make this easier to spot.
Hide the most visited. Restart Chrome. The thumbnails should not flicker

[Miranda Callahan, 2009-10-15 19:36:38]

Hmm -- I just tried this patch out, and when I go to list view for most visited
and refresh the NTP, it shows thumbnail view (even though the list view button
is still toggled).


On 2009/10/15 18:59:36, arv wrote:
>

[arv (Not doing code reviews), 2009-10-15 20:38:47]

You are correct. I can't believe I missed that. I must have reloaded
the page 100 times when working on this and I never tried that case.

I'll ping you when I got it working.

On Thu, Oct 15, 2009 at 12:36,  <mirandac@chromium.org> wrote:
> Hmm -- I just tried this patch out, and when I go to list view for most
> visited
> and refresh the NTP, it shows thumbnail view (even though the list view
> button
> is still toggled).
>
>
> On 2009/10/15 18:59:36, arv wrote:
>
>
>
>
> http://codereview.chromium.org/269095
>

[arv (Not doing code reviews), 2009-10-15 22:18:33]

PTAL

[arv (Not doing code reviews), 2009-10-15 22:28:53]

I find the layout code to be really ugly. I think I might take a stab at
cleaning it up.

[mirandac, 2009-10-15 22:38:12]

That's fine -- just ping me when you want the review.

On Thu, Oct 15, 2009 at 3:28 PM, <arv@chromium.org> wrote:

> I find the layout code to be really ugly. I think I might take a stab at
> cleaning it up.
>
>
> http://codereview.chromium.org/269095
>

[Miranda Callahan, 2009-10-16 00:37:41]

lgtm!

On 2009/10/15 22:38:12, mirandac_google.com wrote:
> That's fine -- just ping me when you want the review.
> 
> On Thu, Oct 15, 2009 at 3:28 PM, <mailto:arv@chromium.org> wrote:
> 
> > I find the layout code to be really ugly. I think I might take a stab at
> > cleaning it up.
> >
> >
> > http://codereview.chromium.org/269095
> >
>