Adding Application Data dialog for isolated apps

chrome/browser/ui/webui/options2/cookies_view_handler2.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_WEBUI_OPTIONS2_COOKIES_VIEW_HANDLER2_H_
 #define CHROME_BROWSER_UI_WEBUI_OPTIONS2_COOKIES_VIEW_HANDLER2_H_
 #pragma once
 
 #include "base/compiler_specific.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 21 matching lines @@
   virtual void TreeNodesRemoved(ui::TreeModel* model,
                                 ui::TreeModelNode* parent,
                                 int start,
                                 int count) OVERRIDE;
   virtual void TreeNodeChanged(ui::TreeModel* model,
                                ui::TreeModelNode* node) OVERRIDE {}
   virtual void TreeModelBeginBatch(CookiesTreeModel* model) OVERRIDE;
   virtual void TreeModelEndBatch(CookiesTreeModel* model) OVERRIDE;
 
  private:
+  enum CookiesViewCallback {
+    onTreeItemAdded,
+    onTreeItemRemoved,
+    loadChildren
+  };
+
   // Creates the CookiesTreeModel if neccessary.
   void EnsureCookiesTreeModelCreated();
 
   // Updates search filter for cookies tree model.
   void UpdateSearchResults(const base::ListValue* args);
 
   // Remove all sites data.
   void RemoveAll(const base::ListValue* args);
 
   // Remove selected sites data.
   void Remove(const base::ListValue* args);
 
   // Get the tree node using the tree path info in |args| and call
   // SendChildren to pass back children nodes data to WebUI.
   void LoadChildren(const base::ListValue* args);
 
   // Get children nodes data and pass it to 'CookiesView.loadChildren' to
   // update the WebUI.
   void SendChildren(CookieTreeNode* parent);
 
+  // Set the context in which this view is used - regular cookies window or
+  // the apps cookies window.
+  void SetViewContext(const base::ListValue* args);
+
+  // Return the proper callback string, depending on the context in which the

[Evan Stade, 2012/06/27 04:53:00]

"the context in which the view operates" is not a phrase that I can understand.

[nasko, 2012/06/27 17:17:01]

Done.

+  // view operates.
+  const std::string GetCallback(CookiesViewCallback callback);

[Evan Stade, 2012/06/27 04:53:00]

const without ref is meaningless for return types. remove the const.

[nasko, 2012/06/27 17:17:01]

Done.

+
+  // Return the proper tree model, depending on the context in which the
+  // view operates.
+  CookiesTreeModel* GetTreeModel();
+
   // The Cookies Tree model
   scoped_ptr<CookiesTreeModel> cookies_tree_model_;
+  scoped_ptr<CookiesTreeModel> app_cookies_tree_model_;
 
   // Flag to indicate whether there is a batch update in progress.
   bool batch_update_;
 
+  // Flag to indicate whether the view is used in the apps dialog context or
+  // the regular cookies context. This will cause different callbacks into
+  // JavaScript to be called.
+  bool app_context_;
+
   DISALLOW_COPY_AND_ASSIGN(CookiesViewHandler);
 };
 
 }  // namespace options2
 
 #endif  // CHROME_BROWSER_UI_WEBUI_OPTIONS2_COOKIES_VIEW_HANDLER2_H_