[MD settings] pulling in chrome.send for using current pages for on startup pages]

chrome/browser/ui/webui/settings/startup_pages_handler.h

+// Copyright (c) 2012-2015 The Chromium Authors. All rights reserved.

[Dan Beam, 2015/09/02 07:31:23]

only 2015, no (c)

[dschuyler, 2015/09/02 19:38:21]

Done.

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_UI_WEBUI_SETTINGS_STARTUP_PAGES_HANDLER_H_
+#define CHROME_BROWSER_UI_WEBUI_SETTINGS_STARTUP_PAGES_HANDLER_H_
+
+#include "base/memory/scoped_ptr.h"
+#include "chrome/browser/ui/webui/settings/md_settings_ui.h"
+#include "ui/base/models/table_model_observer.h"
+
+class CustomHomePagesTableModel;
+
+namespace settings {
+
+// Chrome browser settings page UI handler.
+class StartupPagesHandler : public SettingsPageUIHandler,
+                            public ui::TableModelObserver {
+ public:
+  explicit StartupPagesHandler(content::WebUI* webui);

[Dan Beam, 2015/09/02 07:31:23]

where does content::WebUI come from?

[dschuyler, 2015/09/02 19:38:21]

I'm not sure if you're asking for the file it's declared in, defined in, whether
there should be a forward declaration or where the caller passes it into the
ctor.  It's declared in content/public/browser/web_ui.h.
Also, I've added a forward declaration.

+  ~StartupPagesHandler() override;
+
+  // OptionsPageUIHandler implementation.
+  void RegisterMessages() override;
+
+  // ui::TableModelObserver implementation.

[Dan Beam, 2015/09/02 07:31:23]

just

  // ui::TableModelObserver:

the " implementation." is just fat and redundant to me

[dschuyler, 2015/09/02 19:38:21]

Done.

+  void OnModelChanged() override;
+  void OnItemsChanged(int start, int length) override;
+  void OnItemsAdded(int start, int length) override;
+  void OnItemsRemoved(int start, int length) override;
+
+ private:
+  // Sets the startup page set to the current pages. Called from WebUI.
+  void SetStartupPagesToCurrentPages(const base::ListValue* args);

[Dan Beam, 2015/09/02 07:31:23]

forward declare base::ListValue

[dschuyler, 2015/09/02 19:38:21]

Done.

+
+  scoped_ptr<CustomHomePagesTableModel> startup_custom_pages_table_model_;
+
+  DISALLOW_COPY_AND_ASSIGN(StartupPagesHandler);

[Dan Beam, 2015/09/02 07:31:23]

#include "base/macros.h"

[dschuyler, 2015/09/02 19:38:21]

Done.

+};
+
+}  // namespace settings
+
+#endif  // CHROME_BROWSER_UI_WEBUI_SETTINGS_STARTUP_PAGES_HANDLER_H_