Skip to content

Commit a1dce64

Browse files
committed
Bug 1519636 - Reformat recent changes to the Google coding style r=Ehsan
# ignore-this-changeset Differential Revision: https://phabricator.services.mozilla.com/D28329 --HG-- extra : moz-landing-system : lando
1 parent c518481 commit a1dce64

File tree

67 files changed

+1333
-1382
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

67 files changed

+1333
-1382
lines changed

accessible/base/nsAccessibilityService.cpp

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1395,7 +1395,8 @@ nsAccessibilityService::CreateAccessibleByFrameType(nsIFrame* aFrame,
13951395
table = aContext->Parent();
13961396

13971397
if (table) {
1398-
nsIContent* parentContent = aContent->GetParentOrHostNode()->AsContent();
1398+
nsIContent* parentContent =
1399+
aContent->GetParentOrHostNode()->AsContent();
13991400
nsIFrame* parentFrame = nullptr;
14001401
if (parentContent) {
14011402
parentFrame = parentContent->GetPrimaryFrame();

browser/app/winlauncher/DllBlocklistWin.cpp

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -346,15 +346,14 @@ LauncherVoidResult InitializeDllBlocklistOOP(const wchar_t* aFullImagePath,
346346

347347
// As part of our mitigation of binary tampering, copy our import directory
348348
// from the original in our executable file.
349-
LauncherVoidResult importDirRestored =
350-
RestoreImportDirectory(aFullImagePath, ourExeImage, aChildProcess,
351-
ourModule);
349+
LauncherVoidResult importDirRestored = RestoreImportDirectory(
350+
aFullImagePath, ourExeImage, aChildProcess, ourModule);
352351
if (importDirRestored.isErr()) {
353352
return importDirRestored;
354353
}
355354

356355
Maybe<nt::PEHeaders::IATThunks> ntdllThunks =
357-
ourExeImage.GetIATThunksForModule("ntdll.dll");
356+
ourExeImage.GetIATThunksForModule("ntdll.dll");
358357
if (!ntdllThunks) {
359358
return LAUNCHER_ERROR_FROM_WIN32(ERROR_INVALID_DATA);
360359
}

browser/app/winlauncher/LauncherProcessWin.cpp

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -41,10 +41,9 @@
4141
*
4242
* @return Ok if browser startup should proceed
4343
*/
44-
static mozilla::LauncherVoidResult PostCreationSetup(const wchar_t* aFullImagePath,
45-
HANDLE aChildProcess,
46-
HANDLE aChildMainThread,
47-
const bool aIsSafeMode) {
44+
static mozilla::LauncherVoidResult PostCreationSetup(
45+
const wchar_t* aFullImagePath, HANDLE aChildProcess,
46+
HANDLE aChildMainThread, const bool aIsSafeMode) {
4847
// The launcher process's DLL blocking code is incompatible with ASAN because
4948
// it is able to execute before ASAN itself has even initialized.
5049
// Also, the AArch64 build doesn't yet have a working interceptor.
@@ -332,9 +331,8 @@ Maybe<int> LauncherMain(int& argc, wchar_t* argv[],
332331
nsAutoHandle process(pi.hProcess);
333332
nsAutoHandle mainThread(pi.hThread);
334333

335-
LauncherVoidResult setupResult =
336-
PostCreationSetup(argv[0], process.get(), mainThread.get(),
337-
isSafeMode.value());
334+
LauncherVoidResult setupResult = PostCreationSetup(
335+
argv[0], process.get(), mainThread.get(), isSafeMode.value());
338336
if (setupResult.isErr()) {
339337
HandleLauncherError(setupResult);
340338
::TerminateProcess(process.get(), 1);

caps/OriginAttributes.cpp

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -268,7 +268,8 @@ class MOZ_STACK_CLASS PopulateFromSuffixIterator final
268268
}
269269

270270
if (aName.EqualsLiteral("geckoViewUserContextId")) {
271-
MOZ_RELEASE_ASSERT(mOriginAttributes->mGeckoViewSessionContextId.IsEmpty());
271+
MOZ_RELEASE_ASSERT(
272+
mOriginAttributes->mGeckoViewSessionContextId.IsEmpty());
272273
mOriginAttributes->mGeckoViewSessionContextId.Assign(aValue);
273274
return true;
274275
}

caps/OriginAttributes.h

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -156,7 +156,8 @@ class OriginAttributesPattern : public dom::OriginAttributesPatternDictionary {
156156
}
157157

158158
if (mGeckoViewSessionContextId.WasPassed() &&
159-
mGeckoViewSessionContextId.Value() != aAttrs.mGeckoViewSessionContextId) {
159+
mGeckoViewSessionContextId.Value() !=
160+
aAttrs.mGeckoViewSessionContextId) {
160161
return false;
161162
}
162163

@@ -191,8 +192,10 @@ class OriginAttributesPattern : public dom::OriginAttributesPatternDictionary {
191192
return false;
192193
}
193194

194-
if (mGeckoViewSessionContextId.WasPassed() && aOther.mGeckoViewSessionContextId.WasPassed() &&
195-
mGeckoViewSessionContextId.Value() != aOther.mGeckoViewSessionContextId.Value()) {
195+
if (mGeckoViewSessionContextId.WasPassed() &&
196+
aOther.mGeckoViewSessionContextId.WasPassed() &&
197+
mGeckoViewSessionContextId.Value() !=
198+
aOther.mGeckoViewSessionContextId.Value()) {
196199
return false;
197200
}
198201

caps/nsJSPrincipals.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -289,8 +289,8 @@ static bool ReadPrincipalInfo(JSStructuredCloneReader* aReader, uint32_t aTag,
289289

290290
static StaticRefPtr<nsIPrincipal> sActiveWorkerPrincipal;
291291

292-
nsJSPrincipals::AutoSetActiveWorkerPrincipal::AutoSetActiveWorkerPrincipal
293-
(nsIPrincipal* aPrincipal) {
292+
nsJSPrincipals::AutoSetActiveWorkerPrincipal::AutoSetActiveWorkerPrincipal(
293+
nsIPrincipal* aPrincipal) {
294294
MOZ_ASSERT(NS_IsMainThread());
295295
MOZ_RELEASE_ASSERT(!sActiveWorkerPrincipal);
296296
sActiveWorkerPrincipal = aPrincipal;

caps/tests/gtest/TestOriginAttributes.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
using mozilla::OriginAttributes;
1010
using mozilla::Preferences;
1111

12-
static void TestSuffix(const OriginAttributes& attrs) {
12+
static void TestSuffix(const OriginAttributes &attrs) {
1313
nsAutoCString suffix;
1414
attrs.CreateSuffix(suffix);
1515

dom/base/DocumentOrShadowRoot.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -361,8 +361,7 @@ void DocumentOrShadowRoot::NodesFromRect(float aX, float aY, float aTopSize,
361361
float aRightSize, float aBottomSize,
362362
float aLeftSize,
363363
bool aIgnoreRootScrollFrame,
364-
bool aFlushLayout,
365-
bool aOnlyVisible,
364+
bool aFlushLayout, bool aOnlyVisible,
366365
nsTArray<RefPtr<nsINode>>& aReturn) {
367366
// Following the same behavior of elementFromPoint,
368367
// we don't return anything if either coord is negative

dom/base/SelectionChangeEventDispatcher.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ namespace mozilla {
2020
namespace dom {
2121
class Document;
2222
class Selection;
23-
}
23+
} // namespace dom
2424

2525
class SelectionChangeEventDispatcher final {
2626
public:

dom/base/nsDOMWindowUtils.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1166,8 +1166,7 @@ nsDOMWindowUtils::NodesFromRect(float aX, float aY, float aTopSize,
11661166

11671167
AutoTArray<RefPtr<nsINode>, 8> nodes;
11681168
doc->NodesFromRect(aX, aY, aTopSize, aRightSize, aBottomSize, aLeftSize,
1169-
aIgnoreRootScrollFrame, aFlushLayout, aOnlyVisible,
1170-
nodes);
1169+
aIgnoreRootScrollFrame, aFlushLayout, aOnlyVisible, nodes);
11711170
list->SetCapacity(nodes.Length());
11721171
for (auto& node : nodes) {
11731172
list->AppendElement(node->AsContent());

0 commit comments

Comments
 (0)