Improve syncing between dom distiller and the print preview request

components/dom_distiller/core/dom_distiller_service.cc

Index: components/dom_distiller/core/dom_distiller_service.cc
diff --git a/components/dom_distiller/core/dom_distiller_service.cc b/components/dom_distiller/core/dom_distiller_service.cc
index 47d4f06d62986afcde2b3d376989f3a1380f3fc8..d7f12d467c9f9ae70efd7491a94b1cac9064aede 100644
--- a/components/dom_distiller/core/dom_distiller_service.cc
+++ b/components/dom_distiller/core/dom_distiller_service.cc
@@ -189,6 +189,17 @@ scoped_ptr<ViewerHandle> DomDistillerService::ViewUrl(
   return viewer_handle;
 }
 
+scoped_ptr<ViewerHandle> DomDistillerService::AddViewRequestDelegate(
+    ViewRequestDelegate* delegate,
+    const GURL& url) {
+  TaskTracker* task_tracker = GetTaskTrackerForUrl(url);
+  if (task_tracker) {
+    return task_tracker->AddViewer(delegate);
+  } else {

[nyquist, 2016/02/11 15:57:53]

Nit: There's a return-statement in the if-block, so you can remove 'else' and
then just unindent. Probably with a newline for readability. if-blocks  with a
single line doesn't need curly braces either in fact, so something like:

if (task_tracker)
  return task_tracker->AddViewer(delegate);

return nullptr;

[mvendramini_hp, 2016/02/11 19:00:46]

Done.

+    return nullptr;
+  }
+}
+
 bool DomDistillerService::GetOrCreateTaskTrackerForUrl(
     const GURL& url,
     TaskTracker** task_tracker) {