[webkit-changes] [218745] trunk/Source/WebKit2

Fri, 23 Jun 2017 05:08:27 -0700

Title: [218745] trunk/Source/WebKit2
Revision
218745
Author
[email protected]
Date
2017-06-23 05:08:00 -0700 (Fri, 23 Jun 2017)

Log Message

[CoordinatedGraphics] Clean up type aliases in CoordinatedGraphicsScene
https://bugs.webkit.org/show_bug.cgi?id=173764

Reviewed by Carlos Garcia Campos.

Drop the various type aliases created in the CoordinatedGraphicsScene
class. We can leave without these by leveraging `auto` where necessary.

* Shared/CoordinatedGraphics/CoordinatedGraphicsScene.cpp:
(WebKit::CoordinatedGraphicsScene::updateTilesIfNeeded):
(WebKit::CoordinatedGraphicsScene::updateImageBacking):
(WebKit::CoordinatedGraphicsScene::clearImageBackingContents):
(WebKit::CoordinatedGraphicsScene::assignImageBackingToLayer):
* Shared/CoordinatedGraphics/CoordinatedGraphicsScene.h:

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (218744 => 218745)


--- trunk/Source/WebKit2/ChangeLog	2017-06-23 12:03:11 UTC (rev 218744)
+++ trunk/Source/WebKit2/ChangeLog	2017-06-23 12:08:00 UTC (rev 218745)
@@ -1,3 +1,20 @@
+2017-06-23  Zan Dobersek  <[email protected]>
+
+        [CoordinatedGraphics] Clean up type aliases in CoordinatedGraphicsScene
+        https://bugs.webkit.org/show_bug.cgi?id=173764
+
+        Reviewed by Carlos Garcia Campos.
+
+        Drop the various type aliases created in the CoordinatedGraphicsScene
+        class. We can leave without these by leveraging `auto` where necessary.
+
+        * Shared/CoordinatedGraphics/CoordinatedGraphicsScene.cpp:
+        (WebKit::CoordinatedGraphicsScene::updateTilesIfNeeded):
+        (WebKit::CoordinatedGraphicsScene::updateImageBacking):
+        (WebKit::CoordinatedGraphicsScene::clearImageBackingContents):
+        (WebKit::CoordinatedGraphicsScene::assignImageBackingToLayer):
+        * Shared/CoordinatedGraphics/CoordinatedGraphicsScene.h:
+
 2017-06-23  Carlos Garcia Campos  <[email protected]>
 
         [WPE] Crash in wpe_view_backend_get_renderer_host_fd

Modified: trunk/Source/WebKit2/Shared/CoordinatedGraphics/CoordinatedGraphicsScene.cpp (218744 => 218745)


