Make classes and structures in core/loader fast-allocated.

Source/core/loader/EmptyClients.h

 /*
  * Copyright (C) 2006 Eric Seidel (eric@webkit.org)
  * Copyright (C) 2008, 2009, 2010, 2011, 2012 Apple Inc. All rights reserved.
  * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies).
  * Copyright (C) 2012 Samsung Electronics. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
@@ skipping 245 matching lines @@
     PassOwnPtr<WebServiceWorkerProvider> createServiceWorkerProvider() override;
     bool isControlledByServiceWorker(DocumentLoader&) override { return false; }
     int64_t serviceWorkerID(DocumentLoader&) override { return -1; }
     PassOwnPtr<WebApplicationCacheHost> createApplicationCacheHost(WebApplicationCacheHostClient*) override;
 
 protected:
     EmptyFrameLoaderClient() {}
 };
 
 class CORE_EXPORT EmptyTextCheckerClient : public TextCheckerClient {
+    DISALLOW_ALLOCATION();

[haraken, 2015/08/27 05:44:48]

Don't we want to add this to the parent class?

[tasak, 2015/08/27 07:50:48]

Looking at only EmptyTextCheckerClient, it sounds ok. However, looking at
another subclass of TextCheckerClient, i.e. SpellCheckerClientImpl:

  class  SpellCheckerClientImpl final : public SpellCheckerClient, public
TextCheckerClient { ... }

SpellCheckerClientImpl is a subclass of both SpellCheckerClient and
TextCheckerClient.
TextCheckerClient and SpellCheckerClient work in the almost the same way (i.e.
(just defining interfaces).

I'm not sure which class should be DISALLOW_ALLOCATION.
We need DISALLOW_ALLOCATION_MIXIN?

 public:
     ~EmptyTextCheckerClient() { }
 
     bool shouldEraseMarkersAfterChangeSelection(TextCheckingType) const override { return true; }
     void checkSpellingOfString(const String&, int*, int*) override {}
     String getAutoCorrectSuggestionForMisspelledWord(const String&) override { return String(); }
     void checkGrammarOfString(const String&, Vector<GrammarDetail>&, int*, int*) override {}
     void requestCheckingOfString(PassRefPtrWillBeRawPtr<TextCheckingRequest>) override;
 };
 
@@ skipping 48 matching lines @@
     ~EmptyDragClient() override {}
     DragDestinationAction actionMaskForDrag(DragData*) override { return DragDestinationActionNone; }
     void startDrag(DragImage*, const IntPoint&, const IntPoint&, DataTransfer*, LocalFrame*, bool) override {}
 };
 
 CORE_EXPORT void fillWithEmptyClients(Page::PageClients&);
 
 } // namespace blink
 
 #endif // EmptyClients_h