| Index: chrome/browser/history/history_backend.cc
|
| diff --git a/chrome/browser/history/history_backend.cc b/chrome/browser/history/history_backend.cc
|
| index 1ab3a552d3f9333101796ea24602d752038942af..7c7dcf26ee4129b5ef9334dd6292c44d4f9e3815 100644
|
| --- a/chrome/browser/history/history_backend.cc
|
| +++ b/chrome/browser/history/history_backend.cc
|
| @@ -322,18 +322,18 @@ SegmentID HistoryBackend::UpdateSegments(
|
| const GURL& url,
|
| VisitID from_visit,
|
| VisitID visit_id,
|
| - content::PageTransition transition_type,
|
| + ui::PageTransition transition_type,
|
| const Time ts) {
|
| if (!db_)
|
| return 0;
|
|
|
| // We only consider main frames.
|
| - if (!content::PageTransitionIsMainFrame(transition_type))
|
| + if (!ui::PageTransitionIsMainFrame(transition_type))
|
| return 0;
|
|
|
| SegmentID segment_id = 0;
|
| - content::PageTransition t =
|
| - content::PageTransitionStripQualifier(transition_type);
|
| + ui::PageTransition t =
|
| + ui::PageTransitionStripQualifier(transition_type);
|
|
|
| // Are we at the beginning of a new segment?
|
| // Note that navigating to an existing entry (with back/forward) reuses the
|
| @@ -348,9 +348,9 @@ SegmentID HistoryBackend::UpdateSegments(
|
| // Note also that we should still be updating the visit count for that segment
|
| // which we are not doing now. It should be addressed when
|
| // http://crbug.com/96860 is fixed.
|
| - if ((t == content::PAGE_TRANSITION_TYPED ||
|
| - t == content::PAGE_TRANSITION_AUTO_BOOKMARK) &&
|
| - (transition_type & content::PAGE_TRANSITION_FORWARD_BACK) == 0) {
|
| + if ((t == ui::PAGE_TRANSITION_TYPED ||
|
| + t == ui::PAGE_TRANSITION_AUTO_BOOKMARK) &&
|
| + (transition_type & ui::PAGE_TRANSITION_FORWARD_BACK) == 0) {
|
| // If so, create or get the segment.
|
| std::string segment_name = db_->ComputeSegmentName(url);
|
| URLID url_id = db_->GetRowForURL(url, NULL);
|
| @@ -437,18 +437,18 @@ void HistoryBackend::AddPage(const HistoryAddPageArgs& request) {
|
| if (request.time < first_recorded_time_)
|
| first_recorded_time_ = request.time;
|
|
|
| - content::PageTransition request_transition = request.transition;
|
| - content::PageTransition stripped_transition =
|
| - content::PageTransitionStripQualifier(request_transition);
|
| + ui::PageTransition request_transition = request.transition;
|
| + ui::PageTransition stripped_transition =
|
| + ui::PageTransitionStripQualifier(request_transition);
|
| bool is_keyword_generated =
|
| - (stripped_transition == content::PAGE_TRANSITION_KEYWORD_GENERATED);
|
| + (stripped_transition == ui::PAGE_TRANSITION_KEYWORD_GENERATED);
|
|
|
| // If the user is navigating to a not-previously-typed intranet hostname,
|
| // change the transition to TYPED so that the omnibox will learn that this is
|
| // a known host.
|
| bool has_redirects = request.redirects.size() > 1;
|
| - if (content::PageTransitionIsMainFrame(request_transition) &&
|
| - (stripped_transition != content::PAGE_TRANSITION_TYPED) &&
|
| + if (ui::PageTransitionIsMainFrame(request_transition) &&
|
| + (stripped_transition != ui::PAGE_TRANSITION_TYPED) &&
|
| !is_keyword_generated) {
|
| const GURL& origin_url(has_redirects ?
|
| request.redirects[0] : request.url);
|
| @@ -462,21 +462,21 @@ void HistoryBackend::AddPage(const HistoryAddPageArgs& request) {
|
| net::registry_controlled_domains::EXCLUDE_UNKNOWN_REGISTRIES,
|
| net::registry_controlled_domains::EXCLUDE_PRIVATE_REGISTRIES);
|
| if (registry_length == 0 && !db_->IsTypedHost(host)) {
|
| - stripped_transition = content::PAGE_TRANSITION_TYPED;
|
| + stripped_transition = ui::PAGE_TRANSITION_TYPED;
|
| request_transition =
|
| - content::PageTransitionFromInt(
|
| + ui::PageTransitionFromInt(
|
| stripped_transition |
|
| - content::PageTransitionGetQualifier(request_transition));
|
| + ui::PageTransitionGetQualifier(request_transition));
|
| }
|
| }
|
| }
|
|
|
| if (!has_redirects) {
|
| // The single entry is both a chain start and end.
|
| - content::PageTransition t = content::PageTransitionFromInt(
|
| + ui::PageTransition t = ui::PageTransitionFromInt(
|
| request_transition |
|
| - content::PAGE_TRANSITION_CHAIN_START |
|
| - content::PAGE_TRANSITION_CHAIN_END);
|
| + ui::PAGE_TRANSITION_CHAIN_START |
|
| + ui::PAGE_TRANSITION_CHAIN_END);
|
|
|
| // No redirect case (one element means just the page itself).
|
| last_ids = AddPageVisit(request.url, request.time,
|
| @@ -495,8 +495,8 @@ void HistoryBackend::AddPage(const HistoryAddPageArgs& request) {
|
| } else {
|
| // Redirect case. Add the redirect chain.
|
|
|
| - content::PageTransition redirect_info =
|
| - content::PAGE_TRANSITION_CHAIN_START;
|
| + ui::PageTransition redirect_info =
|
| + ui::PAGE_TRANSITION_CHAIN_START;
|
|
|
| RedirectList redirects = request.redirects;
|
| if (redirects[0].SchemeIs(url::kAboutScheme)) {
|
| @@ -510,8 +510,8 @@ void HistoryBackend::AddPage(const HistoryAddPageArgs& request) {
|
| // In this case, we just don't bother hooking up the source of the
|
| // redirects, so we remove it.
|
| redirects.erase(redirects.begin());
|
| - } else if (request_transition & content::PAGE_TRANSITION_CLIENT_REDIRECT) {
|
| - redirect_info = content::PAGE_TRANSITION_CLIENT_REDIRECT;
|
| + } else if (request_transition & ui::PAGE_TRANSITION_CLIENT_REDIRECT) {
|
| + redirect_info = ui::PAGE_TRANSITION_CLIENT_REDIRECT;
|
| // The first entry in the redirect chain initiated a client redirect.
|
| // We don't add this to the database since the referrer is already
|
| // there, so we skip over it but change the transition type of the first
|
| @@ -532,9 +532,9 @@ void HistoryBackend::AddPage(const HistoryAddPageArgs& request) {
|
| VisitRow visit_row;
|
| if (request.did_replace_entry &&
|
| db_->GetRowForVisit(last_ids.second, &visit_row) &&
|
| - visit_row.transition & content::PAGE_TRANSITION_CHAIN_END) {
|
| - visit_row.transition = content::PageTransitionFromInt(
|
| - visit_row.transition & ~content::PAGE_TRANSITION_CHAIN_END);
|
| + visit_row.transition & ui::PAGE_TRANSITION_CHAIN_END) {
|
| + visit_row.transition = ui::PageTransitionFromInt(
|
| + visit_row.transition & ~ui::PAGE_TRANSITION_CHAIN_END);
|
| db_->UpdateVisitRow(visit_row);
|
| }
|
| }
|
| @@ -542,13 +542,13 @@ void HistoryBackend::AddPage(const HistoryAddPageArgs& request) {
|
|
|
| for (size_t redirect_index = 0; redirect_index < redirects.size();
|
| redirect_index++) {
|
| - content::PageTransition t =
|
| - content::PageTransitionFromInt(stripped_transition | redirect_info);
|
| + ui::PageTransition t =
|
| + ui::PageTransitionFromInt(stripped_transition | redirect_info);
|
|
|
| // If this is the last transition, add a CHAIN_END marker
|
| if (redirect_index == (redirects.size() - 1)) {
|
| - t = content::PageTransitionFromInt(
|
| - t | content::PAGE_TRANSITION_CHAIN_END);
|
| + t = ui::PageTransitionFromInt(
|
| + t | ui::PAGE_TRANSITION_CHAIN_END);
|
| }
|
|
|
| // Record all redirect visits with the same timestamp. We don't display
|
| @@ -557,7 +557,7 @@ void HistoryBackend::AddPage(const HistoryAddPageArgs& request) {
|
| last_ids = AddPageVisit(redirects[redirect_index],
|
| request.time, last_ids.second,
|
| t, request.visit_source);
|
| - if (t & content::PAGE_TRANSITION_CHAIN_START) {
|
| + if (t & ui::PAGE_TRANSITION_CHAIN_START) {
|
| // Update the segment for this visit.
|
| UpdateSegments(redirects[redirect_index],
|
| from_visit_id, last_ids.second, t, request.time);
|
| @@ -568,7 +568,7 @@ void HistoryBackend::AddPage(const HistoryAddPageArgs& request) {
|
|
|
| // Subsequent transitions in the redirect list must all be server
|
| // redirects.
|
| - redirect_info = content::PAGE_TRANSITION_SERVER_REDIRECT;
|
| + redirect_info = ui::PAGE_TRANSITION_SERVER_REDIRECT;
|
| }
|
|
|
| // Last, save this redirect chain for later so we can set titles & favicons
|
| @@ -583,8 +583,8 @@ void HistoryBackend::AddPage(const HistoryAddPageArgs& request) {
|
| // TODO(evanm): Due to http://b/1194536 we lose the referrers of a subframe
|
| // navigation anyway, so last_visit_id is always zero for them. But adding
|
| // them here confuses main frame history, so we skip them for now.
|
| - if (stripped_transition != content::PAGE_TRANSITION_AUTO_SUBFRAME &&
|
| - stripped_transition != content::PAGE_TRANSITION_MANUAL_SUBFRAME &&
|
| + if (stripped_transition != ui::PAGE_TRANSITION_AUTO_SUBFRAME &&
|
| + stripped_transition != ui::PAGE_TRANSITION_MANUAL_SUBFRAME &&
|
| !is_keyword_generated) {
|
| tracker_.AddVisit(request.context_id, request.page_id, request.url,
|
| last_ids.second);
|
| @@ -734,28 +734,28 @@ std::pair<URLID, VisitID> HistoryBackend::AddPageVisit(
|
| const GURL& url,
|
| Time time,
|
| VisitID referring_visit,
|
| - content::PageTransition transition,
|
| + ui::PageTransition transition,
|
| VisitSource visit_source) {
|
| // Top-level frame navigations are visible, everything else is hidden
|
| - bool new_hidden = !content::PageTransitionIsMainFrame(transition);
|
| + bool new_hidden = !ui::PageTransitionIsMainFrame(transition);
|
|
|
| // NOTE: This code must stay in sync with
|
| // ExpireHistoryBackend::ExpireURLsForVisits().
|
| // TODO(pkasting): http://b/1148304 We shouldn't be marking so many URLs as
|
| // typed, which would eliminate the need for this code.
|
| int typed_increment = 0;
|
| - content::PageTransition transition_type =
|
| - content::PageTransitionStripQualifier(transition);
|
| - if ((transition_type == content::PAGE_TRANSITION_TYPED &&
|
| - !content::PageTransitionIsRedirect(transition)) ||
|
| - transition_type == content::PAGE_TRANSITION_KEYWORD_GENERATED)
|
| + ui::PageTransition transition_type =
|
| + ui::PageTransitionStripQualifier(transition);
|
| + if ((transition_type == ui::PAGE_TRANSITION_TYPED &&
|
| + !ui::PageTransitionIsRedirect(transition)) ||
|
| + transition_type == ui::PAGE_TRANSITION_KEYWORD_GENERATED)
|
| typed_increment = 1;
|
|
|
| #if defined(OS_ANDROID)
|
| // Only count the page visit if it came from user browsing and only count it
|
| // once when cycling through a redirect chain.
|
| if (visit_source == SOURCE_BROWSED &&
|
| - (transition & content::PAGE_TRANSITION_CHAIN_END) != 0) {
|
| + (transition & ui::PAGE_TRANSITION_CHAIN_END) != 0) {
|
| RecordTopPageVisitStats(url);
|
| }
|
| #endif
|
| @@ -765,8 +765,8 @@ std::pair<URLID, VisitID> HistoryBackend::AddPageVisit(
|
| URLID url_id = db_->GetRowForURL(url, &url_info);
|
| if (url_id) {
|
| // Update of an existing row.
|
| - if (content::PageTransitionStripQualifier(transition) !=
|
| - content::PAGE_TRANSITION_RELOAD)
|
| + if (ui::PageTransitionStripQualifier(transition) !=
|
| + ui::PAGE_TRANSITION_RELOAD)
|
| url_info.set_visit_count(url_info.visit_count() + 1);
|
| if (typed_increment)
|
| url_info.set_typed_count(url_info.typed_count() + typed_increment);
|
| @@ -854,10 +854,10 @@ void HistoryBackend::AddPagesWithDetails(const URLRows& urls,
|
| if (visit_source != SOURCE_SYNCED) {
|
| // Make up a visit to correspond to the last visit to the page.
|
| VisitRow visit_info(url_id, i->last_visit(), 0,
|
| - content::PageTransitionFromInt(
|
| - content::PAGE_TRANSITION_LINK |
|
| - content::PAGE_TRANSITION_CHAIN_START |
|
| - content::PAGE_TRANSITION_CHAIN_END), 0);
|
| + ui::PageTransitionFromInt(
|
| + ui::PAGE_TRANSITION_LINK |
|
| + ui::PAGE_TRANSITION_CHAIN_START |
|
| + ui::PAGE_TRANSITION_CHAIN_END), 0);
|
| if (!db_->AddVisit(&visit_info, visit_source)) {
|
| NOTREACHED() << "Adding visit failed.";
|
| return;
|
| @@ -1253,7 +1253,7 @@ void HistoryBackend::QueryHistoryBasic(const QueryOptions& options,
|
| // Set whether the visit was blocked for a managed user by looking at the
|
| // transition type.
|
| url_result.set_blocked_visit(
|
| - (visit.transition & content::PAGE_TRANSITION_BLOCKED) != 0);
|
| + (visit.transition & ui::PAGE_TRANSITION_BLOCKED) != 0);
|
|
|
| // We don't set any of the query-specific parts of the URLResult, since
|
| // snippets and stuff don't apply to basic querying.
|
|
|