--- trunk/Source/WebKit2/Shared/CoordinatedGraphics/CoordinatedGraphicsScene.cpp	2017-06-23 12:03:11 UTC (rev 218744)
+++ trunk/Source/WebKit2/Shared/CoordinatedGraphics/CoordinatedGraphicsScene.cpp	2017-06-23 12:08:00 UTC (rev 218745)
@@ -421,7 +421,7 @@
     for (auto& tile : state.tilesToUpdate) {
         const SurfaceUpdateInfo& surfaceUpdateInfo = tile.updateInfo;
 
-        SurfaceMap::iterator surfaceIt = m_surfaces.find(surfaceUpdateInfo.atlasID);
+        auto surfaceIt = m_surfaces.find(surfaceUpdateInfo.atlasID);
         ASSERT(surfaceIt != m_surfaces.end());
 
         backingStore->updateTile(tile.tileID, surfaceUpdateInfo.updateRect, tile.tileRect, surfaceIt->value.copyRef(), surfaceUpdateInfo.surfaceOffset);
@@ -477,7 +477,7 @@
 void CoordinatedGraphicsScene::updateImageBacking(CoordinatedImageBackingID imageID, RefPtr<CoordinatedSurface>&& surface)
 {
     ASSERT(m_imageBackings.contains(imageID));
-    ImageBackingMap::iterator it = m_imageBackings.find(imageID);
+    auto it = m_imageBackings.find(imageID);
     RefPtr<CoordinatedBackingStore> backingStore = it->value;
 
     // CoordinatedImageBacking is realized to CoordinatedBackingStore with only one tile in UI Process.
@@ -494,7 +494,7 @@
 void CoordinatedGraphicsScene::clearImageBackingContents(CoordinatedImageBackingID imageID)
 {
     ASSERT(m_imageBackings.contains(imageID));
-    ImageBackingMap::iterator it = m_imageBackings.find(imageID);
+    auto it = m_imageBackings.find(imageID);
     RefPtr<CoordinatedBackingStore> backingStore = it->value;
     backingStore->removeAllTiles();
     m_backingStoresWithPendingBuffers.add(backingStore);
@@ -515,7 +515,7 @@
         return;
     }
 
-    ImageBackingMap::iterator it = m_imageBackings.find(imageID);
+    auto it = m_imageBackings.find(imageID);
     ASSERT(it != m_imageBackings.end());
     layer->setContentsLayer(it->value.get());
 }

Modified: trunk/Source/WebKit2/Shared/CoordinatedGraphics/CoordinatedGraphicsScene.h (218744 => 218745)


--- trunk/Source/WebKit2/Shared/CoordinatedGraphics/CoordinatedGraphicsScene.h	2017-06-23 12:03:11 UTC (rev 218744)
+++ trunk/Source/WebKit2/Shared/CoordinatedGraphics/CoordinatedGraphicsScene.h	2017-06-23 12:08:00 UTC (rev 218745)
@@ -155,22 +155,17 @@
 
     std::unique_ptr<WebCore::TextureMapper> m_textureMapper;
 
-    typedef HashMap<WebCore::CoordinatedImageBackingID, RefPtr<CoordinatedBackingStore>> ImageBackingMap;
-    ImageBackingMap m_imageBackings;
+    HashMap<WebCore::CoordinatedImageBackingID, RefPtr<CoordinatedBackingStore>> m_imageBackings;
     Vector<RefPtr<CoordinatedBackingStore>> m_releasedImageBackings;
 
-    typedef HashMap<WebCore::TextureMapperLayer*, RefPtr<CoordinatedBackingStore>> BackingStoreMap;
-    BackingStoreMap m_backingStores;
-
+    HashMap<WebCore::TextureMapperLayer*, RefPtr<CoordinatedBackingStore>> m_backingStores;
     HashSet<RefPtr<CoordinatedBackingStore>> m_backingStoresWithPendingBuffers;
 
 #if USE(COORDINATED_GRAPHICS_THREADED)
-    typedef HashMap<WebCore::TextureMapperLayer*, RefPtr<WebCore::TextureMapperPlatformLayerProxy>> PlatformLayerProxyMap;
-    PlatformLayerProxyMap m_platformLayerProxies;
+    HashMap<WebCore::TextureMapperLayer*, RefPtr<WebCore::TextureMapperPlatformLayerProxy>> m_platformLayerProxies;
 #endif
 
-    typedef HashMap<uint32_t /* atlasID */, RefPtr<WebCore::CoordinatedSurface>> SurfaceMap;
-    SurfaceMap m_surfaces;
+    HashMap<uint32_t /* atlasID */, RefPtr<WebCore::CoordinatedSurface>> m_surfaces;
 
     // Below two members are accessed by only the main thread. The painting thread must lock the main thread to access both members.
     CoordinatedGraphicsSceneClient* m_client;
@@ -178,10 +173,8 @@
 
     std::unique_ptr<WebCore::TextureMapperLayer> m_rootLayer;
 
-    typedef HashMap<WebCore::CoordinatedLayerID, std::unique_ptr<WebCore::TextureMapperLayer>> LayerMap;
-    LayerMap m_layers;
-    typedef HashMap<WebCore::CoordinatedLayerID, WebCore::TextureMapperLayer*> LayerRawPtrMap;
-    LayerRawPtrMap m_fixedLayers;
+    HashMap<WebCore::CoordinatedLayerID, std::unique_ptr<WebCore::TextureMapperLayer>> m_layers;
+    HashMap<WebCore::CoordinatedLayerID, WebCore::TextureMapperLayer*> m_fixedLayers;
     WebCore::CoordinatedLayerID m_rootLayerID;
     WebCore::FloatPoint m_scrollPosition;
     WebCore::FloatPoint m_renderedContentsScrollPosition;

_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to