Bug 1245493 - Add comments to existing code to clarify them. r=margaret draft
authorMichael Comella <michael.l.comella@gmail.com>
Mon, 28 Mar 2016 18:41:34 -0700
changeset 357999 c91e633f84b5b8444061fd04a34bd6b17b09abd3
parent 357423 4dc48f3688f053e2fa74bf6c7beb4dff274ffc1f
child 358000 19e538fb2cc16e51ca3ad89a25014766360283fc
child 358039 2de6f3c81ea45b629732cbc28016415bd7630a51
child 358040 803cc3e6f940462168a61f0a12b32a0391611caa
push id16904
push usermichael.l.comella@gmail.com
push dateFri, 29 Apr 2016 22:55:58 +0000
reviewersmargaret
bugs1245493
milestone49.0a1
Bug 1245493 - Add comments to existing code to clarify them. r=margaret MozReview-Commit-ID: KwOy1KMoPaY
mobile/android/base/java/org/mozilla/gecko/Tab.java
mobile/android/base/java/org/mozilla/gecko/Tabs.java
--- a/mobile/android/base/java/org/mozilla/gecko/Tab.java
+++ b/mobile/android/base/java/org/mozilla/gecko/Tab.java
@@ -59,16 +59,18 @@ public class Tab {
     // The set of all available Favicons for this tab, sorted by attractiveness.
     final TreeSet<RemoteFavicon> mAvailableFavicons = new TreeSet<>();
     private boolean mHasFeeds;
     private boolean mHasOpenSearch;
     private final SiteIdentity mSiteIdentity;
     private SiteLogins mSiteLogins;
     private BitmapDrawable mThumbnail;
     private final int mParentId;
+    // Indicates the url was loaded from a source external to the app. This will be cleared
+    // when the user explicitly loads a new url (e.g. clicking a link is not explicit).
     private final boolean mExternal;
     private boolean mBookmark;
     private int mFaviconLoadId;
     private String mContentType;
     private boolean mHasTouchListeners;
     private ZoomConstraints mZoomConstraints;
     private boolean mIsRTL;
     private final ArrayList<View> mPluginViews;
--- a/mobile/android/base/java/org/mozilla/gecko/Tabs.java
+++ b/mobile/android/base/java/org/mozilla/gecko/Tabs.java
@@ -58,16 +58,17 @@ public class Tabs implements GeckoEventL
     public static final int LOADURL_NONE         = 0;
     public static final int LOADURL_NEW_TAB      = 1 << 0;
     public static final int LOADURL_USER_ENTERED = 1 << 1;
     public static final int LOADURL_PRIVATE      = 1 << 2;
     public static final int LOADURL_PINNED       = 1 << 3;
     public static final int LOADURL_DELAY_LOAD   = 1 << 4;
     public static final int LOADURL_DESKTOP      = 1 << 5;
     public static final int LOADURL_BACKGROUND   = 1 << 6;
+    /** Indicates the url has been specified by a source external to the app. */
     public static final int LOADURL_EXTERNAL     = 1 << 7;
 
     private static final long PERSIST_TABS_AFTER_MILLISECONDS = 1000 * 2;
 
     public static final int INVALID_TAB_ID = -1;
 
     private static final AtomicInteger sTabId = new AtomicInteger(0);
     private volatile boolean mInitialTabsAdded;