[Ozone-Dri] Create GbmWrapper

ui/ozone/platform/dri/gbm_wrapper.h

+// Copyright 2015 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 UI_OZONE_PLATFORM_DRI_GBM_WRAPPER_H_
+#define UI_OZONE_PLATFORM_DRI_GBM_WRAPPER_H_
+
+#include "ui/ozone/platform/dri/dri_wrapper.h"
+
+struct gbm_device;
+
+namespace ui {
+
+class GbmWrapper : public DriWrapper {
+ public:
+  GbmWrapper(const char* device_path, bool use_sync_flips);
+  ~GbmWrapper() override;
+
+  gbm_device* device() const { return device_; }
+
+  // DriWrapper overrides:

[alexst (slow to review), 2015/01/21 22:54:52]

The convention elsewhere is

// DriWrapper implementation:

being pedantic, sorry :)

[dnicoara, 2015/01/21 23:05:44]

Done.

$ git gs "implementation:" | wc -l
1176
$ git gs "overrides:" | wc -l
1074

Interestingly the view is split on this one. Of course the the grep does not
include all the broken variations.

[varkha, 2015/03/16 20:45:07]

I think a more popular variant now (I got into similar discussion elsewhere
recently) is just " // <Interface>:" which is a bit shorter.
within src/chrome (about a week ago stats):
82 of // FooBar
648 of // FooBar:
346 of // FooBar [Oo]verride*

+  void Initialize() override;
+
+ private:
+  gbm_device* device_;
+
+  DISALLOW_COPY_AND_ASSIGN(GbmWrapper);
+};
+
+}  // namespace ui
+
+#endif  // UI_OZONE_PLATFORM_DRI_GBM_WRAPPER_H_