DataUseTabModel should check the URL to differentiate history navigation

chrome/browser/android/data_usage/data_use_ui_tab_model.h

 // 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 CHROME_BROWSER_ANDROID_DATA_USAGE_DATA_USE_UI_TAB_MODEL_H_
 #define CHROME_BROWSER_ANDROID_DATA_USAGE_DATA_USE_UI_TAB_MODEL_H_
 
 #include <string>
 
 #include "base/containers/hash_tables.h"
@@ skipping 106 matching lines @@
   // Creates |event| for tab with id |tab_id| and value |event|, if there is no
   // existing entry for |tab_id|, and returns true. Otherwise, returns false
   // without modifying the entry.
   bool MaybeCreateTabEvent(SessionID::id_type tab_id,
                            DataUseTrackingEvent event);
 
   // Removes event entry for |tab_id|, if the entry is equal to |event|, and
   // returns true. Otherwise, returns false without modifying the entry.
   bool RemoveTabEvent(SessionID::id_type tab_id, DataUseTrackingEvent event);
 
-  // Converts |page_transition| to DataUseTabModel::TransitionType enum.
-  // Returns true if conversion was successful, and updates |transition_type|.
-  // Otherwise, returns false, and |transition_type| is not changed.
-  // |transition_type| must not be null.
+  // Converts transition |page_transition| for page url |gurl| to

[tbansal1, 2016/03/17 23:16:53]

s/Converts transition |page_transition| for page url |gurl| to/
Converts |page_transition| to page with GURL |gurl| to/

[Raj, 2016/03/18 00:45:49]

Done.

+  // DataUseTabModel::TransitionType enum. Returns true if conversion was
+  // successful, and updates |transition_type|. Otherwise, returns false, and
+  // |transition_type| is not changed. |transition_type| must not be null.
   bool ConvertTransitionType(
       ui::PageTransition page_transition,
+      const GURL& gurl,
       DataUseTabModel::TransitionType* transition_type) const;
 
   // |tab_events_| stores tracking events of multiple tabs.
   TabEvents tab_events_;
 
   // |data_use_tab_model_| is notified by |this| about browser navigations
   // and tab closures on UI thread.
   base::WeakPtr<DataUseTabModel> data_use_tab_model_;
 
   base::ThreadChecker thread_checker_;
 
   base::WeakPtrFactory<DataUseUITabModel> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(DataUseUITabModel);
 };
 
 }  // namespace android
 
 }  // namespace chrome
 
 #endif  // CHROME_BROWSER_ANDROID_DATA_USAGE_DATA_USE_UI_TAB_MODEL_H_