Lo-Fi snackbar should only be shown for the first q=low response of a page

components/data_reduction_proxy/content/browser/content_lofi_ui_service.h

Index: components/data_reduction_proxy/content/browser/content_lofi_ui_service.h
diff --git a/components/data_reduction_proxy/content/browser/content_lofi_ui_service.h b/components/data_reduction_proxy/content/browser/content_lofi_ui_service.h
new file mode 100644
index 0000000000000000000000000000000000000000..ec939edf0f022b310db17d955406cb5568720912
--- /dev/null
+++ b/components/data_reduction_proxy/content/browser/content_lofi_ui_service.h
@@ -0,0 +1,60 @@
+// Copyright 2015 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_DATA_REDUCTION_PROXY_CONTENT_BROWSER_CONTENT_LOFI_UI_SERVICE_H_
+#define COMPONENTS_DATA_REDUCTION_PROXY_CONTENT_BROWSER_CONTENT_LOFI_UI_SERVICE_H_
+
+#include "base/callback.h"
+#include "base/macros.h"
+#include "base/memory/ref_counted.h"
+#include "components/data_reduction_proxy/core/common/lofi_ui_service.h"
+
+namespace base {
+class SingleThreadTaskRunner;
+}
+
+namespace content {
+class WebContents;
+}
+
+namespace net {
+class URLRequest;
+}
+
+namespace data_reduction_proxy {
+
+// Passes notifications to the UI thread that a Lo-Fi response has been
+// received. These notifications are used to show Lo-Fi UI in Android.

[tbansal1, 2016/01/05 03:10:59]

s/These notifications are used to show Lo-Fi UI in Android./These notifications
may be used to show Lo-Fi UI./

because this class probably does not care about the OS-specific implementation?

[megjablon, 2016/01/05 20:13:03]

Done.

+class ContentLoFiUIService : public LoFiUIService {
+ public:
+  typedef base::Callback<void(content::WebContents* web_contents)>
+      NotifyLoFiResponseReceivedCallback;

[tbansal1, 2016/01/05 03:10:59]

Does this typedef need to be public?

[megjablon, 2016/01/05 20:13:03]

Done.

+
+  ContentLoFiUIService(
+      const scoped_refptr<base::SingleThreadTaskRunner>& ui_task_runner,
+      const NotifyLoFiResponseReceivedCallback&
+          notify_lofi_response_received_callback);
+  ~ContentLoFiUIService() override;
+
+  // Using the |render_process_id| and |render_frame_id|, gets the associated
+  // WebContents if it exists and runs the
+  // |notify_lofi_response_received_callback_|.
+  void NotifyLoFiResponseReceivedOnUI(int render_process_id,

[tbansal1, 2016/01/05 03:10:59]

Does this need to be public?

[megjablon, 2016/01/05 20:13:03]

Done.

+                                      int render_frame_id) const;
+
+  // LoFiUIService implementation:
+  void NotifyLoFiReponseReceived(const net::URLRequest& request) const override;
+
+ private:
+  // A task runner to post calls to NotifyLoFiReponseReceivedOnUI on the UI
+  // thread.
+  scoped_refptr<base::SingleThreadTaskRunner> ui_task_runner_;
+  NotifyLoFiResponseReceivedCallback notify_lofi_response_received_callback_;
+
+  DISALLOW_COPY_AND_ASSIGN(ContentLoFiUIService);
+};
+
+}  // namespace data_reduction_proxy
+
+#endif  // COMPONENTS_DATA_REDUCTION_PROXY_CONTENT_BROWSER_CONTENT_LOFI_UI_SERVICE_H_