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

chrome/browser/custom_home_pages_table_model.h

Index: chrome/browser/custom_home_pages_table_model.h
diff --git a/chrome/browser/custom_home_pages_table_model.h b/chrome/browser/custom_home_pages_table_model.h
index 9c649fa37e06dedfccdc01d1dc09621d254faac6..ebbb80c525f5c8672b71557e1cd067e61360689c 100644
--- a/chrome/browser/custom_home_pages_table_model.h
+++ b/chrome/browser/custom_home_pages_table_model.h
@@ -18,6 +18,10 @@ class Browser;
 class GURL;
 class Profile;
 
+namespace content {
+class WebContents;
+}
+
 namespace history {
 class URLRow;
 }
@@ -49,8 +53,8 @@ class CustomHomePagesTableModel : public ui::TableModel {
   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();