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

Unified Diff: base/win/scoped_hdc.h

Issue 8139022: Rename ScopedHDC to ScopedCreateDC (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Update a comment of ScopedCreateDC Created 9 years, 2 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
« no previous file with comments | « no previous file | chrome/browser/aeropeek_manager.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: base/win/scoped_hdc.h
diff --git a/base/win/scoped_hdc.h b/base/win/scoped_hdc.h
index 8b316caa08d9c3339bef7bff32b19471e741d348..9e2ea623b6cc8837fb6cd1fd24dbf155e50ed684 100644
--- a/base/win/scoped_hdc.h
+++ b/base/win/scoped_hdc.h
@@ -35,19 +35,18 @@ class ScopedGetDC {
private:
HWND hwnd_;
HDC hdc_;
+
DISALLOW_COPY_AND_ASSIGN(ScopedGetDC);
};
// Like ScopedHandle but for HDC. Only use this on HDCs returned from
-// CreateCompatibleDC.
-// TODO(yosin) To eliminate confusion with ScopedGetDC, we should rename
-// ScopedHDC to ScopedCreateDC.
-class ScopedHDC {
+// CreateCompatibleDC, CreateDC and CreateIC.
+class ScopedCreateDC {
public:
- ScopedHDC() : hdc_(NULL) { }
- explicit ScopedHDC(HDC h) : hdc_(h) { }
+ ScopedCreateDC() : hdc_(NULL) { }
+ explicit ScopedCreateDC(HDC h) : hdc_(h) { }
- ~ScopedHDC() {
+ ~ScopedCreateDC() {
Close();
}
@@ -73,7 +72,8 @@ class ScopedHDC {
}
HDC hdc_;
- DISALLOW_COPY_AND_ASSIGN(ScopedHDC);
+
+ DISALLOW_COPY_AND_ASSIGN(ScopedCreateDC);
};
} // namespace win
« no previous file with comments | « no previous file | chrome/browser/aeropeek_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698