MD Settings/On Startup: ignore current tab when using current pages

chrome/browser/custom_home_pages_table_model.h

 // 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.
 
 #ifndef CHROME_BROWSER_CUSTOM_HOME_PAGES_TABLE_MODEL_H_
 #define CHROME_BROWSER_CUSTOM_HOME_PAGES_TABLE_MODEL_H_
 
 #include <string>
 #include <vector>
 
 #include "base/compiler_specific.h"
 #include "base/macros.h"
 #include "base/task/cancelable_task_tracker.h"
 #include "components/history/core/browser/history_types.h"
 #include "ui/base/models/table_model.h"
 
 class Browser;
 class GURL;
 class Profile;
 
+namespace content {
+class WebContents;
+}
+
 namespace history {
 class URLRow;
 }
 
 namespace ui {
 class TableModelObserver;
 }
 
 // CustomHomePagesTableModel is the model for the TableView showing the list
 // of pages the user wants opened on startup.
@@ skipping 11 matching lines @@
   // Expects |index_list| to be ordered ascending.
   void MoveURLs(int insert_before, const std::vector<int>& index_list);
 
   // Adds an entry at the specified index.
   void Add(int index, const GURL& url);
 
   // Removes the entry at the specified index.
   void Remove(int index);
 
   // Clears any entries and fills the list with pages currently opened in the
-  // browser.
-  void SetToCurrentlyOpenPages();
+  // browser. |source_contents| is omitted from the open pages.

[dschuyler, 2017/06/07 01:20:26]

Maybe: s/is/and Chrome dev tools are/

[Dan Beam, 2017/06/07 01:42:09]

ehhh, i don't want to update this every time we change things we don't consider
"currently opened in the browser".

+  void SetToCurrentlyOpenPages(const content::WebContents* source_contents);

[sky, 2017/06/07 16:08:48]

source_contents->ignore_contents ?

[Dan Beam, 2017/06/07 22:56:50]

Done.

 
   // Returns the set of urls this model contains.
   std::vector<GURL> GetURLs();
 
   // TableModel overrides:
   int RowCount() override;
   base::string16 GetText(int row, int column_id) override;
   base::string16 GetTooltip(int row) override;
   void SetObserver(ui::TableModelObserver* observer) override;
 
@@ skipping 50 matching lines @@
   base::CancelableTaskTracker task_tracker_;
 
   // Used to keep track of when it's time to update the observer when loading
   // multiple titles.
   int num_outstanding_title_lookups_;
 
   DISALLOW_COPY_AND_ASSIGN(CustomHomePagesTableModel);
 };
 
 #endif  // CHROME_BROWSER_CUSTOM_HOME_PAGES_TABLE_MODEL_H_