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

Side by Side Diff: Source/modules/speech/SpeechRecognitionResult.h

Issue 139803012: Move speech module over to Oilpan. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: More namespace tweaks for clang 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
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 5 * modification, are permitted provided that the following conditions
6 * are met: 6 * are met:
7 * * Redistributions of source code must retain the above copyright 7 * * Redistributions of source code must retain the above copyright
8 * notice, this list of conditions and the following disclaimer. 8 * notice, this list of conditions and the following disclaimer.
9 * * Redistributions in binary form must reproduce the above copyright 9 * * Redistributions in binary form must reproduce the above copyright
10 * notice, this list of conditions and the following disclaimer in the 10 * notice, this list of conditions and the following disclaimer in the
11 * documentation and/or other materials provided with the distribution. 11 * documentation and/or other materials provided with the distribution.
12 * 12 *
13 * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS ``AS IS'' AND ANY 13 * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS ``AS IS'' AND ANY
14 * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE 14 * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
15 * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR 15 * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
16 * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR 16 * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR
17 * CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, 17 * CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL,
18 * EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, 18 * EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
19 * PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR 19 * PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR
20 * PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY 20 * PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY
21 * OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT 21 * OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
22 * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE 22 * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
23 * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. 23 * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
24 */ 24 */
25 25
26 #ifndef SpeechRecognitionResult_h 26 #ifndef SpeechRecognitionResult_h
27 #define SpeechRecognitionResult_h 27 #define SpeechRecognitionResult_h
28 28
29 #include "bindings/v8/ScriptWrappable.h" 29 #include "bindings/v8/ScriptWrappable.h"
30 #include "heap/Handle.h"
30 #include "modules/speech/SpeechRecognitionAlternative.h" 31 #include "modules/speech/SpeechRecognitionAlternative.h"
31 #include "wtf/RefCounted.h" 32 #include "wtf/RefCounted.h"
32 #include "wtf/Vector.h" 33 #include "wtf/Vector.h"
33 34
34 namespace WebCore { 35 namespace WebCore {
35 36
36 class SpeechRecognitionResult : public ScriptWrappable, public RefCounted<Speech RecognitionResult> { 37 class SpeechRecognitionResult : public RefCountedWillBeRefCountedGarbageCollecte d<SpeechRecognitionResult>, public ScriptWrappable {
38 DECLARE_GC_INFO;
37 public: 39 public:
38 ~SpeechRecognitionResult(); 40 ~SpeechRecognitionResult();
39 static PassRefPtr<SpeechRecognitionResult> create(const Vector<RefPtr<Speech RecognitionAlternative> >&, bool final); 41 // FIXME (oilpan): the platform outer layer depends on holding a RefPtr to a result.
haraken 2014/02/12 05:18:45 FIXME: oilpan: Also move this FIXME to just above
haraken 2014/02/12 05:18:45 BTW, who retains a RefPtr to the SpeechRecognition
sof 2014/02/12 07:48:51 WebSpeechRecognitionResult holds a WebPrivatePtr t
sof 2014/02/12 07:48:51 Done + added a mention of WebSpeecRecognitionResul
haraken 2014/02/12 08:04:50 I see. Let's fix it in a follow-up CL. You can ref
42 // When/once that changes, the return type here can hopefully be adjusted.
43 static PassRefPtr<SpeechRecognitionResult> create(const Vector<RefPtrWillBeR awPtr<SpeechRecognitionAlternative> >&, bool final);
haraken 2014/02/12 05:18:45 This could be WillBeHeapVector<RefPtrWillBeMember<
sof 2014/02/12 10:17:09 I think it worked out (with side benefit of avoidi
40 44
41 unsigned long length() { return m_alternatives.size(); } 45 unsigned long length() { return m_alternatives.size(); }
42 SpeechRecognitionAlternative* item(unsigned long index); 46 SpeechRecognitionAlternative* item(unsigned long index);
43 bool isFinal() { return m_final; } 47 bool isFinal() { return m_final; }
44 48
49 void trace(Visitor*);
50
45 private: 51 private:
46 SpeechRecognitionResult(const Vector<RefPtr<SpeechRecognitionAlternative> >& , bool final); 52 SpeechRecognitionResult(const Vector<RefPtrWillBeRawPtr<SpeechRecognitionAlt ernative> >&, bool final);
haraken 2014/02/12 05:18:45 Ditto.
sof 2014/02/12 10:17:09 Done.
47 53
48 Vector<RefPtr<SpeechRecognitionAlternative> > m_alternatives;
49 bool m_final; 54 bool m_final;
55 Vector<RefPtrWillBeMember<SpeechRecognitionAlternative> > m_alternatives;
haraken 2014/02/12 05:18:45 This works, but we want to use WillBeHeapVector<Re
sof 2014/02/12 10:17:09 Done.
50 }; 56 };
51 57
52 } // namespace WebCore 58 } // namespace WebCore
53 59
54 #endif // SpeechRecognitionResult_h 60 #endif // SpeechRecognitionResult_h
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698