Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1227)

Unified Diff: content/browser/renderer_host/render_widget_host_unittest.cc

Issue 12210030: Linux/ChromeOS Chromium style checker cleanup, content/ edition. (Closed) Base URL: http://src.chromium.org/svn/trunk/src/
Patch Set: Created 7 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/renderer_host/render_widget_host_unittest.cc
===================================================================
--- content/browser/renderer_host/render_widget_host_unittest.cc (revision 181789)
+++ content/browser/renderer_host/render_widget_host_unittest.cc (working copy)
@@ -272,7 +272,7 @@
update_msg_should_reply_(false),
update_msg_reply_flags_(0) {
}
- ~RenderWidgetHostProcess() {
+ virtual ~RenderWidgetHostProcess() {
delete current_update_buf_;
}
@@ -286,12 +286,12 @@
// Fills the given update parameters with resonable default values.
void InitUpdateRectParams(ViewHostMsg_UpdateRect_Params* params);
- virtual bool HasConnection() const { return true; }
+ virtual bool HasConnection() const OVERRIDE { return true; }
protected:
virtual bool WaitForBackingStoreMsg(int render_widget_id,
const base::TimeDelta& max_delay,
- IPC::Message* msg);
+ IPC::Message* msg) OVERRIDE;
TransportDIB* current_update_buf_;
@@ -451,9 +451,9 @@
size_ = size;
}
- void Observe(int type,
- const NotificationSource& source,
- const NotificationDetails& details) {
+ virtual void Observe(int type,
+ const NotificationSource& source,
+ const NotificationDetails& details) OVERRIDE {
if (type == NOTIFICATION_RENDER_WIDGET_HOST_DID_RECEIVE_PAINT_AT_SIZE_ACK) {
std::pair<int, gfx::Size>* size_ack_details =
Details<std::pair<int, gfx::Size> >(details).ptr();
@@ -481,12 +481,12 @@
public:
RenderWidgetHostTest() : process_(NULL) {
}
- ~RenderWidgetHostTest() {
+ virtual ~RenderWidgetHostTest() {
}
protected:
// testing::Test
- void SetUp() {
+ virtual void SetUp() {
browser_context_.reset(new TestBrowserContext());
delegate_.reset(new MockRenderWidgetHostDelegate());
process_ = new RenderWidgetHostProcess(browser_context_.get());
@@ -500,7 +500,7 @@
host_->SetView(view_.get());
host_->Init();
}
- void TearDown() {
+ virtual void TearDown() {
view_.reset();
host_.reset();
delegate_.reset();

Powered by Google App Engine
This is Rietveld 408576698