Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(365)

Unified Diff: components/dom_distiller/content/browser/distillable_page_utils_browsertest.cc

Issue 1231083007: Expose distiller functions to JavaScript (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@refactor-content
Patch Set: Function rename and remove unnecessary code Created 5 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/dom_distiller/content/browser/distillable_page_utils_browsertest.cc
diff --git a/components/dom_distiller/content/browser/distillable_page_utils_browsertest.cc b/components/dom_distiller/content/browser/distillable_page_utils_browsertest.cc
index 7723e67be4ba7562186e4a9ee7b123dc39a25629..5b885da53a02b06860e2dd3cd1ebe704999b1d9a 100644
--- a/components/dom_distiller/content/browser/distillable_page_utils_browsertest.cc
+++ b/components/dom_distiller/content/browser/distillable_page_utils_browsertest.cc
@@ -42,10 +42,8 @@ class DomDistillerDistillablePageUtilsTest : public content::ContentBrowserTest,
base::RunLoop url_loaded_runner;
main_frame_loaded_callback_ = url_loaded_runner.QuitClosure();
current_web_contents->GetController().LoadURL(
- embedded_test_server()->GetURL(url),
cjhopman 2015/07/29 00:26:51 Can we remove all the unrelated style/formatting c
mdjones 2015/07/29 18:36:00 Done.
- content::Referrer(),
- ui::PAGE_TRANSITION_TYPED,
- std::string());
+ embedded_test_server()->GetURL(url), content::Referrer(),
+ ui::PAGE_TRANSITION_TYPED, std::string());
url_loaded_runner.Run();
main_frame_loaded_callback_ = base::Closure();
Observe(nullptr);
@@ -93,9 +91,7 @@ class ResultHolder {
callback_.Run();
}
- bool GetResult() {
- return result_;
- }
+ bool GetResult() { return result_; }
base::Callback<void(bool)> GetCallback() {
return base::Bind(&ResultHolder::OnResult, base::Unretained(this));

Powered by Google App Engine
This is Rietveld 408576698