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

Side by Side Diff: Source/core/dom/ContextFeatures.cpp

Issue 181703004: Have Document::timing() / Document::contextFeatures() return a reference (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | Source/core/dom/Document.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2012 Google Inc. All rights reserved. 2 * Copyright (C) 2012 Google Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions are 5 * modification, are permitted provided that the following conditions are
6 * met: 6 * met:
7 * 7 *
8 * * Redistributions of source code must retain the above copyright 8 * * Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * * Neither the name of Google Inc. nor the names of its 10 * * Neither the name of Google Inc. nor the names of its
(...skipping 36 matching lines...) Expand 10 before | Expand all | Expand 10 after
47 ContextFeatures* ContextFeatures::defaultSwitch() 47 ContextFeatures* ContextFeatures::defaultSwitch()
48 { 48 {
49 DEFINE_STATIC_REF(ContextFeatures, instance, (ContextFeatures::create(Contex tFeaturesClient::empty()))); 49 DEFINE_STATIC_REF(ContextFeatures, instance, (ContextFeatures::create(Contex tFeaturesClient::empty())));
50 return instance; 50 return instance;
51 } 51 }
52 52
53 bool ContextFeatures::dialogElementEnabled(Document* document) 53 bool ContextFeatures::dialogElementEnabled(Document* document)
54 { 54 {
55 if (!document) 55 if (!document)
56 return RuntimeEnabledFeatures::dialogElementEnabled(); 56 return RuntimeEnabledFeatures::dialogElementEnabled();
57 return document->contextFeatures()->isEnabled(document, DialogElement, Runti meEnabledFeatures::dialogElementEnabled()); 57 return document->contextFeatures().isEnabled(document, DialogElement, Runtim eEnabledFeatures::dialogElementEnabled());
58 } 58 }
59 59
60 bool ContextFeatures::styleScopedEnabled(Document* document) 60 bool ContextFeatures::styleScopedEnabled(Document* document)
61 { 61 {
62 if (!document) 62 if (!document)
63 return RuntimeEnabledFeatures::styleScopedEnabled(); 63 return RuntimeEnabledFeatures::styleScopedEnabled();
64 return document->contextFeatures()->isEnabled(document, StyleScoped, Runtime EnabledFeatures::styleScopedEnabled()); 64 return document->contextFeatures().isEnabled(document, StyleScoped, RuntimeE nabledFeatures::styleScopedEnabled());
65 } 65 }
66 66
67 bool ContextFeatures::pagePopupEnabled(Document* document) 67 bool ContextFeatures::pagePopupEnabled(Document* document)
68 { 68 {
69 if (!document) 69 if (!document)
70 return false; 70 return false;
71 return document->contextFeatures()->isEnabled(document, PagePopup, false); 71 return document->contextFeatures().isEnabled(document, PagePopup, false);
72 } 72 }
73 73
74 bool ContextFeatures::mutationEventsEnabled(Document* document) 74 bool ContextFeatures::mutationEventsEnabled(Document* document)
75 { 75 {
76 ASSERT(document); 76 ASSERT(document);
77 if (!document) 77 if (!document)
78 return true; 78 return true;
79 return document->contextFeatures()->isEnabled(document, MutationEvents, true ); 79 return document->contextFeatures().isEnabled(document, MutationEvents, true) ;
80 } 80 }
81 81
82 bool ContextFeatures::pushStateEnabled(Document* document) 82 bool ContextFeatures::pushStateEnabled(Document* document)
83 { 83 {
84 return document->contextFeatures()->isEnabled(document, PushState, true); 84 return document->contextFeatures().isEnabled(document, PushState, true);
85 } 85 }
86 86
87 void provideContextFeaturesTo(Page& page, ContextFeaturesClient* client) 87 void provideContextFeaturesTo(Page& page, ContextFeaturesClient* client)
88 { 88 {
89 RefCountedSupplement<Page, ContextFeatures>::provideTo(page, ContextFeatures ::supplementName(), ContextFeatures::create(client)); 89 RefCountedSupplement<Page, ContextFeatures>::provideTo(page, ContextFeatures ::supplementName(), ContextFeatures::create(client));
90 } 90 }
91 91
92 void provideContextFeaturesToDocumentFrom(Document& document, Page& page) 92 void provideContextFeaturesToDocumentFrom(Document& document, Page& page)
93 { 93 {
94 ContextFeatures* provided = static_cast<ContextFeatures*>(RefCountedSuppleme nt<Page, ContextFeatures>::from(page, ContextFeatures::supplementName())); 94 ContextFeatures* provided = static_cast<ContextFeatures*>(RefCountedSuppleme nt<Page, ContextFeatures>::from(page, ContextFeatures::supplementName()));
95 if (!provided) 95 if (!provided)
96 return; 96 return;
97 document.setContextFeatures(provided); 97 document.setContextFeatures(*provided);
98 } 98 }
99 99
100 } 100 }
OLDNEW
« no previous file with comments | « no previous file | Source/core/dom/Document.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698