Closed Bug 837622 Opened 11 years ago Closed 11 years ago

Wrong logic when handling "Webapps:OfflineCache" (follow-up for bug 702369)

Categories

(Core :: DOM: Core & HTML, defect)

Product:
Core
Component:
DOM: Core & HTML
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED WONTFIX

People

(Reporter: airpingu, Assigned: airpingu)

Details

+      case "Webapps:OfflineCache":
+        if (msg.manifest != this.manifestURL)
+          return;

Why not |msg.manifestURL != this.manifestURL|? I'm not sure if it would potentially cause any bugs but it'd be better to correct it when seeing it.
All right... I misunderstood. Sorry for the noise! Well, it'd be better to differentiate |manifest| and |manifestURL| for terminology since they're totally different concepts.
No longer blocks: 702369
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → WONTFIX
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.