Fix domdistiller for new JS execution

components/dom_distiller/ios/distiller_page_ios.h

 // Copyright 2014 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.
 
 #ifndef COMPONENTS_DOM_DISTILLER_IOS_DISTILLER_PAGE_IOS_H_
 #define COMPONENTS_DOM_DISTILLER_IOS_DISTILLER_PAGE_IOS_H_
 
 #include <memory>
 #include <string>
 
@@ skipping 26 matching lines @@
   bool StringifyOutput() override;
   void DistillPageImpl(const GURL& url, const std::string& script) override;
 
  private:
   friend class DistillerWebStateObserver;
 
   // Called by |web_state_observer_| once the page has finished loading.
   void OnLoadURLDone(web::PageLoadCompletionStatus load_completion_status);
 
   // Called once the |script_| has been evaluated on the page.
-  void HandleJavaScriptResultString(NSString* result);
+  void HandleJavaScriptResult(id result);
+
+  std::unique_ptr<base::Value> ValueResultFromWKResult(id wk_result);

[Eugene But (OOO till 7-30), 2016/09/09 15:28:28]

Optional NIT: Maybe s/WKResult/ScriptResult for consistency with previous method
name?

[Eugene But (OOO till 7-30), 2016/09/09 15:28:28]

Per Style Guide every non-trivial method needs comments.

[lody, 2016/09/19 15:34:35]

Done.

 
   web::BrowserState* browser_state_;
   GURL url_;
   std::string script_;
   std::unique_ptr<ios::WebControllerProvider> provider_;
   std::unique_ptr<DistillerWebStateObserver> web_state_observer_;
   base::WeakPtrFactory<DistillerPageIOS> weak_ptr_factory_;
 };
 
 }  // namespace dom_distiller
 
 #endif  // COMPONENTS_DOM_DISTILLER_IOS_DISTILLER_PAGE_IOS_H_