commit 8b17a2eaeabf6c4fe7d2a3ab8901f5d6e48c8a11
author Karl Wiberg <kwiberg@chromium.org> Mon Sep 25 00:15:05 2017 +0000
committer Commit Bot <commit-bot@chromium.org> Mon Sep 25 00:15:05 2017 +0000
tree 5c3d0d6797ab5a30e21f71177c3f03404b0c35c4
parent ae45b1afe70af871895ff6950f60e394c77164df

gn format: Sort `public` just like we sort `sources`

Since both of them list source files.

Change-Id: I1eb2516ffb0fe1efa8208562384a620eb45903dd
Reviewed-on: https://chromium-review.googlesource.com/674877
Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Reviewed-by: Scott Graham <scottmg@chromium.org>
Commit-Queue: Karl Wiberg <kwiberg@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#503965}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 7f2385e9798276d049f7a0fb90bed734b5545643

tools/gn/command_format.cc

diff --git a/tools/gn/command_format.cc b/tools/gn/command_format.cc
index 0849762..2254317 100644
--- a/tools/gn/command_format.cc
+++ b/tools/gn/command_format.cc
@@ -336,7 +336,7 @@
        binop->op().value() == "-=") &&
       ident && list) {
     const base::StringPiece lhs = ident->value().value();
-    if (lhs == "sources")
+    if (lhs == "public" || lhs == "sources")
       const_cast<ListNode*>(list)->SortAsStringsList();
     else if (lhs == "deps" || lhs == "public_deps")
       const_cast<ListNode*>(list)->SortAsDepsList();