Added already_open column to chrome://suggestions-internals.

chrome/browser/ui/webui/ntp/suggestions_combiner.cc

 // Copyright (c) 2012 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/ui/webui/ntp/suggestions_combiner.h"
 
 #include <algorithm>
 
 #include "base/values.h"
 #include "chrome/browser/extensions/api/discovery/suggested_links_registry.h"
 #include "chrome/browser/extensions/api/discovery/suggested_links_registry_factory.h"
 #include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/ui/browser.h"
+#include "chrome/browser/ui/browser_list.h"
 #include "chrome/browser/ui/webui/ntp/suggestions_page_handler.h"
 #include "chrome/browser/ui/webui/ntp/suggestions_source.h"
 #include "chrome/browser/ui/webui/ntp/suggestions_source_discovery.h"
 #include "chrome/browser/ui/webui/ntp/suggestions_source_top_sites.h"
+#include "content/public/browser/web_contents.h"
 
 namespace {
 
 static const size_t kSuggestionsCount = 8;
 
 }  // namespace
 
 SuggestionsCombiner::SuggestionsCombiner(
-    SuggestionsCombiner::Delegate* delegate)
+    SuggestionsCombiner::Delegate* delegate,
+    Profile* profile)
     : sources_fetching_count_(0),
       delegate_(delegate),
       suggestions_count_(kSuggestionsCount),
       page_values_(new base::ListValue()),
-      debug_enabled_(false) {
+      debug_enabled_(false),
+      profile_(profile) {
 }
 
 SuggestionsCombiner::~SuggestionsCombiner() {
 }
 
 void SuggestionsCombiner::AddSource(SuggestionsSource* source) {
   source->SetCombiner(this);
   source->SetDebug(debug_enabled_);
   sources_.push_back(source);
 }
@@ skipping 25 matching lines @@
   }
 }
 
 void SuggestionsCombiner::SetSuggestionsCount(size_t suggestions_count) {
   suggestions_count_ = suggestions_count;
 }
 
 // static
 SuggestionsCombiner* SuggestionsCombiner::Create(
     SuggestionsCombiner::Delegate* delegate, Profile* profile) {
-  SuggestionsCombiner* combiner = new SuggestionsCombiner(delegate);
+  SuggestionsCombiner* combiner = new SuggestionsCombiner(delegate, profile);
   combiner->AddSource(new SuggestionsSourceTopSites());
 
   extensions::SuggestedLinksRegistry* registry =
       extensions::SuggestedLinksRegistryFactory::GetForProfile(profile);
   scoped_ptr<std::vector<std::string> > list = registry->GetExtensionIds();
   for (std::vector<std::string>::iterator it = list->begin();
       it != list->end(); ++it) {
     combiner->AddSource(new SuggestionsSourceDiscovery(*it));
   }
 
@@ skipping 34 matching lines @@
   for (size_t i = 0; i < sources_.size() &&
       page_values_->GetSize() < suggestions_count_; ++i) {
 
     size_t index = next_item_index_for_source[i] + extra_items_added;
     while (page_values_->GetSize() < suggestions_count_ &&
         (item = sources_[i]->PopItem())) {
       page_values_->Insert(index++, item);
       extra_items_added++;
     }
   }
+
+  // Add page value information common to all sources.
+  for (size_t i = 0; i < page_values_->GetSize(); i++) {
+    base::DictionaryValue* page_value;
+    if (page_values_->GetDictionary(i, &page_value))
+      AddExtendedInformation(page_value);
+  }
 }
+
+void SuggestionsCombiner::AddExtendedInformation(
+    base::DictionaryValue* page_value) {
+  if (debug_enabled_) {
+    std::string url_string;
+    if (page_value->GetString("url", &url_string)) {
+      GURL url(url_string);
+      page_value->SetBoolean("already_open", IsURLAlreadyOpen(url));
+    }
+  }
+}
+
+bool SuggestionsCombiner::IsURLAlreadyOpen(const GURL &url) {
+  for (BrowserList::const_iterator it = BrowserList::begin();
+       it != BrowserList::end(); ++it) {
+    const Browser* browser = *it;
+    if (browser->profile()->IsOffTheRecord() ||
+        !browser->profile()->IsSameProfile(profile_))
+      continue;

[Evan Stade, 2012/06/28 19:13:52]

\n

[Rune Fevang, 2012/06/28 21:37:53]

Done.

+    for (int i = 0; i < browser->tab_count(); i++) {
+      const content::WebContents* tab = browser->GetWebContentsAt(i);
+      if (tab->GetURL() == url)
+        return true;
+    }
+  }
+  return false;
+}