First pass on intents options UI.

chrome/browser/ui/intents/intents_model.h

+// Copyright (c) 2011 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_UI_INTENTS_INTENTS_MODEL_H_
+#define CHROME_BROWSER_UI_INTENTS_INTENTS_MODEL_H_
+#pragma once
+
+#include "base/values.h"
+#include "chrome/browser/intents/web_intents_registry.h"
+#include "ui/base/models/tree_node_model.h"
+
+class WebIntentsRegistry;
+
+// The tree structure is a TYPE_ROOT node with title="",
+// children are TYPE_ORIGIN nodes with title=origin, whose
+// children are TYPE_SERVICE nodes with title=origin, and
+// will be of type ServiceTreeNode with data on individual
+// services.
+class IntentsTreeNode : public ui::TreeNode<IntentsTreeNode> {
+ public:
+  IntentsTreeNode()
+      : ui::TreeNode<IntentsTreeNode>(string16()),
+        type_(TYPE_ROOT) {}
+
+  explicit IntentsTreeNode(const string16& title)
+      : ui::TreeNode<IntentsTreeNode>(title),
+        type_(TYPE_ORIGIN) {}
+
+  virtual ~IntentsTreeNode() {}
+
+  enum NodeType {
+    TYPE_ROOT,
+    TYPE_ORIGIN,
+    TYPE_SERVICE,
+  };
+
+  NodeType Type() const { return type_; }
+
+ protected:
+  IntentsTreeNode(const string16& title, NodeType type)
+      : ui::TreeNode<IntentsTreeNode>(title),
+        type_(type) {}
+
+ private:
+  NodeType type_;
+};
+
+// Tree node representing particular services presented by an origin.
+class ServiceTreeNode : public IntentsTreeNode {

[James Hawkins, 2011/08/17 21:34:15]

Can this just be a struct? It seems it's only getters and setters.

[Greg Billock, 2011/08/17 22:44:02]

It needs to fit into the TreeNode<T> system, at least the way I have the model
going right now. Given my understanding of how we want the controls to work, I
think a tree model is a reasonable thing to use, but you're right that it ends
up kind of heavy here.

+ public:
+  explicit ServiceTreeNode(const string16& title)
+      : IntentsTreeNode(title, IntentsTreeNode::TYPE_SERVICE) {}
+  virtual ~ServiceTreeNode() {}
+
+  const string16& ServiceName() const { return service_name_; }
+  const string16& ServiceUrl() const { return service_url_; }
+  const string16& IconUrl() const { return icon_url_; }
+  const string16& Action() const { return action_; }
+  const base::ListValue& Types() const { return types_; }
+  bool IsBlocked() const { return blocked_; }
+  bool IsDisabled() const { return disabled_; }
+
+  void SetServiceName(string16 name) { service_name_ = name; }
+  void SetServiceUrl(string16 url) { service_url_ = url; }
+  void SetIconUrl(string16 url) { icon_url_ = url; }
+  void SetAction(string16 action) { action_ = action; }
+  void AddType(string16 type) { types_.Append(Value::CreateStringValue(type)); }
+  void SetBlocked(bool blocked) { blocked_ = blocked; }
+  void SetDisabled(bool disabled) { disabled_ = disabled; }
+
+ private:
+  string16 service_name_;
+  string16 icon_url_;
+  string16 service_url_;
+  string16 action_;
+  base::ListValue types_;
+
+  // TODO(gbillock): these are kind of a placeholder for exceptions data.
+  bool blocked_;
+  bool disabled_;
+};
+
+class IntentsModel : public ui::TreeNodeModel<IntentsTreeNode>,

[James Hawkins, 2011/08/17 21:34:15]

Document the class.

[Greg Billock, 2011/08/17 22:44:02]

Done.

+                     public WebIntentsRegistry::Consumer {
+ public:
+  // Because nodes are fetched in a background thread, they are not
+  // present at the time the Model is created. The Model then notifies its
+  // observers for every item added.
+  class Observer : public ui::TreeModelObserver {
+   public:
+    virtual void TreeModelBeginBatch(IntentsModel* model) {}
+    virtual void TreeModelEndBatch(IntentsModel* model) {}
+  };
+
+  explicit IntentsModel(WebIntentsRegistry* intents_registry);
+  virtual ~IntentsModel() {}
+
+  void AddIntentsTreeObserver(Observer* observer);
+  void RemoveIntentsTreeObserver(Observer* observer);
+
+  string16 GetTreeNodeId(IntentsTreeNode* node);
+  IntentsTreeNode* GetTreeNode(std::string path_id);
+  void GetChildNodeList(IntentsTreeNode* parent, int start, int count,
+                        base::ListValue* nodes);
+  void GetIntentsTreeNodeDictionary(const IntentsTreeNode& node,
+                                    base::DictionaryValue* dict);
+
+  virtual void OnIntentsQueryDone(
+      WebIntentsRegistry::QueryID query_id,
+      const std::vector<WebIntentData>& intents) OVERRIDE;
+
+ private:
+  void LoadModel();

[James Hawkins, 2011/08/17 21:34:15]

These are not self-documenting, please document.

[Greg Billock, 2011/08/17 22:44:02]

Done.

+  void NotifyObserverBeginBatch();
+  void NotifyObserverEndBatch();
+
+  WebIntentsRegistry* intents_registry_;

[James Hawkins, 2011/08/17 21:34:15]

Always document all vars. Note ownership.

[Greg Billock, 2011/08/17 22:44:02]

Done.

+  int batch_update_;
+  ObserverList<Observer> intents_observer_list_;
+};
+
+#endif  // CHROME_BROWSER_UI_INTENTS_INTENTS_MODEL_H_