|
ABANDONED CL (see CR comments for more details and discussion)
----------------------------
Rename readyState() to getReadyState().
The rename is needed to avoid a naming collision after changing from
Blink to Chromium naming style. Right now we have a |ReadyState| enum
type and |readyState| accessor methods (differing by case of the first
character); after a naive rename by the rewrite_to_chrome_style tool we
would end up with |ReadyState| being the name of both the type and the
accessor methods (with both living in the same namespace).
Prepending a "get" prefix to the name of the accessor method is the
workaround that fits into the guidance on the recommended
post-Blink-to-Chromium-rename style suggested by esprehn@ in
https://crbug.com/582312#c17:
- Getters favor not using "Get", ex. FirstChild()
- Unless the type name conflicts, in which case you can either rename
the type if it's easy and makes sense, or add "Get", ex. GetContext().
BUG= 582312
Total comments: 8
|
Unified diffs |
Side-by-side diffs |
Delta from patch set |
Stats (+89 lines, -89 lines) |
Patch |
|
M |
third_party/WebKit/Source/core/dom/Document.h
|
View
|
|
1 chunk |
+1 line, -1 line |
1 comment
|
Download
|
|
M |
third_party/WebKit/Source/core/dom/Document.cpp
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/core/dom/Document.idl
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/core/inspector/InspectorTraceEvents.cpp
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/core/xmlhttprequest/XMLHttpRequest.h
|
View
|
|
1 chunk |
+1 line, -1 line |
1 comment
|
Download
|
|
M |
third_party/WebKit/Source/core/xmlhttprequest/XMLHttpRequest.cpp
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/core/xmlhttprequest/XMLHttpRequest.idl
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/core/xmlhttprequest/XMLHttpRequestProgressEventThrottle.cpp
|
View
|
|
4 chunks |
+5 lines, -5 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/modules/eventsource/EventSource.h
|
View
|
|
1 chunk |
+1 line, -1 line |
1 comment
|
Download
|
|
M |
third_party/WebKit/Source/modules/eventsource/EventSource.cpp
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/modules/eventsource/EventSource.idl
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/modules/imagecapture/ImageCapture.cpp
|
View
|
|
1 chunk |
+2 lines, -2 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/modules/indexeddb/IDBRequest.h
|
View
|
|
1 chunk |
+1 line, -1 line |
1 comment
|
Download
|
|
M |
third_party/WebKit/Source/modules/indexeddb/IDBRequest.cpp
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/modules/indexeddb/IDBRequest.idl
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/modules/indexeddb/IDBRequestTest.cpp
|
View
|
|
4 chunks |
+4 lines, -4 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/modules/mediasource/MediaSource.h
|
View
|
|
1 chunk |
+1 line, -1 line |
1 comment
|
Download
|
|
M |
third_party/WebKit/Source/modules/mediasource/MediaSource.cpp
|
View
|
|
4 chunks |
+4 lines, -4 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/modules/mediasource/MediaSource.idl
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/modules/mediastream/MediaStreamTrack.h
|
View
|
|
1 chunk |
+1 line, -1 line |
1 comment
|
Download
|
|
M |
third_party/WebKit/Source/modules/mediastream/MediaStreamTrack.cpp
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/modules/mediastream/MediaStreamTrack.idl
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/modules/peerconnection/RTCDataChannel.h
|
View
|
|
1 chunk |
+1 line, -1 line |
1 comment
|
Download
|
|
M |
third_party/WebKit/Source/modules/peerconnection/RTCDataChannel.cpp
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/modules/peerconnection/RTCDataChannel.idl
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/modules/websockets/DOMWebSocket.h
|
View
|
|
1 chunk |
+1 line, -1 line |
1 comment
|
Download
|
|
M |
third_party/WebKit/Source/modules/websockets/DOMWebSocket.cpp
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/modules/websockets/DOMWebSocketTest.cpp
|
View
|
|
32 chunks |
+50 lines, -50 lines |
0 comments
|
Download
|
|
M |
third_party/WebKit/Source/modules/websockets/WebSocket.idl
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
Total messages: 13 (7 generated)
|