Make it possible to hide "most visited" on nnnnnnntp

chrome/browser/dom_ui/shown_sections_handler.cc

 // Copyright (c) 2010 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.
 
 #include "chrome/browser/dom_ui/shown_sections_handler.h"
 
 #include "base/callback.h"
 #include "base/command_line.h"
 #include "base/string_number_conversions.h"
 #include "base/values.h"
 #include "chrome/browser/metrics/user_metrics.h"
 #include "chrome/browser/prefs/pref_service.h"
 #include "chrome/browser/profile.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/extensions/extension.h"
 #include "chrome/common/notification_details.h"
 #include "chrome/common/notification_type.h"
 #include "chrome/common/pref_names.h"
 
 namespace {
 
 // Will cause an UMA notification if the mode of the new tab page
 // was changed to hide/show the most visited thumbnails.
+// TODO(aa): Needs to be updated to match newest NTP - http://crbug.com/57440
 void NotifySectionDisabled(int new_mode, int old_mode, Profile *profile) {
   // If the oldmode HAD either thumbs or lists visible.
-  bool old_had_it = old_mode & THUMB;
-  bool new_has_it = new_mode & THUMB;
+  bool old_had_it = (old_mode & THUMB) && !(old_mode & MINIMIZED_THUMB);
+  bool new_has_it = (new_mode & THUMB) && !(new_mode & MINIMIZED_THUMB);
 
   if (old_had_it && !new_has_it) {
     UserMetrics::RecordAction(
         UserMetricsAction("ShowSections_RecentSitesDisabled"),
         profile);
   }
 
   if (new_has_it && !old_had_it) {
     UserMetrics::RecordAction(
         UserMetricsAction("ShowSections_RecentSitesEnabled"),
@@ skipping 70 matching lines @@
       shown_sections = APPS;
     else
       shown_sections = THUMB;
 
     changed = true;
   }
 
   if (changed)
     pref_service->SetInteger(prefs::kNTPShownSections, shown_sections);
 }