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

Side by Side Diff: Source/core/dom/custom/CustomElementMicrotaskImportStep.h

Issue 288323004: HTML Imports: Get rid of needsProcessOrStop() from dom/custom/ (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 7 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
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2013 Google Inc. All rights reserved. 2 * Copyright (C) 2013 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 5 * modification, are permitted provided that the following conditions
6 * are met: 6 * are met:
7 * 7 *
8 * 1. Redistributions of source code must retain the above copyright 8 * 1. 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 * 2. Redistributions in binary form must reproduce the above copyright 10 * 2. Redistributions in binary form must reproduce the above copyright
(...skipping 41 matching lines...) Expand 10 before | Expand all | Expand 10 after
52 class CustomElementMicrotaskImportStep : public CustomElementMicrotaskStep { 52 class CustomElementMicrotaskImportStep : public CustomElementMicrotaskStep {
53 WTF_MAKE_NONCOPYABLE(CustomElementMicrotaskImportStep); 53 WTF_MAKE_NONCOPYABLE(CustomElementMicrotaskImportStep);
54 public: 54 public:
55 static PassOwnPtr<CustomElementMicrotaskImportStep> create(HTMLImportChild*) ; 55 static PassOwnPtr<CustomElementMicrotaskImportStep> create(HTMLImportChild*) ;
56 virtual ~CustomElementMicrotaskImportStep(); 56 virtual ~CustomElementMicrotaskImportStep();
57 57
58 // API for HTML Imports 58 // API for HTML Imports
59 void parentWasChanged(); 59 void parentWasChanged();
60 void importDidFinishLoading(); 60 void importDidFinishLoading();
61 WeakPtr<CustomElementMicrotaskImportStep> weakPtr() { return m_weakFactory.c reateWeakPtr(); } 61 WeakPtr<CustomElementMicrotaskImportStep> weakPtr() { return m_weakFactory.c reateWeakPtr(); }
62 bool isSync() const;
dominicc (has gone to gerrit) 2014/05/27 23:41:25 What about isForSyncImport? The step itself is bas
62 63
63 private: 64 private:
64 CustomElementMicrotaskImportStep(HTMLImportChild*); 65 CustomElementMicrotaskImportStep(HTMLImportChild*);
65 66
66 void didUpgradeAllCustomElements(); 67 void didUpgradeAllCustomElements();
67 bool shouldWaitForImport() const; 68 bool shouldWaitForImport() const;
68 bool shouldStopProcessing() const;
69 69
70 // CustomElementMicrotaskStep 70 // CustomElementMicrotaskStep
71 virtual Result process() OVERRIDE FINAL; 71 virtual Result process() OVERRIDE FINAL;
72 virtual bool needsProcessOrStop() const OVERRIDE FINAL;
73 72
74 #if !defined(NDEBUG) 73 #if !defined(NDEBUG)
75 virtual void show(unsigned indent) OVERRIDE; 74 virtual void show(unsigned indent) OVERRIDE;
76 #endif 75 #endif
77 WeakPtr<HTMLImportChild> m_import; 76 WeakPtr<HTMLImportChild> m_import;
78 RefPtr<CustomElementMicrotaskQueue> m_queue; 77 RefPtr<CustomElementMicrotaskQueue> m_queue;
79 WeakPtrFactory<CustomElementMicrotaskImportStep> m_weakFactory; 78 WeakPtrFactory<CustomElementMicrotaskImportStep> m_weakFactory;
80 }; 79 };
81 80
82 } 81 }
83 82
84 #endif // CustomElementMicrotaskImportStep_h 83 #endif // CustomElementMicrotaskImportStep_h
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698