Chromium Code Reviews
Help | Chromium Project | Sign in
(23)

Issue 3092007: Delete chrome/browser/page_state.*. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
4 years, 10 months ago by viettrungluu
Modified:
3 years, 11 months ago
Reviewers:
Mohamed Mansour
CC:
chromium-reviews, ben+cc_chromium.org
Visibility:
Public.

Description

Delete chrome/browser/page_state.*. There are exactly zero references to them. BUG=none TEST=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=55025

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -176 lines) Patch
D chrome/browser/page_state.h View 1 chunk +0 lines, -63 lines 0 comments Download
D chrome/browser/page_state.cc View 1 chunk +0 lines, -111 lines 0 comments Download
M chrome/chrome_browser.gypi View 1 chunk +0 lines, -2 lines 0 comments Download
Commit: CQ not working?

Messages

Total messages: 1 (0 generated)
Mohamed Mansour
4 years, 10 months ago (2010-08-05 03:29:53 UTC) #1
LGTM, this class was never used. If in any case someone wants it back, they can
revive it. We currently use different ways to store navigation states.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld ec887be