Add support for multipage distillation.

components/dom_distiller/content/distiller_page_web_contents.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/dom_distiller/content/distiller_page_web_contents.h"
 
 #include "base/callback.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/strings/utf_string_conversions.h"
 #include "components/dom_distiller/core/distiller_page.h"
@@ skipping 44 matching lines @@
       base::string16(),  // frame_xpath
       base::UTF8ToUTF16(script),
       base::Bind(&DistillerPage::OnExecuteJavaScriptDone,
                  base::Unretained(this)));
 }
 
 void DistillerPageWebContents::DidFinishLoad(int64 frame_id,
                                              const GURL& validated_url,
                                              bool is_main_frame,
                                              RenderViewHost* render_view_host) {
+  if (validated_url != web_contents_->GetURL()) {

[Yaron, 2014/01/29 20:03:41]

Why are these necessary? To ensure the page wasn't redirected?

[shashi, 2014/01/29 22:51:37]

Done, I can use is_main_frame.

+    return;
+  }
   content::WebContentsObserver::Observe(NULL);
   OnLoadURLDone();
 }
+
 void DistillerPageWebContents::DidFailLoad(
     int64 frame_id,
     const GURL& validated_url,
     bool is_main_frame,
     int error_code,
     const base::string16& error_description,
     RenderViewHost* render_view_host) {
+  if (validated_url != web_contents_->GetURL()) {
+    return;
+  }
   content::WebContentsObserver::Observe(NULL);
   OnLoadURLFailed();
 }
 
 }  // namespace dom_distiller