Share TabFirstRenderWatcher with HtmlDialogView.

chrome/browser/ui/webui/test_html_dialog_ui_delegate.h

Index: chrome/browser/ui/webui/test_html_dialog_ui_delegate.h
diff --git a/chrome/browser/ui/webui/test_html_dialog_ui_delegate.h b/chrome/browser/ui/webui/test_html_dialog_ui_delegate.h
new file mode 100644
index 0000000000000000000000000000000000000000..ddb51576eb22b8a1c41ba7e3d26e4c36ef30c4ff
--- /dev/null
+++ b/chrome/browser/ui/webui/test_html_dialog_ui_delegate.h
@@ -0,0 +1,42 @@
+// 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_TEST_HTML_DIALOG_UI_DELEGATE_H_
+#define CHROME_BROWSER_UI_WEBUI_TEST_HTML_DIALOG_UI_DELEGATE_H_
+#pragma once
+
+#include <string>
+
+#include "base/compiler_specific.h"
+#include "chrome/browser/ui/webui/html_dialog_ui.h"
+#include "ui/gfx/size.h"
+
+class TestHtmlDialogUIDelegate : public HtmlDialogUIDelegate {

[oshima, 2011/11/01 18:28:27]

put this in test namespace. (it's getting common pattern)

[xiyuan, 2011/11/01 20:18:15]

Done.

+ public:
+  explicit TestHtmlDialogUIDelegate(const GURL& url);
+  virtual ~TestHtmlDialogUIDelegate();
+
+  void set_size(int width, int height) {
+    size_.SetSize(width, height);
+  }
+
+  // HTMLDialogUIDelegate implementation:
+  virtual bool IsDialogModal() const OVERRIDE;
+  virtual string16 GetDialogTitle() const OVERRIDE;
+  virtual GURL GetDialogContentURL() const OVERRIDE;
+  virtual void GetWebUIMessageHandlers(
+      std::vector<WebUIMessageHandler*>* handlers) const OVERRIDE {}
+  virtual void GetDialogSize(gfx::Size* size) const OVERRIDE;
+  virtual std::string GetDialogArgs() const OVERRIDE;
+  virtual void OnDialogClosed(const std::string& json_retval) OVERRIDE {}
+  virtual void OnCloseContents(TabContents* source, bool* out_close_dialog)
+      OVERRIDE;
+  virtual bool ShouldShowDialogTitle() const OVERRIDE;
+
+ protected:
+  GURL url_;

[oshima, 2011/11/01 18:28:27]

const? (not sure)

[xiyuan, 2011/11/01 20:18:15]

Done. Think it should be const as we are not supposed to changing it on the fly.

+  gfx::Size size_;
+};
+
+#endif  // CHROME_BROWSER_UI_WEBUI_TEST_HTML_DIALOG_UI_DELEGATE_H_