Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(47)

Issue 174030: Change default homepage experience. (Closed)

Created:
11 years, 4 months ago by kuchhal
Modified:
9 years, 7 months ago
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

Change default homepage experience. We want to make google.com homepage by default and also show the home button (see the bug below). BUG=http://crbug.com/19392 TEST=Delete user profile and launch Chrome - homepage should be set to google.com and should be enabled. also home button should show up.

Patch Set 1 #

Patch Set 2 : Update so only new users see this #

Unified diffs Side-by-side diffs Delta from patch set Stats (+31 lines, -13 lines) Patch
M chrome/browser/browser.h View 1 chunk +6 lines, -0 lines 0 comments Download
M chrome/browser/browser.cc View 1 2 chunks +9 lines, -0 lines 0 comments Download
M chrome/browser/browser_main.cc View 1 chunk +14 lines, -11 lines 0 comments Download
M chrome/browser/google_url_tracker.h View 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 11 (0 generated)
kuchhal
Ben, can you OK this? Only couple of lines of code change but significant functional ...
11 years, 4 months ago (2009-08-18 23:07:52 UTC) #1
Ben Goodger (Google)
On 2009/08/18 23:07:52, kuchhal wrote: > Ben, can you OK this? Only couple of lines ...
11 years, 4 months ago (2009-08-18 23:15:50 UTC) #2
kuchhal
Ben/Glen, Any feedback yet?
11 years, 4 months ago (2009-08-21 16:52:22 UTC) #3
Peter Kasting
I think we should not make these changes yet, and left comments on the bug ...
11 years, 4 months ago (2009-08-21 18:23:28 UTC) #4
Ben Goodger (Google)
Glen and I need to discuss this with Brian/Linus next week. We are not ready ...
11 years, 4 months ago (2009-08-22 07:06:36 UTC) #5
Ben Goodger (Google)
From the looks of this change, it seems to me as if you're effectively resetting ...
11 years, 4 months ago (2009-08-22 23:38:33 UTC) #6
brakowski_google.com
2009/8/22 Ben Goodger (Google) <ben@chromium.org> > > From the looks of this change, it seems ...
11 years, 4 months ago (2009-08-24 03:22:01 UTC) #7
kuchhal
Ohh.. I was trying to change it only for new installs. Let me take a ...
11 years, 4 months ago (2009-08-24 16:40:20 UTC) #8
kuchhal
Changed to set new values only during First Run as that seems to be a ...
11 years, 4 months ago (2009-08-25 00:29:28 UTC) #9
kuchhal
Ben can you take a look at the new changes? On Mon, Aug 24, 2009 ...
11 years, 3 months ago (2009-08-27 22:25:36 UTC) #10
Ben Goodger (Google)
11 years, 3 months ago (2009-08-27 22:29:17 UTC) #11
OK.

Powered by Google App Engine
This is Rietveld 408576698