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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 454103003: Cleanup namespace usage in public/web/Web[N-Z]*.h. (Closed)

Created:
6 years, 4 months ago by tkent
Modified:
6 years, 4 months ago
Reviewers:
keishi, abarth-chromium
CC:
abarth-chromium, blink-reviews, dglazkov+blink, eric.carlson_apple.com, feature-media-reviews_chromium.org, gavinp+prerender_chromium.org, jamesr, philipj_slow
Project:
blink
Visibility:
Public.

Description

Cleanup namespace usage in public/web/Web[N-Z]*.h. - Merge multiple |namespace blink {}| blocks - Remove unnecessary blink:: prefixes - Make blank lines consistent BUG= Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=179893

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+102 lines, -138 lines) Patch
M public/web/WebNode.h View 3 chunks +6 lines, -6 lines 0 comments Download
M public/web/WebNodeList.h View 2 chunks +5 lines, -4 lines 0 comments Download
M public/web/WebNotification.h View 2 chunks +4 lines, -5 lines 0 comments Download
M public/web/WebOptionElement.h View 2 chunks +5 lines, -7 lines 0 comments Download
M public/web/WebPageSerializer.h View 1 chunk +1 line, -0 lines 0 comments Download
M public/web/WebPageSerializerClient.h View 1 chunk +1 line, -0 lines 0 comments Download
M public/web/WebPerformance.h View 2 chunks +5 lines, -5 lines 0 comments Download
M public/web/WebPluginDocument.h View 2 chunks +5 lines, -7 lines 0 comments Download
M public/web/WebPrerendererClient.h View 1 chunk +1 line, -1 line 0 comments Download
M public/web/WebRange.h View 2 chunks +4 lines, -5 lines 0 comments Download
M public/web/WebScopedUserGesture.h View 2 chunks +2 lines, -5 lines 0 comments Download
M public/web/WebScopedWindowFocusAllowedIndicator.h View 2 chunks +2 lines, -4 lines 0 comments Download
M public/web/WebSearchableFormData.h View 1 chunk +1 line, -0 lines 0 comments Download
M public/web/WebSecurityOrigin.h View 2 chunks +5 lines, -5 lines 0 comments Download
M public/web/WebSelectElement.h View 2 chunks +5 lines, -7 lines 0 comments Download
M public/web/WebSerializedScriptValue.h View 2 chunks +6 lines, -6 lines 0 comments Download
M public/web/WebServiceWorkerContextClient.h View 2 chunks +3 lines, -3 lines 0 comments Download
M public/web/WebServiceWorkerNetworkProvider.h View 1 chunk +1 line, -1 line 0 comments Download
M public/web/WebSharedWorker.h View 1 chunk +0 lines, -4 lines 0 comments Download
M public/web/WebSocket.h View 1 chunk +0 lines, -2 lines 0 comments Download
M public/web/WebSpeechGrammar.h View 2 chunks +4 lines, -6 lines 0 comments Download
M public/web/WebSpeechRecognitionHandle.h View 2 chunks +5 lines, -8 lines 0 comments Download
M public/web/WebSpeechRecognitionResult.h View 2 chunks +3 lines, -5 lines 0 comments Download
M public/web/WebStorageEventDispatcher.h View 1 chunk +1 line, -1 line 0 comments Download
M public/web/WebSurroundingText.h View 2 chunks +2 lines, -5 lines 0 comments Download
M public/web/WebTextAreaElement.h View 2 chunks +5 lines, -7 lines 0 comments Download
M public/web/WebTextCheckingCompletion.h View 1 chunk +1 line, -0 lines 0 comments Download
M public/web/WebTextCheckingResult.h View 2 chunks +2 lines, -4 lines 0 comments Download
M public/web/WebTextRun.h View 2 chunks +3 lines, -5 lines 0 comments Download
M public/web/WebUserGestureToken.h View 2 chunks +4 lines, -6 lines 0 comments Download
M public/web/WebUserMediaRequest.h View 2 chunks +4 lines, -6 lines 0 comments Download
M public/web/WebWindowFeatures.h View 2 chunks +3 lines, -3 lines 0 comments Download
M public/web/mac/WebSubstringUtil.h View 2 chunks +3 lines, -5 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
tkent
Please review this.
6 years, 4 months ago (2014-08-10 08:00:00 UTC) #1
abarth-chromium
lgtm
6 years, 4 months ago (2014-08-10 15:40:33 UTC) #2
tkent
The CQ bit was checked by tkent@chromium.org
6 years, 4 months ago (2014-08-10 15:43:32 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/tkent@chromium.org/454103003/1
6 years, 4 months ago (2014-08-10 15:43:45 UTC) #4
commit-bot: I haz the power
6 years, 4 months ago (2014-08-10 15:47:25 UTC) #5
Message was sent while issue was closed. 
Change committed as 179893

Powered by Google App Engine
This is Rietveld 408576698