add pagination info to DistilledPageProto

components/dom_distiller/core/distiller.cc

Index: components/dom_distiller/core/distiller.cc
diff --git a/components/dom_distiller/core/distiller.cc b/components/dom_distiller/core/distiller.cc
index f4cdabfa322aa9d866609a1bb8955247f3de66af..f54e24cb8b272dabff5d19e9d4c4c6a3bd95f525 100644
--- a/components/dom_distiller/core/distiller.cc
+++ b/components/dom_distiller/core/distiller.cc
@@ -170,6 +170,8 @@ void DistillerImpl::OnPageDistillationFinished(
           // The pages should be in same origin.
           DCHECK_EQ(next_page_url.GetOrigin(), page_url.GetOrigin());
           AddToDistillationQueue(page_num + 1, next_page_url);
+          page_data->distilled_page_proto->data.mutable_pagination_info()->
+              set_next_page(next_page_url.spec());
         }
       }
 
@@ -178,6 +180,8 @@ void DistillerImpl::OnPageDistillationFinished(
         if (prev_page_url.is_valid()) {
           DCHECK_EQ(prev_page_url.GetOrigin(), page_url.GetOrigin());
           AddToDistillationQueue(page_num - 1, prev_page_url);
+          page_data->distilled_page_proto->data.mutable_pagination_info()->

[nyquist, 2014/10/24 20:04:02]

Since you added canonical_page to the DistilledPage::PaginationInfo proto, any
specific reason you do not copy the value over from the distiller result here?

[kuan, 2014/10/24 20:57:32]

Done.  no reason, except that it's not used, but i guess the same argument cld
be made for prev_page.

+              set_prev_page(prev_page_url.spec());
         }
       }
     }